diff options
author | Vladimir Voroshilov <voroshil@gmail.com> | 2007-10-13 21:38:58 +0400 |
---|---|---|
committer | Vladimir Voroshilov <voroshil@gmail.com> | 2007-10-13 21:38:58 +0400 |
commit | 3c3704d28179e533e2f42a20e3234827b3f52685 (patch) | |
tree | e94a57ec72b323f890ba4ac1cea38d987333791a | |
parent | 4d570f94ba055d75f469aacadfccf0bdffcbae6c (diff) |
Fix MJPEG decoder for AMV files.
Since decoding is doing from the end and aligned by 16
previous code worked correctly only when picture height was dividable by 16,
otherwise it provides garbage in top lines and truncates bottom.
New code adjusts data[] pointers taking in account alignment issue.
Originally committed as revision 10727 to svn://svn.ffmpeg.org/ffmpeg/trunk
-rw-r--r-- | libavcodec/mjpegdec.c | 3 | ||||
-rw-r--r-- | libavcodec/sp5xdec.c | 4 |
2 files changed, 4 insertions, 3 deletions
diff --git a/libavcodec/mjpegdec.c b/libavcodec/mjpegdec.c index d1d81d3493..ea9e4ccf55 100644 --- a/libavcodec/mjpegdec.c +++ b/libavcodec/mjpegdec.c @@ -671,7 +671,8 @@ static int mjpeg_decode_scan(MJpegDecodeContext *s, int nb_components, int ss, i linesize[c]=s->linesize[c]; if(s->avctx->codec->id==CODEC_ID_AMV) { //picture should be flipped upside-down for this codec - data[c] += (linesize[c] * (s->v_scount[i] * 8 * s->mb_height - 1)); + assert(!(s->avctx->flags & CODEC_FLAG_EMU_EDGE)); + data[c] += (linesize[c] * (s->v_scount[i] * (8 * s->mb_height -((s->height/s->v_max)&7)) - 1 )); linesize[c] *= -1; } } diff --git a/libavcodec/sp5xdec.c b/libavcodec/sp5xdec.c index ada46ca353..104ed31b7b 100644 --- a/libavcodec/sp5xdec.c +++ b/libavcodec/sp5xdec.c @@ -87,6 +87,7 @@ static int sp5x_decode_frame(AVCodecContext *avctx, recoded[j++] = 0xFF; recoded[j++] = 0xD9; + avctx->flags &= ~CODEC_FLAG_EMU_EDGE; i = ff_mjpeg_decode_frame(avctx, data, data_size, recoded, j); av_free(recoded); @@ -207,6 +208,5 @@ AVCodec amv_decoder = { ff_mjpeg_decode_init, NULL, ff_mjpeg_decode_end, - sp5x_decode_frame, - CODEC_CAP_DR1 + sp5x_decode_frame }; |