summaryrefslogtreecommitdiffstats
path: root/sound/soc/codecs/bt-sco.c
diff options
context:
space:
mode:
authorMeng Tang <tangmeng@uniontech.com>2022-03-02 10:43:51 +0100
committerMark Brown <broonie@kernel.org>2022-03-03 13:48:36 +0100
commite94769900f4302b4034945e5d9ec8262a2f5e086 (patch)
treec2d630a64aaeb61d6ad807553c9cb8f9116303ca /sound/soc/codecs/bt-sco.c
parentASoC: Intel: machine driver updates for 5.18 (diff)
downloadlinux-e94769900f4302b4034945e5d9ec8262a2f5e086.tar.xz
linux-e94769900f4302b4034945e5d9ec8262a2f5e086.zip
ASoC: hdac_hda: Avoid unexpected match when pcm_name is "Analog"
pcm name can be "Analog" and "Alt Analog", cpcm->name can be "Analog Codec DAI" and "Alt Analog Codec DAI". When pcm_name is "Analog", "Analog Codec DAI" and "Alt Analog Codec DAI" are both satisfy the 'if (strstr(cpcm->name, pcm_name))' condition, which may cause the returned cpcm to be "Alt Analog Codec DAI". Even if we get the pcm name by id, and "Analog Codec DAI" goes into the loop before "Alt Analog Codec DAI", but I still think we'd better have multiple insurances against unexpected return values. After, we can correctly return the expected result even if other relevant places are changed. Signed-off-by: Meng Tang <tangmeng@uniontech.com> Link: https://lore.kernel.org/r/20220302094351.3487-1-tangmeng@uniontech.com Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'sound/soc/codecs/bt-sco.c')
0 files changed, 0 insertions, 0 deletions