vc1dec: Fix leak on error for array allocations
authorVittorio Giovara <vittorio.giovara@gmail.com>
Wed, 24 Feb 2016 19:40:18 +0000 (14:40 -0500)
committerVittorio Giovara <vittorio.giovara@gmail.com>
Thu, 25 Feb 2016 20:20:53 +0000 (15:20 -0500)
The deinit function in the 'error' section will correctly free
everything.

libavcodec/vc1dec.c

index 2ccbbd5..9d0f890 100644 (file)
@@ -355,8 +355,11 @@ av_cold int ff_vc1_decode_init_alloc_tables(VC1Context *v)
     ff_intrax8_common_init(&v->x8,s);
 
     if (s->avctx->codec_id == AV_CODEC_ID_WMV3IMAGE || s->avctx->codec_id == AV_CODEC_ID_VC1IMAGE) {
-        for (i = 0; i < 4; i++)
-            if (!(v->sr_rows[i >> 1][i & 1] = av_malloc(v->output_width))) return -1;
+        for (i = 0; i < 4; i++) {
+            v->sr_rows[i >> 1][i & 1] = av_malloc(v->output_width);
+            if (!v->sr_rows[i >> 1][i & 1])
+                goto error;
+        }
     }
 
     if (!v->mv_type_mb_plane || !v->direct_mb_plane || !v->acpred_plane || !v->over_flags_plane ||