Commit a9c470d2 authored by Yu Jiaoliang's avatar Yu Jiaoliang Committed by Linus Walleij
Browse files

pinctrl: nomadik: Use kmemdup_array instead of kmemdup for multiple allocation 



Let the kememdup_array() take care about multiplication and possible
overflows.

Signed-off-by: default avatarYu Jiaoliang <yujiaoliang@vivo.com>
Link: https://lore.kernel.org/20240826074037.2313468-1-yujiaoliang@vivo.com


Signed-off-by: default avatarLinus Walleij <linus.walleij@linaro.org>
parent cd77b0dd

drivers/pinctrl/nomadik/pinctrl-abx500.c

+1 −2
Original line number Diff line number Diff line
@@ -716,8 +716,7 @@ static int abx500_dt_add_map_configs(struct pinctrl_map **map, 
	if (*num_maps == *reserved_maps)

		return -ENOSPC;



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

			      GFP_KERNEL);

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

	if (!dup_configs)

		return -ENOMEM;

drivers/pinctrl/nomadik/pinctrl-nomadik.c

+1 −2
Original line number Diff line number Diff line
@@ -601,8 +601,7 @@ static int nmk_dt_add_map_configs(struct pinctrl_map **map, 
	if (*num_maps == *reserved_maps)

		return -ENOSPC;



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

			      GFP_KERNEL);

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

	if (!dup_configs)

		return -ENOMEM;