aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMichael Niedermayer <michael@niedermayer.cc>2018-05-05 22:00:01 +0200
committerMichael Niedermayer <michael@niedermayer.cc>2018-07-16 19:06:27 +0200
commit38fd80c656b4550ac6a7aef0bf3d83027784226f (patch)
treee4895510f2c005574605e25ea4e92d18a6f65588
parentd48738bd674f70131df4581936ef0125f1c26c5c (diff)
downloadffmpeg-38fd80c656b4550ac6a7aef0bf3d83027784226f.tar.gz
avcodec/fic: Check available input space for cursor
Fixes: out of array read Fixes: 6546/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_FIC_fuzzer-6317064647081984 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer <michael@niedermayer.cc> (cherry picked from commit cb2f7ea96b4f6e03ebf0c0563677745fc65f148e) Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
-rw-r--r--libavcodec/fic.c4
1 files changed, 4 insertions, 0 deletions
diff --git a/libavcodec/fic.c b/libavcodec/fic.c
index 8691d49932..b7e0da6ce9 100644
--- a/libavcodec/fic.c
+++ b/libavcodec/fic.c
@@ -333,6 +333,10 @@ static int fic_decode_frame(AVCodecContext *avctx, void *data,
skip_cursor = 1;
}
+ if (!skip_cursor && avpkt->size < CURSOR_OFFSET + sizeof(ctx->cursor_buf)) {
+ skip_cursor = 1;
+ }
+
/* Slice height for all but the last slice. */
ctx->slice_h = 16 * (ctx->aligned_height >> 4) / nslices;
if (ctx->slice_h % 16)