[FFmpeg-cvslog] avformat/utils: fix memory leak in avformat_free_context

Steven Siloti git at videolan.org
Sun Aug 6 00:24:28 EEST 2017


ffmpeg | branch: master | Steven Siloti <ssiloti at bittorrent.com> | Tue Jul 18 11:26:39 2017 -0700| [949debd1d1df3a96315b3a3083831162845c1188] | committer: Michael Niedermayer

avformat/utils: fix memory leak in avformat_free_context

The pointer to the packet queue is stored in the internal structure
so the queue needs to be flushed before internal is freed.

Signed-off-by: Steven Siloti <ssiloti at bittorrent.com>
Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=949debd1d1df3a96315b3a3083831162845c1188
---

 libavformat/utils.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libavformat/utils.c b/libavformat/utils.c
index 38d247c6cd..58283616dc 100644
--- a/libavformat/utils.c
+++ b/libavformat/utils.c
@@ -4333,8 +4333,8 @@ void avformat_free_context(AVFormatContext *s)
     av_dict_free(&s->metadata);
     av_dict_free(&s->internal->id3v2_meta);
     av_freep(&s->streams);
-    av_freep(&s->internal);
     flush_packet_queue(s);
+    av_freep(&s->internal);
     av_free(s);
 }
 



More information about the ffmpeg-cvslog mailing list