ALSA: hda/ca0132: Fix buffer overflow in add_tuning_control
[ Upstream commit a409c60111e6bb98fcabab2aeaa069daa9434ca0 ] The 'sprintf' call in 'add_tuning_control' may exceed the 44-byte buffer if either string argument is too long. This triggers a compiler warning. Replaced 'sprintf' with 'snprintf' to limit string lengths to prevent overflow. Reported-by: kernel test robot <lkp@intel.com> Closes: https://lore.kernel.org/oe-kbuild-all/202506100642.95jpuMY1-lkp@intel.com/ Signed-off-by: Lucy Thrun <lucy.thrun@digital-rabbithole.de> Link: https://patch.msgid.link/20250610175012.918-3-lucy.thrun@digital-rabbithole.de Signed-off-by: Takashi Iwai <tiwai@suse.de> Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:
committed by
Greg Kroah-Hartman
parent
85b3b3966a
commit
fb8116c38d
@@ -4279,7 +4279,7 @@ static int add_tuning_control(struct hda_codec *codec,
|
||||
}
|
||||
knew.private_value =
|
||||
HDA_COMPOSE_AMP_VAL(nid, 1, 0, type);
|
||||
sprintf(namestr, "%s %s Volume", name, dirstr[dir]);
|
||||
snprintf(namestr, sizeof(namestr), "%s %s Volume", name, dirstr[dir]);
|
||||
return snd_hda_ctl_add(codec, nid, snd_ctl_new1(&knew, codec));
|
||||
}
|
||||
|
||||
|
||||
Reference in New Issue
Block a user