diff options
| author | Linux Build Service Account <lnxbuild@localhost> | 2016-11-14 13:20:17 -0800 |
|---|---|---|
| committer | Gerrit - the friendly Code Review server <code-review@localhost> | 2016-11-14 13:20:17 -0800 |
| commit | ec208c0142db2cc3b91b29b5069831d973d9400c (patch) | |
| tree | 918925a5ad8f7f60f77c4b754b41eaf9d1030318 /visualizer/offload_visualizer.c | |
| parent | 58d2ad86a800165f561ec114136aa1cd1675f1bf (diff) | |
| parent | d8c80c2fb0e7b2ce6fdb754751a4c47ca146dab7 (diff) | |
Merge "Fix potential overflow in Visualizer effect"
Diffstat (limited to 'visualizer/offload_visualizer.c')
| -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 f49c4345..ef2835ad 100644 --- a/visualizer/offload_visualizer.c +++ b/visualizer/offload_visualizer.c @@ -896,6 +896,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; |