mwl8k: Use named struct for memcpy() region
authorKees Cook <keescook@chromium.org>
Fri, 19 Nov 2021 00:49:05 +0000 (16:49 -0800)
committerKalle Valo <kvalo@codeaurora.org>
Mon, 29 Nov 2021 10:46:07 +0000 (12:46 +0200)
commitf01b3774309fc60f8bf27354014459b63e667b32
tree09e7c6cda603ec09ca972f5d03772766014c1339
parent601d2293e27f6fb37b30b836d724333a4ed79fe9
mwl8k: Use named struct for memcpy() region

In preparation for FORTIFY_SOURCE performing compile-time and run-time
field bounds checking for memcpy(), memmove(), and memset(), avoid
intentionally writing across neighboring fields.

Use named struct in struct mwl8k_cmd_set_key around members key_material,
tkip_tx_mic_key, and tkip_rx_mic_key so they can be referenced
together. This will allow memcpy() and sizeof() to more easily reason
about sizes, improve readability, and avoid future warnings about writing
beyond the end of key_material.

"pahole" shows no size nor member offset changes to struct
mwl8k_cmd_set_key. "objdump -d" shows no object code changes.

Signed-off-by: Kees Cook <keescook@chromium.org>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
Link: https://lore.kernel.org/r/20211119004905.2348143-1-keescook@chromium.org
drivers/net/wireless/marvell/mwl8k.c