diff mbox

[FFmpeg-devel,1/2] avcodec/mpegaudiodec_template: Fix 2 runtime error: signed integer overflow

Message ID 20170320012150.6470-1-michael@niedermayer.cc
State Accepted
Commit b15818642b4e8c4ea61bf93bc6920e71a834a535
Headers show

Commit Message

Michael Niedermayer March 20, 2017, 1:21 a.m. UTC
Fixes: 873/clusterfuzz-testcase-5714546230558720

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

Comments

Michael Niedermayer March 21, 2017, 7:50 p.m. UTC | #1
On Mon, Mar 20, 2017 at 02:21:49AM +0100, Michael Niedermayer wrote:
> Fixes: 873/clusterfuzz-testcase-5714546230558720
> 
> Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/targets/ffmpeg
> Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
> ---
>  libavcodec/mpegaudiodec_template.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)

patchset applied

[...]
diff mbox

Patch

diff --git a/libavcodec/mpegaudiodec_template.c b/libavcodec/mpegaudiodec_template.c
index a5ac5817f3..6e94cf75f3 100644
--- a/libavcodec/mpegaudiodec_template.c
+++ b/libavcodec/mpegaudiodec_template.c
@@ -1038,7 +1038,8 @@  static void compute_stereo(MPADecodeContext *s, GranuleDef *g0, GranuleDef *g1)
 {
     int i, j, k, l;
     int sf_max, sf, len, non_zero_found;
-    INTFLOAT (*is_tab)[16], *tab0, *tab1, tmp0, tmp1, v1, v2;
+    INTFLOAT (*is_tab)[16], *tab0, *tab1, v1, v2;
+    SUINTFLOAT tmp0, tmp1;
     int non_zero_found_short[3];
 
     /* intensity stereo */