summaryrefslogtreecommitdiffstats
path: root/libavcodec
diff options
context:
space:
mode:
authorMichael Niedermayer <michael@niedermayer.cc>2019-01-23 00:19:14 +0100
committerMichael Niedermayer <michael@niedermayer.cc>2019-01-31 00:24:38 +0100
commit9f4af97aff899571663342fbe68df8caee30097f (patch)
treeacd26b56aacc3a5d0d8f970e2682a92c1a26f0a0 /libavcodec
parentf4079d5174c20eddbc99eef6ebe98d411f8014c5 (diff)
downloadffmpeg-streaming-9f4af97aff899571663342fbe68df8caee30097f.zip
ffmpeg-streaming-9f4af97aff899571663342fbe68df8caee30097f.tar.gz
avcodec/rasc: Move ff_get_buffer() after frame checks
If the frame1/2 checks fail this avoids doing the allocation of a new frame Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
Diffstat (limited to 'libavcodec')
-rw-r--r--libavcodec/rasc.c6
1 files changed, 3 insertions, 3 deletions
diff --git a/libavcodec/rasc.c b/libavcodec/rasc.c
index 6e32c15..21fc43f 100644
--- a/libavcodec/rasc.c
+++ b/libavcodec/rasc.c
@@ -723,12 +723,12 @@ static int decode_frame(AVCodecContext *avctx,
return ret;
}
- if ((ret = ff_get_buffer(avctx, s->frame, 0)) < 0)
- return ret;
-
if (!s->frame2->data[0] || !s->frame1->data[0])
return AVERROR_INVALIDDATA;
+ if ((ret = ff_get_buffer(avctx, s->frame, 0)) < 0)
+ return ret;
+
copy_plane(avctx, s->frame2, s->frame);
if (avctx->pix_fmt == AV_PIX_FMT_PAL8)
memcpy(s->frame->data[1], s->frame2->data[1], 1024);
OpenPOWER on IntegriCloud