[FFmpeg-cvslog] avformat/mov: fix memleaks

Zhao Zhili git at videolan.org
Wed Jul 1 16:54:04 EEST 2020


ffmpeg | branch: release/3.2 | Zhao Zhili <quinkblack at foxmail.com> | Sun Jun 28 11:15:39 2020 +0800| [bd51c9dfaf82e9789273b6524cd61647e45c9fcc] | committer: Andreas Rheinhardt

avformat/mov: fix memleaks

Fix two cases of memleaks:
1. The leak of dv_demux
2. The leak of dv_fctx upon dv_demux allocate failure

Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at gmail.com>
(cherry picked from commit f3dc38a186b2326ce03e50969897ea703817ddb0)
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at gmail.com>

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

 libavformat/mov.c | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/libavformat/mov.c b/libavformat/mov.c
index e01fe1a137..c9740bfa2b 100644
--- a/libavformat/mov.c
+++ b/libavformat/mov.c
@@ -5378,10 +5378,9 @@ static int mov_read_close(AVFormatContext *s)
         av_aes_ctr_free(sc->cenc.aes_ctr);
     }
 
-    if (mov->dv_demux) {
-        avformat_free_context(mov->dv_fctx);
-        mov->dv_fctx = NULL;
-    }
+    av_freep(&mov->dv_demux);
+    avformat_free_context(mov->dv_fctx);
+    mov->dv_fctx = NULL;
 
     if (mov->meta_keys) {
         for (i = 1; i < mov->meta_keys_count; i++) {



More information about the ffmpeg-cvslog mailing list