Commit 5a66ac21 authored by Zichen Xie's avatar Zichen Xie Committed by Wen Zhiwei
Browse files

ASoC: qcom: Fix NULL Dereference in asoc_qcom_lpass_cpu_platform_probe() 

stable inclusion
from stable-v6.6.59
commit 73cc3f905ca9aa95694eea3dfa1acadc90686368
category: bugfix
bugzilla: https://gitee.com/openeuler/kernel/issues/IB3ENL

Reference: https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=73cc3f905ca9aa95694eea3dfa1acadc90686368



--------------------------------

commit 49da1463c9e3d2082276c3e0e2a8b65a88711cd2 upstream.

A devm_kzalloc() in asoc_qcom_lpass_cpu_platform_probe() could
possibly return NULL pointer. NULL Pointer Dereference may be
triggerred without addtional check.
Add a NULL check for the returned pointer.

Fixes: b5022a36 ("ASoC: qcom: lpass: Use regmap_field for i2sctl and dmactl registers")
Cc: stable@vger.kernel.org
Signed-off-by: default avatarZichen Xie <zichenxie0106@gmail.com>
Link: https://patch.msgid.link/20241006205737.8829-1-zichenxie0106@gmail.com


Signed-off-by: default avatarMark Brown <broonie@kernel.org>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: default avatarWen Zhiwei <wenzhiwei@kylinos.cn>
parent 54462813

sound/soc/qcom/lpass-cpu.c

+2 −0
Original line number Diff line number Diff line
@@ -1246,6 +1246,8 @@ int asoc_qcom_lpass_cpu_platform_probe(struct platform_device *pdev) 
	/* Allocation for i2sctl regmap fields */

	drvdata->i2sctl = devm_kzalloc(&pdev->dev, sizeof(struct lpaif_i2sctl),

					GFP_KERNEL);

	if (!drvdata->i2sctl)

		return -ENOMEM;



	/* Initialize bitfields for dai I2SCTL register */

	ret = lpass_cpu_init_i2sctl_bitfields(dev, drvdata->i2sctl,