diff options
author | Derek Buitenhuis <derek.buitenhuis@gmail.com> | 2016-04-17 19:32:26 +0100 |
---|---|---|
committer | Derek Buitenhuis <derek.buitenhuis@gmail.com> | 2016-04-17 19:32:26 +0100 |
commit | 61400a1dc751c82cf6f33a7510c3804b7b201f25 (patch) | |
tree | d1d1e0f627418472e93ede72627e07a356ac62cf /libavcodec/vc1dec.c | |
parent | f64b53f2bf5f3f43402564bf57a8555a709d012c (diff) | |
parent | d909f43b5c773a73c8d526638744547ba4aa8c59 (diff) | |
download | ffmpeg-61400a1dc751c82cf6f33a7510c3804b7b201f25.tar.gz |
Merge commit 'd909f43b5c773a73c8d526638744547ba4aa8c59'
* commit 'd909f43b5c773a73c8d526638744547ba4aa8c59':
vc1dec: wmv2dec: Validate ff_intrax8_common_init return value
Merged-by: Derek Buitenhuis <derek.buitenhuis@gmail.com>
Diffstat (limited to 'libavcodec/vc1dec.c')
-rw-r--r-- | libavcodec/vc1dec.c | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/libavcodec/vc1dec.c b/libavcodec/vc1dec.c index 75d069dd39..dae697bd9c 100644 --- a/libavcodec/vc1dec.c +++ b/libavcodec/vc1dec.c @@ -324,7 +324,7 @@ static void vc1_sprite_flush(AVCodecContext *avctx) av_cold int ff_vc1_decode_init_alloc_tables(VC1Context *v) { MpegEncContext *s = &v->s; - int i; + int i, ret = AVERROR(ENOMEM); int mb_height = FFALIGN(s->mb_height, 2); /* Allocate mb bitplanes */ @@ -381,19 +381,21 @@ av_cold int ff_vc1_decode_init_alloc_tables(VC1Context *v) v->mv_f_next[0] = v->mv_f_next_base + s->b8_stride + 1; v->mv_f_next[1] = v->mv_f_next[0] + (s->b8_stride * (mb_height * 2 + 1) + s->mb_stride * (mb_height + 1) * 2); - 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 AVERROR(ENOMEM); } + ret = ff_intrax8_common_init(&v->x8, s); + if (ret < 0) + goto error; + return 0; error: ff_vc1_decode_end(s->avctx); - return AVERROR(ENOMEM); + return ret; } av_cold void ff_vc1_init_transposed_scantables(VC1Context *v) |