Skip to content

Commit d0fa875

Browse files
qianfengronglag-linaro
authored andcommitted
backlight: led_bl: Use devm_kcalloc() for array space allocation
Replace calls of devm_kzalloc() with devm_kcalloc() in led_bl_get_leds() and led_bl_parse_levels() for safer memory allocation with built-in overflow protection. Signed-off-by: Qianfeng Rong <rongqianfeng@vivo.com> Reviewed-by: "Daniel Thompson (RISCstar)" <danielt@kernel.org> Link: https://lore.kernel.org/r/20250819035804.433615-1-rongqianfeng@vivo.com Signed-off-by: Lee Jones <lee@kernel.org>
1 parent fe85a39 commit d0fa875

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

drivers/video/backlight/led_bl.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -89,7 +89,7 @@ static int led_bl_get_leds(struct device *dev,
8989
return -EINVAL;
9090
}
9191

92-
leds = devm_kzalloc(dev, sizeof(struct led_classdev *) * nb_leds,
92+
leds = devm_kcalloc(dev, nb_leds, sizeof(struct led_classdev *),
9393
GFP_KERNEL);
9494
if (!leds)
9595
return -ENOMEM;
@@ -137,7 +137,7 @@ static int led_bl_parse_levels(struct device *dev,
137137
unsigned int db;
138138
u32 *levels = NULL;
139139

140-
levels = devm_kzalloc(dev, sizeof(u32) * num_levels,
140+
levels = devm_kcalloc(dev, num_levels, sizeof(u32),
141141
GFP_KERNEL);
142142
if (!levels)
143143
return -ENOMEM;

0 commit comments

Comments
 (0)