Re: [PATCH] media: atomisp: use kmalloc_array() for array space allocation

From: Andy Shevchenko
Date: Wed Aug 20 2025 - 11:16:46 EST


On Sun, Aug 17, 2025 at 05:29:39PM +0800, Qianfeng Rong wrote:
> Replace kmalloc(count * sizeof) with kmalloc_array() for safer memory
> allocation and overflow prevention.

...

> - descr->in_info = kmalloc(descr->num_stage *
> - sizeof(struct ia_css_frame_info),
> - GFP_KERNEL);
> + descr->in_info = kmalloc_array(descr->num_stage,
> + sizeof(struct ia_css_frame_info),
> + GFP_KERNEL);

At the same time it would be nice to use sizeof(*...) variants instead of using
type-based.

--
With Best Regards,
Andy Shevchenko