Commit 2bc34e89 authored Jul 31, 2018 by Soumya Managoli Committed by Gerrit - the friendly Code Review server Aug 30, 2018

asoc : msm: Fix zero size pointer issue



APPS crashes randomly due to invalid memory allocation
in q6asm_audio_client_buf_free_contiguous.
Added check to return error if memory allocation size is 0.

Change-Id: I40f49aa147d513b29b56224a5ee77ccbb2dcc110
CRs-Fixed: 2285272
Signed-off-by: Soumya Managoli <smanag@codeaurora.org>

parent 62df730c

sound/soc/msm/qdsp6v2/q6asm.c

+6 −0

Original line number	Diff line number	Diff line
		@@ -1337,6 +1337,12 @@ int q6asm_audio_client_buf_alloc_contiguous(unsigned int dir,
		pr_err("%s: buffer already allocated\n", __func__);
		return 0;
		}

		if (bufcnt == 0) {
		pr_err("%s: invalid buffer count\n", __func__);
		return -EINVAL;
		}

		mutex_lock(&ac->cmd_lock);
		buf = kzalloc(((sizeof(struct audio_buffer))*bufcnt),
		GFP_KERNEL);