[FFmpeg-cvslog] oggdec: free the ogg streams on read_header failure

Luca Barbato git at videolan.org
Mon Jan 7 14:04:06 CET 2013


ffmpeg | branch: master | Luca Barbato <lu_zero at gentoo.org> | Fri Jan  4 15:44:02 2013 +0100| [89b51b570daa80e6e3790fcd449fe61fc5574e07] | committer: Luca Barbato

oggdec: free the ogg streams on read_header failure

Plug an annoying memory leak on broken files.

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

 libavformat/oggdec.c |   30 ++++++++++++++++--------------
 1 file changed, 16 insertions(+), 14 deletions(-)

diff --git a/libavformat/oggdec.c b/libavformat/oggdec.c
index 07af5cf..a6cd31c 100644
--- a/libavformat/oggdec.c
+++ b/libavformat/oggdec.c
@@ -526,6 +526,19 @@ static int ogg_get_length(AVFormatContext *s)
     return 0;
 }
 
+static int ogg_read_close(AVFormatContext *s)
+{
+    struct ogg *ogg = s->priv_data;
+    int i;
+
+    for (i = 0; i < ogg->nstreams; i++) {
+        av_free(ogg->streams[i].buf);
+        av_free(ogg->streams[i].private);
+    }
+    av_free(ogg->streams);
+    return 0;
+}
+
 static int ogg_read_header(AVFormatContext *s)
 {
     struct ogg *ogg = s->priv_data;
@@ -533,8 +546,10 @@ static int ogg_read_header(AVFormatContext *s)
     ogg->curidx = -1;
     //linear headers seek from start
     ret = ogg_get_headers(s);
-    if (ret < 0)
+    if (ret < 0) {
+        ogg_read_close(s);
         return ret;
+    }
 
     for (i = 0; i < ogg->nstreams; i++)
         if (ogg->streams[i].header < 0)
@@ -619,19 +634,6 @@ retry:
     return psize;
 }
 
-static int ogg_read_close(AVFormatContext *s)
-{
-    struct ogg *ogg = s->priv_data;
-    int i;
-
-    for (i = 0; i < ogg->nstreams; i++) {
-        av_free(ogg->streams[i].buf);
-        av_free(ogg->streams[i].private);
-    }
-    av_free(ogg->streams);
-    return 0;
-}
-
 static int64_t ogg_read_timestamp(AVFormatContext *s, int stream_index,
                                   int64_t *pos_arg, int64_t pos_limit)
 {



More information about the ffmpeg-cvslog mailing list