diff options
| author | rago <rago@google.com> | 2016-08-22 17:59:38 -0700 |
|---|---|---|
| committer | Jessica Wagantall <jwagantall@cyngn.com> | 2016-11-16 15:26:25 -0700 |
| commit | 90d8e26979f82c77c0b71f81143a0db05b863ccf (patch) | |
| tree | af9156945074227cb863c014b69cf925cdb9c517 | |
| parent | c9eed8342b4ac3ad606a2e95d636422aedf92d5d (diff) | |
| download | android_hardware_qcom_audio-stable/cm-12.1-caf-8916-YOG4P.tar.gz android_hardware_qcom_audio-stable/cm-12.1-caf-8916-YOG4P.tar.bz2 android_hardware_qcom_audio-stable/cm-12.1-caf-8916-YOG4P.zip | |
Fix potential overflow in Visualizer effectstable/cm-12.1-caf-8916-YOG4P
CYNGNOS-3303
Bug: 30229821
Change-Id: Iea1c4a21735e893aeded95b980044ec0861a7ea8
(cherry picked from commit 2fa52194ef64843c2908c69527384c6c2fcdbafa)
(cherry picked from commit 57ac66340ad488a17fc285b6fc2635cb7375d72b)
(cherry picked from commit ae92537deb2006b163d8d9c8e0ff37848475931a)
| -rw-r--r-- | visualizer/offload_visualizer.c | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/visualizer/offload_visualizer.c b/visualizer/offload_visualizer.c index 94c44a59..f4cad35a 100644 --- a/visualizer/offload_visualizer.c +++ b/visualizer/offload_visualizer.c @@ -846,6 +846,14 @@ int visualizer_command(effect_context_t * context, uint32_t cmdCode, uint32_t cm break; case VISUALIZER_CMD_MEASURE: { + if (pReplyData == NULL || replySize == NULL || + *replySize < (sizeof(int32_t) * MEASUREMENT_COUNT)) { + ALOGV("%s VISUALIZER_CMD_MEASURE error *replySize %d <" + "(sizeof(int32_t) * MEASUREMENT_COUNT) %d", + __func__, *replySize, sizeof(int32_t) * MEASUREMENT_COUNT); + android_errorWriteLog(0x534e4554, "30229821"); + return -EINVAL; + } uint16_t peak_u16 = 0; float sum_rms_squared = 0.0f; uint8_t nb_valid_meas = 0; |