]> git.ipfire.org Git - thirdparty/linux.git/commitdiff
ASoC: amd: sof_amd_sdw: Fix unlikely uninitialized variable use in create_sdw_dailinks()
authorVijendar Mukunda <Vijendar.Mukunda@amd.com>
Tue, 6 May 2025 12:07:23 +0000 (17:37 +0530)
committerMark Brown <broonie@kernel.org>
Tue, 6 May 2025 13:48:25 +0000 (22:48 +0900)
Initialize current_be_id to 0 in SOF based AMD generic SoundWire machine
driver to handle the unlikely case when there are no devices connected to
a DAI.
In this case create_sdw_dailink() would return without touching the passed
pointer to current_be_id.

Found by gcc -fanalyzer

Cc: stable@vger.kernel.org
Fixes: 6d8348ddc56ed ("ASoC: amd: acp: refactor SoundWire machine driver code")
Signed-off-by: Vijendar Mukunda <Vijendar.Mukunda@amd.com>
Link: https://patch.msgid.link/20250506120823.3621604-2-Vijendar.Mukunda@amd.com
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/amd/acp/acp-sdw-sof-mach.c

index c09b1f118a6cc1d0626fcb7542c6ea437aeeb275..75bdd843ca3681835aa94bbeb750ca9655e8155e 100644 (file)
@@ -219,7 +219,7 @@ static int create_sdw_dailinks(struct snd_soc_card *card,
 
        /* generate DAI links by each sdw link */
        while (sof_dais->initialised) {
-               int current_be_id;
+               int current_be_id = 0;
 
                ret = create_sdw_dailink(card, sof_dais, dai_links,
                                         &current_be_id, codec_conf);