On Tue, 2018-05-08 at 00:11 +0100, Mark Thompson wrote:
> The user should only interact directly with the data length, not the
> payload size.
> ---
> libavcodec/cbs_h264_syntax_template.c | 7 +++++--
> 1 file changed, 5 insertions(+), 2 deletions(-)
>
> diff --git a/libavcodec/cbs_h264_syntax_template.c
> b/libavcodec/cbs_h264_syntax_template.c
> index 3ec4299915..b789207e33 100644
> --- a/libavcodec/cbs_h264_syntax_template.c
> +++ b/libavcodec/cbs_h264_syntax_template.c
> @@ -761,8 +761,11 @@ static int FUNC(sei_payload)(CodedBitstreamContext *ctx,
> RWContext *rw,
> break;
> default:
> {
> - allocate(current->payload.other.data, current->payload_size);
> - for (i = 0; i < current->payload_size; i++)
> +#ifdef READ
> + current->payload.other.data_length = current->payload_size;
> +#endif
> + allocate(current->payload.other.data, current-
> >payload.other.data_length);
> + for (i = 0; i < current->payload.other.data_length; i++)
> xu(8, payload_byte[i], current->payload.other.data[i], 0,
> 255, 1, i);
> }
> }
LGTM, I will need to fix the H265 one as well.
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
