diff options
| author | Jasmine Cha <chajasmine@google.com> | 2018-05-10 11:05:36 +0800 |
|---|---|---|
| committer | Jasmine Cha <chajasmine@google.com> | 2018-05-10 13:16:58 +0000 |
| commit | d9e9f33b897a70209257404e14c7ac1be8d3c181 (patch) | |
| tree | 388b070df057e5fbc56524f46072e3df8f713ac6 /post_proc | |
| parent | acbca59e60e1b97c95a04b3abb468b23adaf82a0 (diff) | |
| download | platform_hardware_qcom_audio-d9e9f33b897a70209257404e14c7ac1be8d3c181.tar.gz platform_hardware_qcom_audio-d9e9f33b897a70209257404e14c7ac1be8d3c181.tar.bz2 platform_hardware_qcom_audio-d9e9f33b897a70209257404e14c7ac1be8d3c181.zip | |
ma_listener: correct bounds of array
crashed by array index out of bounds.
Bug: 79468907
Bug: 79472606
Bug: 79495729
Test: manual audio test rington/music/alarm/notification
Change-Id: I327ef3b77c7c65064b67dfdc3404cf59d5d84203
Signed-off-by: Jasmine Cha <chajasmine@google.com>
Diffstat (limited to 'post_proc')
| -rw-r--r-- | post_proc/ma_listener.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/post_proc/ma_listener.c b/post_proc/ma_listener.c index 8e57db1..02d45d3 100644 --- a/post_proc/ma_listener.c +++ b/post_proc/ma_listener.c @@ -61,7 +61,7 @@ struct ma_state { static const audio_stream_type_t MIN_STREAM_TYPES = AUDIO_STREAM_VOICE_CALL; static const audio_stream_type_t MAX_STREAM_TYPES = AUDIO_STREAM_NOTIFICATION; -static struct ma_state g_cur_state[MAX_STREAM_TYPES]; +static struct ma_state g_cur_state[MAX_STREAM_TYPES + 1]; struct ma_listener_context_s { const struct effect_interface_s *itfe; @@ -211,6 +211,8 @@ static void check_and_set_ma_parameter(uint32_t stream_type) else if (max_vol > 1.0) max_vol = 1.0; if (send_ma_parameter != NULL && + stream_type >= MIN_STREAM_TYPES && + stream_type <= MAX_STREAM_TYPES && (g_cur_state[stream_type].vol != max_vol || g_cur_state[stream_type].active != active)) { |