ASoC: amd/sdw_utils: avoid NULL deref when devm_kasprintf() fails
[ Upstream commit 5726b68473f7153a7f6294185e5998b7e2a230a2 ]
devm_kasprintf() may return NULL on memory allocation failure,
but the debug message prints cpus->dai_name before checking it.
Move the dev_dbg() call after the NULL check to prevent potential
NULL pointer dereference.
Fixes: cb8ea62e64 ("ASoC: amd/sdw_utils: add sof based soundwire generic machine driver")
Signed-off-by: Li Qiang <liqiang01@kylinos.cn>
Link: https://patch.msgid.link/20251015075530.146851-1-liqiang01@kylinos.cn
Signed-off-by: Mark Brown <broonie@kernel.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:
committed by
Greg Kroah-Hartman
parent
8ad0aea42f
commit
a1cccbd196
@@ -176,9 +176,9 @@ static int create_sdw_dailink(struct snd_soc_card *card,
|
|||||||
cpus->dai_name = devm_kasprintf(dev, GFP_KERNEL,
|
cpus->dai_name = devm_kasprintf(dev, GFP_KERNEL,
|
||||||
"SDW%d Pin%d",
|
"SDW%d Pin%d",
|
||||||
link_num, cpu_pin_id);
|
link_num, cpu_pin_id);
|
||||||
dev_dbg(dev, "cpu->dai_name:%s\n", cpus->dai_name);
|
|
||||||
if (!cpus->dai_name)
|
if (!cpus->dai_name)
|
||||||
return -ENOMEM;
|
return -ENOMEM;
|
||||||
|
dev_dbg(dev, "cpu->dai_name:%s\n", cpus->dai_name);
|
||||||
|
|
||||||
codec_maps[j].cpu = 0;
|
codec_maps[j].cpu = 0;
|
||||||
codec_maps[j].codec = j;
|
codec_maps[j].codec = j;
|
||||||
|
|||||||
Reference in New Issue
Block a user