[Intel-wired-lan] [PATCH 4/6] iavf: use struct_size() in kzalloc()
Bowers, AndrewX
andrewx.bowers at intel.com
Tue Apr 2 18:44:32 UTC 2019
> -----Original Message-----
> From: Intel-wired-lan [mailto:intel-wired-lan-bounces at osuosl.org] On
> Behalf Of Jeff Kirsher
> Sent: Friday, March 29, 2019 4:39 PM
> To: intel-wired-lan at lists.osuosl.org
> Subject: [Intel-wired-lan] [PATCH 4/6] iavf: use struct_size() in kzalloc()
>
> From: "Gustavo A. R. Silva" <gustavo at embeddedor.com>
>
> One of the more common cases of allocation size calculations is finding the
> size of a structure that has a zero-sized array at the end, along with memory
> for some number of elements for that array. For example:
>
> struct foo {
> int stuff;
> struct boo entry[];
> };
>
> size = sizeof(struct foo) + count * sizeof(struct boo); instance = kzalloc(size,
> GFP_KERNEL)
>
> Instead of leaving these open-coded and prone to type mistakes, we can
> now use the new struct_size() helper:
>
> instance = kzalloc(struct_size(instance, entry, count), GFP_KERNEL)
>
> Notice that, in this case, variable bufsz is not necessary, hence it is removed.
>
> This code was detected with the help of Coccinelle.
>
> Signed-off-by: "Gustavo A. R. Silva" <gustavo at embeddedor.com>
> ---
> drivers/net/ethernet/intel/iavf/iavf_main.c | 9 ++++-----
> 1 file changed, 4 insertions(+), 5 deletions(-)
Tested-by: Andrew Bowers <andrewx.bowers at intel.com>
More information about the Intel-wired-lan
mailing list