diff options
| author | Michael Niedermayer <michael@niedermayer.cc> | 2019-07-07 00:03:51 +0200 |
|---|---|---|
| committer | Michael Niedermayer <michael@niedermayer.cc> | 2019-07-21 11:26:36 +0200 |
| commit | b880b3b236ddd00f85ea502b4c17a145fd26c790 (patch) | |
| tree | 8cf68a338d559e21d6a8c141ed682ed4fc737cce /libavcodec/alsdec.c | |
| parent | 9cd0d94f59d05e7bfaae9690e827752e7717eda3 (diff) | |
| download | ffmpeg-b880b3b236ddd00f85ea502b4c17a145fd26c790.tar.gz | |
avcodec/alsdec: fix undefined shift in multiply()
Fixes: left shift of negative value -6
Fixes: 15564/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_ALS_fuzzer-5701655938465792
Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
Diffstat (limited to 'libavcodec/alsdec.c')
| -rw-r--r-- | libavcodec/alsdec.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/libavcodec/alsdec.c b/libavcodec/alsdec.c index 8201deb366..6b5774175b 100644 --- a/libavcodec/alsdec.c +++ b/libavcodec/alsdec.c @@ -1416,7 +1416,7 @@ static SoftFloat_IEEE754 multiply(SoftFloat_IEEE754 a, SoftFloat_IEEE754 b) { return_val = 0x80000000U; } - return_val |= (a.exp + b.exp + bit_count - 47) << 23; + return_val |= ((unsigned)av_clip(a.exp + b.exp + bit_count - 47, -126, 127) << 23) & 0x7F800000; return_val |= mantissa; return av_bits2sf_ieee754(return_val); } |