net: sundance: Replace one-element array with non-array object
authorGustavo A. R. Silva <gustavoars@kernel.org>
Fri, 4 Feb 2022 23:29:06 +0000 (17:29 -0600)
committerDavid S. Miller <davem@davemloft.net>
Sat, 5 Feb 2022 15:30:32 +0000 (15:30 +0000)
commit5f2155132c5b9dbbf842db134a48407e5aad0958
tree993c4b0256bdcb2604e9896cfc57059d3b959aaf
parent76ad950c8fd7a4625908c679374f70d22272dfb3
net: sundance: Replace one-element array with non-array object

It seems this one-element array is not actually being used as an
array of variable size, so we can just replace it with just a
non-array object of type struct desc_frag and refactor a bit the
rest of the code.

This helps with the ongoing efforts to globally enable -Warray-bounds
and get us closer to being able to tighten the FORTIFY_SOURCE routines
on memcpy().

This issue was found with the help of Coccinelle and audited and fixed,
manually.

[1] https://en.wikipedia.org/wiki/Flexible_array_member
[2] https://www.kernel.org/doc/html/v5.16/process/deprecated.html#zero-length-and-one-element-arrays

Link: https://github.com/KSPP/linux/issues/79
Signed-off-by: Gustavo A. R. Silva <gustavoars@kernel.org>
Reviewed-by: Jakub Kicinski <kuba@kernel.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/dlink/sundance.c