diff options
| author | Michael Niedermayer <michael@niedermayer.cc> | 2020-10-23 20:39:33 +0200 |
|---|---|---|
| committer | Michael Niedermayer <michael@niedermayer.cc> | 2021-09-10 16:04:26 +0200 |
| commit | b98d0d4d33d81b3996ebb50f7533b8b32f41fe07 (patch) | |
| tree | c4c0f4b8273e8d5a649fbe0a5c06b6aaad21d5e7 | |
| parent | bd58373224b8b009ee3b5298000b410150bf9501 (diff) | |
| download | ffmpeg-b98d0d4d33d81b3996ebb50f7533b8b32f41fe07.tar.gz | |
avcodec/magicyuv: Check slice size before reading flags and pred
Fixes: heap-buffer-overflow
Fixes: 26487/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_MAGICYUV_fuzzer-5742553675333632
Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg
Reviewed-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com>
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
(cherry picked from commit 0dc42147b6843b133d4fa46bf1c2568a837b4bec)
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
| -rw-r--r-- | libavcodec/magicyuv.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/libavcodec/magicyuv.c b/libavcodec/magicyuv.c index c1dd9de68c..5415e63207 100644 --- a/libavcodec/magicyuv.c +++ b/libavcodec/magicyuv.c @@ -713,6 +713,9 @@ static int magy_decode_frame(AVCodecContext *avctx, void *data, s->slices[i][j].start = offset + header_size; s->slices[i][j].size = avpkt->size - s->slices[i][j].start; + + if (s->slices[i][j].size < 2) + return AVERROR_INVALIDDATA; } if (bytestream2_get_byte(&gbyte) != s->planes) |