Commit 70a195aa authored Sep 05, 2018 by Soumya Managoli Committed by Gerrit - the friendly Code Review server Sep 27, 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.

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

parent c2eefcbd

dsp/q6asm.c

+6 −0

Original line number	Diff line number	Diff line
		@@ -1560,6 +1560,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);