diff options
author | Gustavo A. R. Silva <gustavo@embeddedor.com> | 2019-06-05 15:38:27 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2019-06-05 16:54:43 -0700 |
commit | 856e6d9f9dd223dea978f1300f7c26903b9f9f6d (patch) | |
tree | 615acb75601a1451a7b18542c06e314595c5100e /net | |
parent | i40e: Check and set the PF driver state first in i40e_ndo_set_vf_mac (diff) | |
download | linux-dev-856e6d9f9dd223dea978f1300f7c26903b9f9f6d.tar.xz linux-dev-856e6d9f9dd223dea978f1300f7c26903b9f9f6d.zip |
nfp: flower: use struct_size() helper
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 nfp_tun_active_tuns {
...
struct route_ip_info {
__be32 ipv4;
__be32 egress_port;
__be32 extra[2];
} tun_info[];
};
Make use of the struct_size() helper instead of an open-coded version
in order to avoid any potential type mistakes.
So, replace the following form:
sizeof(struct nfp_tun_active_tuns) + sizeof(struct route_ip_info) * count
with:
struct_size(payload, tun_info, count)
This code was detected with the help of Coccinelle.
Signed-off-by: Gustavo A. R. Silva <gustavo@embeddedor.com>
Acked-by: Jakub Kicinski <jakub.kicinski@netronome.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net')
0 files changed, 0 insertions, 0 deletions