diff mbox

[FFmpeg-devel,1/4] avcodec/wavpack: Fix runtime error: shift exponent 137 is too large for 32-bit type 'int'

Message ID 20170313023637.1848-1-michael@niedermayer.cc
State Accepted
Commit 1467143a6ebf08a16ec0b833ae462f88345828bd
Headers show

Commit Message

Michael Niedermayer March 13, 2017, 2:36 a.m. UTC 
Fixes: 808/clusterfuzz-testcase-4715513349406720

Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/targets/ffmpeg
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
---
 libavcodec/wavpack.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Michael Niedermayer March 14, 2017, 1:03 a.m. UTC | #1 
On Mon, Mar 13, 2017 at 03:36:34AM +0100, Michael Niedermayer wrote:
> Fixes: 808/clusterfuzz-testcase-4715513349406720
> 
> Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/targets/ffmpeg
> Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
> ---
>  libavcodec/wavpack.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

applied

[...]
diff mbox

Patch

diff --git a/libavcodec/wavpack.h b/libavcodec/wavpack.h
index 445d593c3b..c949390f51 100644
--- a/libavcodec/wavpack.h
+++ b/libavcodec/wavpack.h
@@ -171,7 +171,7 @@  static av_always_inline int wp_exp2(int16_t val)
 
     res   = wp_exp2_table[val & 0xFF] | 0x100;
     val >>= 8;
-    if (val > 31)
+    if (val > 31U)
         return INT_MIN;
     res   = (val > 9) ? (res << (val - 9)) : (res >> (9 - val));
     return neg ? -res : res;