]> git.sesse.net Git - ffmpeg/commitdiff
avcodec/aacdec_fixed: Fix integer overflow in apply_independent_coupling_fixed()
authorMichael Niedermayer <michael@niedermayer.cc>
Sat, 31 Mar 2018 19:19:19 +0000 (21:19 +0200)
committerMichael Niedermayer <michael@niedermayer.cc>
Mon, 2 Apr 2018 21:27:51 +0000 (23:27 +0200)
I was not able to reproduce this, this fix is based on just the fuzzer log.
Fixes: 4959/clusterfuzz-testcase-minimized-6035350934781952
Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
libavcodec/aacdec_fixed.c

index f96999efb41aebec8c7814477380d8787326ea30..5c3613e06ce6394ad78204812412af05351faf65 100644 (file)
@@ -417,7 +417,7 @@ static void apply_independent_coupling_fixed(AACContext *ac,
     int i, c, shift, round, tmp;
     const int gain = cce->coup.gain[index][0];
     const int *src = cce->ch[0].ret;
-    int *dest = target->ret;
+    unsigned int *dest = target->ret;
     const int len = 1024 << (ac->oc[1].m4ac.sbr == 1);
 
     c = cce_scale_fixed[gain & 7];