Difuse/u-boot
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

pinctrl: fix buffer size for pinctrl_generic_set_state_prefix()

Browse source 
This buffer has the concatenated prefix and name written into it, so it
must be large enough to cover both strings plus the terminating NUL.

Fixes: 92c4a95ec7 ("pinctrl: Add new function pinctrl_generic_set_state_prefix()")
Signed-off-by: John Keeping <john@metanate.com>
Reviewed-by: Pali Rohár <pali@kernel.org>
Reviewed-by: Simon Glass <sjg@chromium.org>
This commit is contained in:
John Keeping 2022-09-07 12:11:42 +01:00 committed by Tom Rini
parent 45455e8ff5
commit 7eda1a9533
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
drivers/pinctrl/pinctrl-generic.c
View file

@ -237,7 +237,7 @@ enum pinmux_subnode_type {
static const char *alloc_name_with_prefix(const char *name, const char *prefix)
{
if (prefix) {
char *name_with_prefix = malloc(strlen(prefix) + sizeof("pins"));
char *name_with_prefix = malloc(strlen(prefix) + strlen(name) + 1);
if (name_with_prefix)
sprintf(name_with_prefix, "%s%s", prefix, name);
return name_with_prefix;