diff options
| author | TreeHugger Robot <treehugger-gerrit@google.com> | 2021-05-01 00:17:00 +0000 |
|---|---|---|
| committer | Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> | 2021-05-01 00:17:00 +0000 |
| commit | 4c350fc3fcbdddd8d1d1057fb2156650e74d4402 (patch) | |
| tree | 7ff384d92bb6caf2ad80e9b9702366ab639a95c2 | |
| parent | 46a636829cd2f190d3bdc5febfd747c00ea9b4f0 (diff) | |
| parent | ab332a940d511e4995ca1b9f8051a5093f4828bd (diff) | |
| download | fdk-aac-4c350fc3fcbdddd8d1d1057fb2156650e74d4402.tar.gz fdk-aac-4c350fc3fcbdddd8d1d1057fb2156650e74d4402.tar.bz2 fdk-aac-4c350fc3fcbdddd8d1d1057fb2156650e74d4402.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: I12fcc01fba0918d33c9d802a614c97b7a3c33d6e
| -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++; } |