[FFmpeg-devel] [PATCH 01/16] avformat/hlsenc: Fix leak of child AVFormatContext

Steven Liu lq at chinaffmpeg.org
Mon Dec 16 04:49:47 EET 2019



> 在 2019年12月16日,08:04,Andreas Rheinhardt <andreas.rheinhardt at gmail.com> 写道:
> 
> Before ed897633, the hls muxer would free its child AVFormatContexts
> and reset the pointer to these contexts to NULL immediately afterwards;
> ed897633 moved the former to later (into a separate function), but kept
> the resetting, ensuring that the child context leaks.
> 
> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at gmail.com>
> ---
> libavformat/hlsenc.c | 1 -
> 1 file changed, 1 deletion(-)
> 
> diff --git a/libavformat/hlsenc.c b/libavformat/hlsenc.c
> index 6f36a23cbe..2b87070d62 100644
> --- a/libavformat/hlsenc.c
> +++ b/libavformat/hlsenc.c
> @@ -2631,7 +2631,6 @@ failed:
>         }
>         ffio_free_dyn_buf(&oc->pb);
> 
> -        vs->avf = NULL;
>         av_free(old_filename);
>     }
> 
> -- 
> 2.20.1
> 
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> 
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".

LGTM

Thanks
Steven






More information about the ffmpeg-devel mailing list