2015-06-12 0:46 GMT+09:00 Dan Carpenter <[email protected]>: > We allocate sizeof("-grp") which is 5 bytes but then we pass 4 to the > snprintf() so the last 'p' char is truncated away. > > The kzalloc() can be made into kmalloc() since we are going to fill > the whole buffer. But I know that Walter Harms is going to grumble if I > don't use kasprintf(). :P And also checkpatch.pl these days complains > that the "allocation failed" printks aren't needed so I removed them. > > Signed-off-by: Dan Carpenter <[email protected]> > > diff --git a/drivers/pinctrl/samsung/pinctrl-exynos5440.c > b/drivers/pinctrl/samsung/pinctrl-exynos5440.c > index f5619fb..f804a61c 100644 > --- a/drivers/pinctrl/samsung/pinctrl-exynos5440.c > +++ b/drivers/pinctrl/samsung/pinctrl-exynos5440.c > @@ -215,12 +215,9 @@ static int exynos5440_dt_node_to_map(struct pinctrl_dev > *pctldev, > * Allocate memory for pin group name. The pin group name is derived > * from the node name from which these map entries are be created. > */ > - gname = kzalloc(strlen(np->name) + GSUFFIX_LEN, GFP_KERNEL); > - if (!gname) { > - dev_err(dev, "failed to alloc memory for group name\n"); > + gname = kasprintf(GFP_KERNEL, "%s%s", np->name, GROUP_SUFFIX); > + if (!gname) > goto free_map; > - } > - snprintf(gname, strlen(np->name) + 4, "%s%s", np->name, GROUP_SUFFIX);
I would prefer splitting this into two patches: one for fixing truncated name (by usage of kasprintf) and second for ENOMEM message. The second patch can get rid of ENOMEM message in other places as well. As for the first issue I think that function suffix would also be truncated in the same way. Best regards, Krzysztof -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
