James Almer:
> FFFormatContext.packet_buffer is used in mux.c, and if a muxing process fails 
> at a
> point where packets remained in said queue, they will leak.
> 
> Fixes ticket #11419
> 
> Signed-off-by: James Almer <jamr...@gmail.com>
> ---
>  libavformat/avformat.c | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
> 
> diff --git a/libavformat/avformat.c b/libavformat/avformat.c
> index eb9afad837..10aad7bb93 100644
> --- a/libavformat/avformat.c
> +++ b/libavformat/avformat.c
> @@ -192,8 +192,7 @@ void avformat_free_context(AVFormatContext *s)
>      av_packet_free(&si->parse_pkt);
>      av_freep(&s->streams);
>      av_freep(&s->stream_groups);
> -    if (s->iformat)
> -        ff_flush_packet_queue(s);
> +    ff_flush_packet_queue(s);
>      av_freep(&s->url);
>      av_free(s);
>  }

This is regression since cb80ec0b6cd7f954839427b4c86958d1475f2df4 which
should be mentioned in the commit message.

- Andreas

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Reply via email to