diff options
| author | Jean-Michel Trivi <jmtrivi@google.com> | 2018-12-27 09:26:34 -0800 |
|---|---|---|
| committer | android-build-merger <android-build-merger@google.com> | 2018-12-27 09:26:34 -0800 |
| commit | 7d959f2f303a6149c49d057b085beed8a9d6426c (patch) | |
| tree | b96fd83a1fe74e30432ddd931cc8aee845f71b8b | |
| parent | ca0d8e7461269c6d169aa70086ef8f26f43bcc90 (diff) | |
| parent | 55930c28afb15d2a05ef5106e887d3e6b998524d (diff) | |
| download | fdk-aac-7d959f2f303a6149c49d057b085beed8a9d6426c.tar.gz fdk-aac-7d959f2f303a6149c49d057b085beed8a9d6426c.tar.bz2 fdk-aac-7d959f2f303a6149c49d057b085beed8a9d6426c.zip | |
Merge "Signed Integer Overflows in imlt_block()"
am: 55930c28af
Change-Id: Ie1e1f5d57cd6a8bf0f56461ceef36d15b3dd0db6
| -rw-r--r-- | libAACdec/src/usacdec_fac.cpp | 16 |
1 files changed, 10 insertions, 6 deletions
diff --git a/libAACdec/src/usacdec_fac.cpp b/libAACdec/src/usacdec_fac.cpp index c10a3fe..0d3d844 100644 --- a/libAACdec/src/usacdec_fac.cpp +++ b/libAACdec/src/usacdec_fac.cpp @@ -534,10 +534,12 @@ INT CLpd_FAC_Acelp2Mdct(H_MDCT hMdct, FIXP_DBL *output, FIXP_DBL *_pSpec, /* Optional scaling of time domain - no yet windowed - of current spectrum */ if (total_gain != (FIXP_DBL)0) { - scaleValuesWithFactor(pSpec, total_gain, tl, spec_scale[0] + scale); - } else { - scaleValuesSaturate(pSpec, tl, spec_scale[0] + scale); + for (i = 0; i < tl; i++) { + pSpec[i] = fMult(pSpec[i], total_gain); + } } + int loc_scale = fixmin_I(spec_scale[0] + scale, (INT)DFRACT_BITS - 1); + scaleValuesSaturate(pSpec, tl, loc_scale); pOut1 += fl / 2 - 1; pCurr = pSpec + tl - fl / 2; @@ -623,10 +625,12 @@ INT CLpd_FAC_Acelp2Mdct(H_MDCT hMdct, FIXP_DBL *output, FIXP_DBL *_pSpec, */ /* and de-scale current spectrum signal (time domain, no yet windowed) */ if (total_gain != (FIXP_DBL)0) { - scaleValuesWithFactor(pSpec, total_gain, tl, spec_scale[w] + scale); - } else { - scaleValuesSaturate(pSpec, tl, spec_scale[w] + scale); + for (i = 0; i < tl; i++) { + pSpec[i] = fMult(pSpec[i], total_gain); + } } + loc_scale = fixmin_I(spec_scale[w] + scale, (INT)DFRACT_BITS - 1); + scaleValuesSaturate(pSpec, tl, loc_scale); if (noOutSamples <= nrSamples) { /* Divert output first half to overlap buffer if we already got enough |