diff options
| author | TreeHugger Robot <treehugger-gerrit@google.com> | 2021-05-01 00:15:35 +0000 |
|---|---|---|
| committer | Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> | 2021-05-01 00:15:35 +0000 |
| commit | 24e69d45522bcb4848de96d89470d13b8fe59f86 (patch) | |
| tree | 7ff384d92bb6caf2ad80e9b9702366ab639a95c2 /libSACdec | |
| parent | 740116e266472fd6c84812f4251aa8bfb5a243e7 (diff) | |
| parent | ab332a940d511e4995ca1b9f8051a5093f4828bd (diff) | |
| download | fdk-aac-24e69d45522bcb4848de96d89470d13b8fe59f86.tar.gz fdk-aac-24e69d45522bcb4848de96d89470d13b8fe59f86.tar.bz2 fdk-aac-24e69d45522bcb4848de96d89470d13b8fe59f86.zip | |
Merge "Avoid signed integer overflow in combineSignalCplxScale2()." into sc-dev am: ab332a940d
Original change: https://googleplex-android-review.googlesource.com/c/platform/external/aac/+/14387474
Change-Id: I88ee25478125c79da035237e4f1835df9f99020b
Diffstat (limited to 'libSACdec')
| -rw-r--r-- | libSACdec/src/sac_stp.cpp | 15 |
1 files changed, 9 insertions, 6 deletions
diff --git a/libSACdec/src/sac_stp.cpp b/libSACdec/src/sac_stp.cpp index b328c82..be332c7 100644 --- a/libSACdec/src/sac_stp.cpp +++ b/libSACdec/src/sac_stp.cpp @@ -252,12 +252,15 @@ inline void combineSignalCplxScale2(FIXP_DBL *hybOutputRealDry, int n; for (n = bands - 1; n >= 0; n--) { - *hybOutputRealDry = - *hybOutputRealDry + - (fMultDiv2(*hybOutputRealWet, scaleX) << (SF_SCALE + 1)); - *hybOutputImagDry = - *hybOutputImagDry + - (fMultDiv2(*hybOutputImagWet, scaleX) << (SF_SCALE + 1)); + *hybOutputRealDry = SATURATE_LEFT_SHIFT( + (*hybOutputRealDry >> 1) + + (fMultDiv2(*hybOutputRealWet, scaleX) << SF_SCALE), + 1, DFRACT_BITS); + *hybOutputImagDry = SATURATE_LEFT_SHIFT( + (*hybOutputImagDry >> 1) + + (fMultDiv2(*hybOutputImagWet, scaleX) << SF_SCALE), + 1, DFRACT_BITS); + ; hybOutputRealDry++, hybOutputRealWet++; hybOutputImagDry++, hybOutputImagWet++; } |