[FFmpeg-devel] [PATCH 4/17] avformat/segment: Don't set extradata size twice

Gyan Doshi ffmpeg at gyani.pro
Mon Mar 2 07:17:29 EET 2020



On 02-03-2020 10:05 am, Andreas Rheinhardt wrote:
> ff_alloc_extradata() already sets the size of the extradata so doing it
> again is unnecessary.
>
> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at gmail.com>
> ---
>   libavformat/segment.c | 1 -
>   1 file changed, 1 deletion(-)
>
> diff --git a/libavformat/segment.c b/libavformat/segment.c
> index 5f7fe76600..d565183349 100644
> --- a/libavformat/segment.c
> +++ b/libavformat/segment.c
> @@ -882,7 +882,6 @@ static int seg_write_packet(AVFormatContext *s, AVPacket *pkt)
>                   goto calc_times;
>               }
>               memcpy(st->codecpar->extradata, pkt_extradata, pkt_extradata_size);
> -            st->codecpar->extradata_size = pkt_extradata_size;
>           }
>       }
>   

LGTM.

Gyan


More information about the ffmpeg-devel mailing list