summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMichael Niedermayer <michaelni@gmx.at>2014-08-08 13:51:59 +0200
committerMichael Niedermayer <michaelni@gmx.at>2014-08-08 13:55:06 +0200
commitc074feed292eb1d9371aaa3c63136d4cb241f8c5 (patch)
tree83b56ed20f6c47ab39fd380ee4c9abf1b0cb301a
parent53c3abc1083d630cb50427e09783cf6ce84d4a45 (diff)
parente8ff7972064631afbdf240ec6bfd9dec30cf2ce8 (diff)
downloadffmpeg-c074feed292eb1d9371aaa3c63136d4cb241f8c5.tar.gz
Merge commit 'e8ff7972064631afbdf240ec6bfd9dec30cf2ce8' into release/1.1
* commit 'e8ff7972064631afbdf240ec6bfd9dec30cf2ce8': eamad: use the bytestream2 API instead of AV_RL Conflicts: libavcodec/eamad.c Merged-by: Michael Niedermayer <michaelni@gmx.at>
-rw-r--r--libavcodec/eamad.c41
1 files changed, 22 insertions, 19 deletions
diff --git a/libavcodec/eamad.c b/libavcodec/eamad.c
index e71bf9adb3..bca038f826 100644
--- a/libavcodec/eamad.c
+++ b/libavcodec/eamad.c
@@ -29,6 +29,7 @@
*/
#include "avcodec.h"
+#include "bytestream.h"
#include "get_bits.h"
#include "dsputil.h"
#include "aandcttab.h"
@@ -232,32 +233,34 @@ static int decode_frame(AVCodecContext *avctx,
{
const uint8_t *buf = avpkt->data;
int buf_size = avpkt->size;
- const uint8_t *buf_end = buf+buf_size;
MadContext *s = avctx->priv_data;
+ GetByteContext gb;
int width, height, ret;
int chunk_type;
int inter;
- if (buf_size < 26) {
- av_log(avctx, AV_LOG_ERROR, "Input buffer too small\n");
- *got_frame = 0;
- return AVERROR_INVALIDDATA;
- }
+ bytestream2_init(&gb, buf, buf_size);
- chunk_type = AV_RL32(&buf[0]);
+ chunk_type = bytestream2_get_le32(&gb);
inter = (chunk_type == MADm_TAG || chunk_type == MADe_TAG);
- buf += 8;
+ bytestream2_skip(&gb, 10);
av_reduce(&avctx->time_base.num, &avctx->time_base.den,
- AV_RL16(&buf[6]), 1000, 1<<30);
+ bytestream2_get_le16(&gb), 1000, 1<<30);
+
+ width = bytestream2_get_le16(&gb);
+ height = bytestream2_get_le16(&gb);
+ bytestream2_skip(&gb, 1);
+ calc_quant_matrix(s, bytestream2_get_byte(&gb));
+ bytestream2_skip(&gb, 2);
- width = AV_RL16(&buf[8]);
- height = AV_RL16(&buf[10]);
- calc_quant_matrix(s, buf[13]);
- buf += 16;
+ if (bytestream2_get_bytes_left(&gb) < 2) {
+ av_log(avctx, AV_LOG_ERROR, "Input data too small\n");
+ return AVERROR_INVALIDDATA;
+ }
if (avctx->width != width || avctx->height != height) {
- if((width * height)/2048*7 > buf_end-buf)
+ if((width * height)/2048*7 > bytestream2_get_bytes_left(&gb))
return AVERROR_INVALIDDATA;
if ((ret = av_image_check_size(width, height, 0, avctx)) < 0)
return ret;
@@ -292,13 +295,13 @@ static int decode_frame(AVCodecContext *avctx,
}
av_fast_padded_malloc(&s->bitstream_buf, &s->bitstream_buf_size,
- buf_end - buf);
+ bytestream2_get_bytes_left(&gb));
if (!s->bitstream_buf)
return AVERROR(ENOMEM);
- s->dsp.bswap16_buf(s->bitstream_buf, (const uint16_t*)buf, (buf_end-buf)/2);
- memset((uint8_t*)s->bitstream_buf + (buf_end-buf), 0, FF_INPUT_BUFFER_PADDING_SIZE);
- init_get_bits(&s->gb, s->bitstream_buf, 8*(buf_end-buf));
-
+ s->dsp.bswap16_buf(s->bitstream_buf, (const uint16_t *)(buf + bytestream2_tell(&gb)),
+ bytestream2_get_bytes_left(&gb) / 2);
+ memset((uint8_t*)s->bitstream_buf + bytestream2_get_bytes_left(&gb), 0, FF_INPUT_BUFFER_PADDING_SIZE);
+ init_get_bits(&s->gb, s->bitstream_buf, 8*(bytestream2_get_bytes_left(&gb)));
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++)
if(decode_mb(s, inter) < 0)