avcodec/cook: Enlarge gain table

Fixes: index 25 out of bounds for type 'float [23]'
Fixes: 18355/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_COOK_fuzzer-5641398941908992

Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
pull/327/head
Michael Niedermayer 5 years ago
parent 57750bb629
commit 50001cd440
  1. 8
      libavcodec/cook.c

@ -143,7 +143,7 @@ typedef struct cook {
/* generate tables and related variables */ /* generate tables and related variables */
int gain_size_factor; int gain_size_factor;
float gain_table[23]; float gain_table[31];
/* data buffers */ /* data buffers */
@ -185,8 +185,8 @@ static av_cold void init_gain_table(COOKContext *q)
{ {
int i; int i;
q->gain_size_factor = q->samples_per_channel / 8; q->gain_size_factor = q->samples_per_channel / 8;
for (i = 0; i < 23; i++) for (i = 0; i < 31; i++)
q->gain_table[i] = pow(pow2tab[i + 52], q->gain_table[i] = pow(pow2tab[i + 48],
(1.0 / (double) q->gain_size_factor)); (1.0 / (double) q->gain_size_factor));
} }
@ -670,7 +670,7 @@ static void interpolate_float(COOKContext *q, float *buffer,
for (i = 0; i < q->gain_size_factor; i++) for (i = 0; i < q->gain_size_factor; i++)
buffer[i] *= fc1; buffer[i] *= fc1;
} else { // smooth gain } else { // smooth gain
fc2 = q->gain_table[11 + (gain_index_next - gain_index)]; fc2 = q->gain_table[15 + (gain_index_next - gain_index)];
for (i = 0; i < q->gain_size_factor; i++) { for (i = 0; i < q->gain_size_factor; i++) {
buffer[i] *= fc1; buffer[i] *= fc1;
fc1 *= fc2; fc1 *= fc2;

Loading…
Cancel
Save