[FFmpeg-devel] [PATCH 4/5] avformat/mpegts: fix av_freep for dovi pointer

Andreas Rheinhardt andreas.rheinhardt at gmail.com
Thu Apr 30 16:11:31 EEST 2020


lance.lmwang at gmail.com:
> From: Limin Wang <lance.lmwang at gmail.com>
> 
> Signed-off-by: Limin Wang <lance.lmwang at gmail.com>
> ---
>  libavformat/mpegts.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/libavformat/mpegts.c b/libavformat/mpegts.c
> index ff3898c3a5..3bd98e5743 100644
> --- a/libavformat/mpegts.c
> +++ b/libavformat/mpegts.c
> @@ -2180,7 +2180,7 @@ int ff_parse_mpeg2_descriptor(AVFormatContext *fc, AVStream *st, int stream_type
>              ret = av_stream_add_side_data(st, AV_PKT_DATA_DOVI_CONF,
>                                            (uint8_t *)dovi, dovi_size);
>              if (ret < 0) {
> -                av_freep(dovi);
> +                av_freep(&dovi);
>                  return ret;
>              }
>  
> 
LGTM. Same remark as for patch 3.

- Andreas


More information about the ffmpeg-devel mailing list