lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Sun, 9 Aug 2020 23:24:16 +0200 (CEST) From: Michal Kubecek <mkubecek@...e.cz> To: netdev@...r.kernel.org Subject: [PATCH ethtool 0/7] compiler warnings cleanup, part 2 Two compiler warnings still appear when compiling current source: comparison between signed and unsigned values and missing struct member initializations. The former are mostly handled by declaring variables (loop iterators, mostly) as unsigned, only few required an explicit cast. The latter are handled by using named field initializers; in link_mode_info[] array, helper macros are also used to make code easier to read and check. As the final step, add "-Wextra" to default CFLAGS to catch any future warnings as early as possible. Michal Kubecek (7): netlink: get rid of signed/unsigned comparison warnings ioctl: check presence of eeprom length argument properly ioctl: get rid of signed/unsigned comparison warnings get rid of signed/unsigned comparison warnings in register dump parsers settings: simplify link_mode_info[] initializers ioctl: convert cmdline_info arrays to named initializers build: add -Wextra to default CFLAGS Makefile.am | 2 +- dsa.c | 2 +- ethtool.c | 435 ++++++++++++++++++++++++++++++++++----------- fec.c | 2 +- ibm_emac.c | 2 +- marvell.c | 2 +- natsemi.c | 2 +- netlink/features.c | 4 +- netlink/netlink.c | 4 +- netlink/netlink.h | 2 +- netlink/nlsock.c | 2 +- netlink/parser.c | 2 +- netlink/settings.c | 242 ++++++++++--------------- rxclass.c | 8 +- sfpdiag.c | 2 +- tg3.c | 4 +- 16 files changed, 439 insertions(+), 278 deletions(-) -- 2.28.0
Powered by blists - more mailing lists