From: Michael Niedermayer Date: Fri, 5 May 2017 11:02:22 +0000 (+0200) Subject: avcodec/pixlet: Fix shift exponent 4294967268 is too large for 32-bit type 'int' X-Git-Tag: n3.4~1757 X-Git-Url: http://git.videolan.org/?p=ffmpeg.git;a=commitdiff_plain;h=1f5b6c7e1ee604b1525b3ab84ea6e8817fe66f36;hp=ce551a3925a1cf9c7824e26a246b99b6773bda4b avcodec/pixlet: Fix shift exponent 4294967268 is too large for 32-bit type 'int' Fixes: 1336/clusterfuzz-testcase-minimized-4761381930795008 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/targets/ffmpeg Signed-off-by: Michael Niedermayer --- diff --git a/libavcodec/pixlet.c b/libavcodec/pixlet.c index c4f7597866..d031305155 100644 --- a/libavcodec/pixlet.c +++ b/libavcodec/pixlet.c @@ -154,7 +154,7 @@ static int read_low_coeffs(AVCodecContext *avctx, int16_t *dst, int size, int wi state = 120 * (escape + flag) + state - (120 * state >> 8); flag = 0; - if (state * 4 > 0xFF || i >= size) + if (state * 4ULL > 0xFF || i >= size) continue; nbits = ((state + 8) >> 5) + (state ? ff_clz(state) : 32) - 24; @@ -260,7 +260,7 @@ static int read_high_coeffs(AVCodecContext *avctx, uint8_t *src, int16_t *dst, i flag = 0; - if (state * 4 > 0xFF || i >= size) + if (state * 4ULL > 0xFF || i >= size) continue; pfx = ((state + 8) >> 5) + (state ? ff_clz(state): 32) - 24;