diff mbox

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

Message ID 20170313023637.1848-4-michael@niedermayer.cc
State Accepted
Commit 7cebc5a9ccba0de7bddf7900ae85652ebc66141c
Headers show

Commit Message

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

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.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Michael Niedermayer March 14, 2017, 1:04 a.m. UTC | #1 
On Mon, Mar 13, 2017 at 03:36:37AM +0100, Michael Niedermayer wrote:
> Fixes: 822/clusterfuzz-testcase-4873433189974016
> 
> 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.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

applied

[...]
diff mbox

Patch

diff --git a/libavcodec/wavpack.c b/libavcodec/wavpack.c
index 943e46a25c..bc94b27c04 100644
--- a/libavcodec/wavpack.c
+++ b/libavcodec/wavpack.c
@@ -846,7 +846,7 @@  static int wavpack_decode_block(AVCodecContext *avctx, int block_no,
                 continue;
             }
             bytestream2_get_buffer(&gb, val, 4);
-            if (val[0] > 32) {
+            if (val[0] > 31) {
                 av_log(avctx, AV_LOG_ERROR,
                        "Invalid INT32INFO, extra_bits = %d (> 32)\n", val[0]);
                 continue;