summaryrefslogtreecommitdiff
path: root/libavcodec
diff options
context:
space:
mode:
authorMichael Niedermayer <michael@niedermayer.cc>2022-07-03 00:43:21 +0200
committerMichael Niedermayer <michael@niedermayer.cc>2022-09-25 13:51:42 +0200
commite5c8b53c683ba7683df418b11a59649f1ee819d8 (patch)
tree8bea65989af6a68cee31f5c04f8331386c0106bb /libavcodec
parent4ecf6ca4509bd9957589e41d95d7151fd3efff9d (diff)
downloadffmpeg-e5c8b53c683ba7683df418b11a59649f1ee819d8.tar.gz
avcodec/qdrw: adjust max colors to array size
Fixes: out of array access Fixes: 48429/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_QDRAW_fuzzer-4608329791438848 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Reviewed-by: Paul B Mahol <onemda@gmail.com> Signed-off-by: Michael Niedermayer <michael@niedermayer.cc> (cherry picked from commit cd847f86d31f87f0f7733ca6ab7a2c022a1398bd) Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
Diffstat (limited to 'libavcodec')
-rw-r--r--libavcodec/qdrw.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/libavcodec/qdrw.c b/libavcodec/qdrw.c
index 65279c9805..c04c756d71 100644
--- a/libavcodec/qdrw.c
+++ b/libavcodec/qdrw.c
@@ -369,7 +369,7 @@ static int decode_frame(AVCodecContext *avctx,
bytestream2_skip(&gbc, 18);
colors = bytestream2_get_be16(&gbc);
- if (colors < 0 || colors > 256) {
+ if (colors < 0 || colors > 255) {
av_log(avctx, AV_LOG_ERROR,
"Error color count - %i(0x%X)\n", colors, colors);
return AVERROR_INVALIDDATA;