On Sat, 5 Jul 2014, Anton Khirnov wrote:

---
libavformat/mov.c | 8 ++------
1 file changed, 2 insertions(+), 6 deletions(-)

diff --git a/libavformat/mov.c b/libavformat/mov.c
index 4a2d265..d8993fa 100644
--- a/libavformat/mov.c
+++ b/libavformat/mov.c
@@ -3041,12 +3041,8 @@ static int mov_read_close(AVFormatContext *s)
    }

    if (mov->dv_demux) {
-        for (i = 0; i < mov->dv_fctx->nb_streams; i++) {
-            av_freep(&mov->dv_fctx->streams[i]->codec);
-            av_freep(&mov->dv_fctx->streams[i]);
-        }
-        av_freep(&mov->dv_fctx);
-        av_freep(&mov->dv_demux);
+        avformat_free_context(mov->dv_fctx);
+        mov->dv_fctx = NULL;
    }

    av_freep(&mov->trex_data);
--
2.0.0

Probably ok. Does fate exercise this piece of code, did you check it with valgrind that nothing bad happens?

// Martin
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to