diff options
| author | Michael Niedermayer <michaelni@gmx.at> | 2011-12-19 04:13:37 +0100 |
|---|---|---|
| committer | Reinhard Tartler <siretart@tauware.de> | 2012-05-23 20:42:46 +0200 |
| commit | 90290a5150e84fb138ccde57657dc03830f08c1c (patch) | |
| tree | 24a8c859520bd44fa53a13de0f1da0a0811c6dc9 | |
| parent | 3f9d6e423978f5e905def374e9c2e9166e3ebb2c (diff) | |
| download | ffmpeg-90290a5150e84fb138ccde57657dc03830f08c1c.tar.gz | |
tqi: Pass errors from the MB decoder
This silences some valgrind warnings.
CC: libav-stable@libav.org
Fixes second half of http://ffmpeg.org/trac/ffmpeg/ticket/794
Bug found by: Oana Stratulat
Signed-off-by: Michael Niedermayer <michaelni@gmx.at>
Signed-off-by: Reinhard Tartler <siretart@tauware.de>
(cherry picked from commit f85334f58e1286287d0547a49fa9c93b40cbf48f)
| -rw-r--r-- | libavcodec/eatqi.c | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/libavcodec/eatqi.c b/libavcodec/eatqi.c index 2d7da5a603..9824940d97 100644 --- a/libavcodec/eatqi.c +++ b/libavcodec/eatqi.c @@ -57,12 +57,15 @@ static av_cold int tqi_decode_init(AVCodecContext *avctx) return 0; } -static void tqi_decode_mb(MpegEncContext *s, DCTELEM (*block)[64]) +static int tqi_decode_mb(MpegEncContext *s, DCTELEM (*block)[64]) { int n; s->dsp.clear_blocks(block[0]); for (n=0; n<6; n++) - ff_mpeg1_decode_block_intra(s, block[n], n); + if (ff_mpeg1_decode_block_intra(s, block[n], n) < 0) + return -1; + + return 0; } static inline void tqi_idct_put(TqiContext *t, DCTELEM (*block)[64]) @@ -135,7 +138,8 @@ static int tqi_decode_frame(AVCodecContext *avctx, for (s->mb_y=0; s->mb_y<(avctx->height+15)/16; s->mb_y++) for (s->mb_x=0; s->mb_x<(avctx->width+15)/16; s->mb_x++) { - tqi_decode_mb(s, t->block); + if (tqi_decode_mb(s, t->block) < 0) + break; tqi_idct_put(t, t->block); } |