From 8b58f6b5c286ba14ec9c6c9e03f9ede8c6be9181 Mon Sep 17 00:00:00 2001 From: Michael Niedermayer Date: Tue, 27 Mar 2012 02:11:03 +0200 Subject: [PATCH] Revert "raw: move buffer size check up." This reverts commit cc5dd632cecc5114717d0b90f8c2be162b1c6ee8. The change was redundant, it has been fixed long ago (422e3a7) Conflicts: libavcodec/rawdec.c Signed-off-by: Michael Niedermayer --- libavcodec/rawdec.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/libavcodec/rawdec.c b/libavcodec/rawdec.c index b59224302a..f27df2ad2d 100644 --- a/libavcodec/rawdec.c +++ b/libavcodec/rawdec.c @@ -155,9 +155,6 @@ static int raw_decode(AVCodecContext *avctx, frame->top_field_first = context->tff; } - if(buf_size < context->length - (avctx->pix_fmt==PIX_FMT_PAL8 ? 256*4 : 0)) - return -1; - if (avctx->width <= 0 || avctx->height <= 0) { av_log(avctx, AV_LOG_ERROR, "w/h is invalid\n"); return AVERROR(EINVAL); @@ -190,6 +187,9 @@ static int raw_decode(AVCodecContext *avctx, avctx->codec_tag == MKTAG('A', 'V', 'u', 'p')) buf += buf_size - context->length; + if(buf_size < context->length - (avctx->pix_fmt==PIX_FMT_PAL8 ? 256*4 : 0)) + return -1; + avpicture_fill(picture, buf, avctx->pix_fmt, avctx->width, avctx->height); if((avctx->pix_fmt==PIX_FMT_PAL8 && buf_size < context->length) || (av_pix_fmt_descriptors[avctx->pix_fmt].flags & PIX_FMT_PSEUDOPAL)) { -- 2.11.0