xmv: do not leak memory in the error paths in xmv_read_header()
authorAnton Khirnov <anton@khirnov.net>
Thu, 28 Mar 2013 09:34:47 +0000 (10:34 +0100)
committerAnton Khirnov <anton@khirnov.net>
Thu, 4 Apr 2013 05:54:45 +0000 (07:54 +0200)
CC: libav-stable@libav.org
libavformat/xmv.c

index e9b2b0d..bc7c3c9 100644 (file)
@@ -188,8 +188,10 @@ static int xmv_read_header(AVFormatContext *s)
         return AVERROR(ENOMEM);
 
     xmv->audio = av_malloc(xmv->audio_track_count * sizeof(XMVAudioPacket));
-    if (!xmv->audio)
-        return AVERROR(ENOMEM);
+    if (!xmv->audio) {
+        ret = AVERROR(ENOMEM);
+        goto fail;
+    }
 
     for (audio_track = 0; audio_track < xmv->audio_track_count; audio_track++) {
         XMVAudioTrack  *track  = &xmv->audio_tracks[audio_track];
@@ -230,8 +232,10 @@ static int xmv_read_header(AVFormatContext *s)
         }
 
         ast = avformat_new_stream(s, NULL);
-        if (!ast)
-            return AVERROR(ENOMEM);
+        if (!ast) {
+            ret = AVERROR(ENOMEM);
+            goto fail;
+        }
 
         ast->codec->codec_type            = AVMEDIA_TYPE_AUDIO;
         ast->codec->codec_id              = track->codec_id;