diff options
author | wjiang <wjiang@codeaurora.org> | 2014-05-15 19:38:26 +0800 |
---|---|---|
committer | wjiang <wjiang@codeaurora.org> | 2014-05-16 23:11:15 +0800 |
commit | ebb69fae687766e4c737b4e31752cde553f68779 (patch) | |
tree | 02191544cff6b7f8cd403eac0596149e025d3727 /visualizer/offload_visualizer.c | |
parent | d72dde4c99abfa8fc870e16c9efe2e3829dc2a7f (diff) |
post_proc: fix post and pre process KW issues
KW issue fix includes:
- handle memory allocation failure
- array index boundary check
Change-Id: I083952ba58d348a5b650601a83e6f492b0d686bb
Diffstat (limited to 'visualizer/offload_visualizer.c')
-rw-r--r-- | visualizer/offload_visualizer.c | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/visualizer/offload_visualizer.c b/visualizer/offload_visualizer.c index 95b46874..94c44a59 100644 --- a/visualizer/offload_visualizer.c +++ b/visualizer/offload_visualizer.c @@ -431,6 +431,11 @@ int visualizer_hal_start_output(audio_io_handle_t output, int pcm_id) { } output_context_t *out_ctxt = (output_context_t *)malloc(sizeof(output_context_t)); + if (out_ctxt == NULL) { + ALOGE("%s fail to allocate memory", __func__); + ret = -ENOMEM; + goto exit; + } out_ctxt->handle = output; list_init(&out_ctxt->effects_list); @@ -927,6 +932,10 @@ int effect_lib_create(const effect_uuid_t *uuid, if (memcmp(uuid, &visualizer_descriptor.uuid, sizeof(effect_uuid_t)) == 0) { visualizer_context_t *visu_ctxt = (visualizer_context_t *)calloc(1, sizeof(visualizer_context_t)); + if (visu_ctxt == NULL) { + ALOGE("%s fail to allocate memory", __func__); + return -ENOMEM; + } context = (effect_context_t *)visu_ctxt; context->ops.init = visualizer_init; context->ops.reset = visualizer_reset; |