aboutsummaryrefslogtreecommitdiffstats
path: root/libSBRdec/src
diff options
context:
space:
mode:
authorJean-Michel Trivi <jmtrivi@google.com>2018-12-28 15:17:50 -0800
committerandroid-build-merger <android-build-merger@google.com>2018-12-28 15:17:50 -0800
commit0cd0a834f6b72e6c8d071a31496681dac4c93422 (patch)
treef32bbc8c524e337f266f928a4145dc2b2192a6d8 /libSBRdec/src
parentf04d154c74d0cd70d400834020ccab3238e0af78 (diff)
parent3d9793633e2a5077735b283a20ae826641a6af31 (diff)
downloadfdk-aac-0cd0a834f6b72e6c8d071a31496681dac4c93422.tar.gz
fdk-aac-0cd0a834f6b72e6c8d071a31496681dac4c93422.tar.bz2
fdk-aac-0cd0a834f6b72e6c8d071a31496681dac4c93422.zip
Merge "Fix potential invalid memory access for concealment in decodeEnvelope()" am: 6acfe3b799
am: 3d9793633e Change-Id: Iba356069f8d51e6a2afd199f62b84867dd34902e
Diffstat (limited to 'libSBRdec/src')
-rw-r--r--libSBRdec/src/env_dec.cpp13
1 files changed, 9 insertions, 4 deletions
diff --git a/libSBRdec/src/env_dec.cpp b/libSBRdec/src/env_dec.cpp
index 05accd1..95807c9 100644
--- a/libSBRdec/src/env_dec.cpp
+++ b/libSBRdec/src/env_dec.cpp
@@ -506,15 +506,20 @@ static void decodeEnvelope(
*/
for (i = 0; i < hHeaderData->freqBandData.nSfb[1]; i++) {
/* Former Level-Channel will be used for both channels */
- if (h_prev_data->coupling == COUPLING_BAL)
- h_prev_data->sfb_nrg_prev[i] = otherChannel->sfb_nrg_prev[i];
+ if (h_prev_data->coupling == COUPLING_BAL) {
+ h_prev_data->sfb_nrg_prev[i] =
+ (otherChannel != NULL) ? otherChannel->sfb_nrg_prev[i]
+ : (FIXP_SGL)SBR_ENERGY_PAN_OFFSET;
+ }
/* Former L/R will be combined as the new Level-Channel */
- else if (h_sbr_data->coupling == COUPLING_LEVEL)
+ else if (h_sbr_data->coupling == COUPLING_LEVEL &&
+ otherChannel != NULL) {
h_prev_data->sfb_nrg_prev[i] = (h_prev_data->sfb_nrg_prev[i] +
otherChannel->sfb_nrg_prev[i]) >>
1;
- else if (h_sbr_data->coupling == COUPLING_BAL)
+ } else if (h_sbr_data->coupling == COUPLING_BAL) {
h_prev_data->sfb_nrg_prev[i] = (FIXP_SGL)SBR_ENERGY_PAN_OFFSET;
+ }
}
}
}