mp3dec: adjust IMDCT scaling to avoid overflows

Fixes ticket268

Signed-off-by: Michael Niedermayer <michaelni@gmx.at>
pull/28/head
Michael Niedermayer 13 years ago
parent 98a2f4e87b
commit 571572fcdd
  1. 2
      libavcodec/mpegaudio.h
  2. 4
      libavcodec/mpegaudio_tablegen.h
  3. 2
      libavcodec/mpegaudiodsp_template.c

@ -52,6 +52,8 @@
#define WFRAC_BITS 16 /* fractional bits for window */ #define WFRAC_BITS 16 /* fractional bits for window */
#endif #endif
#define IMDCT_SCALAR 1.759
#define FRAC_ONE (1 << FRAC_BITS) #define FRAC_ONE (1 << FRAC_BITS)
#define FIX(a) ((int)((a) * FRAC_ONE)) #define FIX(a) ((int)((a) * FRAC_ONE))

@ -47,7 +47,7 @@ static void mpegaudio_tableinit(void)
double value = i / 4; double value = i / 4;
double f, fm; double f, fm;
int e, m; int e, m;
f = value * cbrtf(value) * pow(2, (i & 3) * 0.25); f = value / IMDCT_SCALAR * cbrtf(value) * pow(2, (i & 3) * 0.25);
fm = frexp(f, &e); fm = frexp(f, &e);
m = (uint32_t)(fm * (1LL << 31) + 0.5); m = (uint32_t)(fm * (1LL << 31) + 0.5);
e += FRAC_BITS - 31 + 5 - 100; e += FRAC_BITS - 31 + 5 - 100;
@ -58,7 +58,7 @@ static void mpegaudio_tableinit(void)
} }
for (exponent = 0; exponent < 512; exponent++) { for (exponent = 0; exponent < 512; exponent++) {
for (value = 0; value < 16; value++) { for (value = 0; value < 16; value++) {
double f = (double)value * cbrtf(value) * pow(2, (exponent - 400) * 0.25 + FRAC_BITS + 5); double f = (double)value * cbrtf(value) * pow(2, (exponent - 400) * 0.25 + FRAC_BITS + 5) / IMDCT_SCALAR;
expval_table_fixed[exponent][value] = llrint(f); expval_table_fixed[exponent][value] = llrint(f);
expval_table_float[exponent][value] = f; expval_table_float[exponent][value] = f;
} }

@ -243,7 +243,7 @@ void RENAME(ff_init_mpadsp_tabs)(void)
else if (i < 18) d = 1; else if (i < 18) d = 1;
} }
//merge last stage of imdct into the window coefficients //merge last stage of imdct into the window coefficients
d *= 0.5 / cos(M_PI * (2 * i + 19) / 72); d *= 0.5 * IMDCT_SCALAR / cos(M_PI * (2 * i + 19) / 72);
if (j == 2) if (j == 2)
RENAME(ff_mdct_win)[j][i/3] = FIXHR((d / (1<<5))); RENAME(ff_mdct_win)[j][i/3] = FIXHR((d / (1<<5)));

Loading…
Cancel
Save