Date: Thu, 14 Nov 2019 21:07:10 -0800 From: Kees Cook <keescook@...omium.org> To: "David S. Miller" <davem@...emloft.net> Cc: Kees Cook <keescook@...omium.org>, Ariel Elior <aelior@...vell.com>, Sudarsana Kalluru <skalluru@...vell.com>, GR-everest-linux-l2@...vell.com, Sami Tolvanen <samitolvanen@...gle.com>, netdev@...r.kernel.org, kernel-hardening@...ts.openwall.com, linux-kernel@...r.kernel.org Subject: [PATCH 0/5] bnx2x: Remove function casts In order to make the entire kernel usable under Clang's Control Flow Integrity protections, function prototype casts need to be avoided because this will trip CFI checks at runtime (i.e. a mismatch between the caller's expected function prototype and the destination function's prototype). Many of these cases can be found with -Wcast-function-type, which found that bnx2x had a bunch of needless (or at least confusing) function casts. This series removes them all. -Kees Kees Cook (5): bnx2x: Drop redundant callback function casts bnx2x: Remove read_status_t function casts bnx2x: Remove config_init_t function casts bnx2x: Remove format_fw_ver_t function casts bnx2x: Remove hw_reset_t function casts .../net/ethernet/broadcom/bnx2x/bnx2x_link.c | 351 +++++++++--------- .../net/ethernet/broadcom/bnx2x/bnx2x_link.h | 6 +- 2 files changed, 171 insertions(+), 186 deletions(-) -- 2.17.1
Powered by blists - more mailing lists
Confused about mailing lists and their use? Read about mailing lists on Wikipedia and check out these guidelines on proper formatting of your messages.