Commit 5e633f57 authored by Shen Lichuan's avatar Shen Lichuan Committed by Geert Uytterhoeven
Browse files

pinctrl: renesas: Switch to use kmemdup_array() 



Let kmemdup_array() take care of multiplication and possible overflow.

Signed-off-by: default avatarShen Lichuan <shenlichuan@vivo.com>
Reviewed-by: default avatarGeert Uytterhoeven <geert+renesas@glider.be>
Link: https://lore.kernel.org/20240822061438.14617-1-shenlichuan@vivo.com


Signed-off-by: default avatarGeert Uytterhoeven <geert+renesas@glider.be>
parent f73f63b2

drivers/pinctrl/renesas/pinctrl-rzg2l.c

+1 −2
Original line number Diff line number Diff line
@@ -528,8 +528,7 @@ static int rzg2l_map_add_config(struct pinctrl_map *map, 
{

	unsigned long *cfgs;



	cfgs = kmemdup(configs, num_configs * sizeof(*cfgs),

		       GFP_KERNEL);

	cfgs = kmemdup_array(configs, num_configs, sizeof(*cfgs), GFP_KERNEL);

	if (!cfgs)

		return -ENOMEM;

drivers/pinctrl/renesas/pinctrl-rzv2m.c

+1 −2
Original line number Diff line number Diff line
@@ -196,8 +196,7 @@ static int rzv2m_map_add_config(struct pinctrl_map *map, 
{

	unsigned long *cfgs;



	cfgs = kmemdup(configs, num_configs * sizeof(*cfgs),

		       GFP_KERNEL);

	cfgs = kmemdup_array(configs, num_configs, sizeof(*cfgs), GFP_KERNEL);

	if (!cfgs)

		return -ENOMEM;

drivers/pinctrl/renesas/pinctrl.c

+1 −2
Original line number Diff line number Diff line
@@ -83,8 +83,7 @@ static int sh_pfc_map_add_config(struct pinctrl_map *map, 
{

	unsigned long *cfgs;



	cfgs = kmemdup(configs, num_configs * sizeof(*cfgs),

		       GFP_KERNEL);

	cfgs = kmemdup_array(configs, num_configs, sizeof(*cfgs), GFP_KERNEL);

	if (cfgs == NULL)

		return -ENOMEM;