You've already forked FFmpeg
mirror of
https://github.com/FFmpeg/FFmpeg.git
synced 2025-08-04 22:03:09 +02:00
avcodec/osq: Fixes several undefined overflows in do_decode()
Fixes: signed integer overflow: 1239596184 + 2119376059 cannot be represented in type 'int' Fixes: 376136844/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_OSQ_fuzzer-6581164455821312 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
This commit is contained in:
@ -300,7 +300,7 @@ static int do_decode(AVCodecContext *avctx, AVFrame *frame, int decorrelate, int
|
||||
dst[n] += (int)(P2 + P3) / 2 + (unsigned)p;
|
||||
break;
|
||||
case 8:
|
||||
dst[n] += (int)(P2 + P3) / 2;
|
||||
dst[n] += (int)(P2 + P3) / 2 + 0U;
|
||||
break;
|
||||
case 9:
|
||||
dst[n] += (int)(P2 * 2 + P3) / 3 + (unsigned)p;
|
||||
@ -309,13 +309,13 @@ static int do_decode(AVCodecContext *avctx, AVFrame *frame, int decorrelate, int
|
||||
dst[n] += (int)(P2 + P3 * 2) / 3 + (unsigned)p;
|
||||
break;
|
||||
case 11:
|
||||
dst[n] += (int)((unsigned)dst[A] + dst[B]) / 2;
|
||||
dst[n] += (int)((unsigned)dst[A] + dst[B]) / 2 + 0U;
|
||||
break;
|
||||
case 12:
|
||||
dst[n] += (unsigned)dst[B];
|
||||
break;
|
||||
case 13:
|
||||
dst[n] += (int)(unsigned)(dst[D] + dst[B]) / 2;
|
||||
dst[n] += (int)((unsigned)dst[D] + dst[B]) / 2 + 0U;
|
||||
break;
|
||||
case 14:
|
||||
dst[n] += (int)((unsigned)P2 + dst[A]) / 2 + (unsigned)p;
|
||||
|
Reference in New Issue
Block a user