OSDN Git Service

libertas_tf: Use struct_group() for memcpy() region
authorKees Cook <keescook@chromium.org>
Thu, 18 Nov 2021 18:41:21 +0000 (10:41 -0800)
committerKalle Valo <kvalo@codeaurora.org>
Mon, 29 Nov 2021 10:44:55 +0000 (12:44 +0200)
commit642a57475b30213ebac1f7061eaf4d0415d9c5a1
tree5ad723126b60363489b4ca29c6f710aded72f5b6
parent5fd32ae0433a7af37eac27ace3aabae8cb7c2fc4
libertas_tf: Use struct_group() for memcpy() region

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

Use struct_group() in struct txpd around members tx_dest_addr_high
and tx_dest_addr_low 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 tx_dest_addr_high.

"pahole" shows no size nor member offset changes to struct txpd.
"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/20211118184121.1283821-1-keescook@chromium.org
drivers/net/wireless/marvell/libertas_tf/libertas_tf.h
drivers/net/wireless/marvell/libertas_tf/main.c