Source-Changes-HG archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

[src/trunk]: src/usr.sbin/traceroute6 add support for new GCC 9 warnings that...



details:   https://anonhg.NetBSD.org/src/rev/f70dcb4d0915
branches:  trunk
changeset: 954546:f70dcb4d0915
user:      mrg <mrg%NetBSD.org@localhost>
date:      Sun Sep 06 07:20:26 2020 +0000

description:
add support for new GCC 9 warnings that may be too much to fix
right now.  new address-of-packed-member and format-overflow
warnings have new GCC_NO_ADDR_OF_PACKED_MEMBER amd
GCC_NO_FORMAT_OVERFLOW variables to remove these warnings.

apply to a bunch of the tree.  mostly, these are real bugs that
should be fixed, but in many cases, only by removing the 'packed'
attribute from some structure that doesn't really need it.  (i
looked at many different ones, and while perhaps 60-80% were
already properly aligned, it wasn't clear to me that the uses
were always coming from sane data vs network alignment, so it
doesn't seem safe to remove packed without careful research for
each affect struct.)  clang already warned (and was not erroring)
for many of these cases, but gcc picked up dozens more.

diffstat:

 external/apache2/mDNSResponder/usr.sbin/mdnsd/Makefile |   3 ++-
 external/bsd/ipf/Makefile.inc                          |   3 ++-
 external/bsd/ipf/bin/ipmon/Makefile                    |   4 +++-
 external/bsd/ntp/bin/ntpd/Makefile                     |  13 +++++--------
 external/bsd/tcpdump/bin/Makefile                      |   3 ++-
 external/bsd/wpa/bin/hostapd/Makefile                  |   4 +++-
 libexec/identd/Makefile                                |   3 ++-
 libexec/lfs_cleanerd/Makefile                          |   4 +++-
 sbin/dump/Makefile                                     |   4 +++-
 sbin/dump_lfs/Makefile                                 |   4 ++--
 sbin/fsck_ffs/Makefile                                 |   4 +++-
 sbin/fsck_lfs/Makefile                                 |   4 +++-
 sbin/fsck_v7fs/Makefile                                |   4 +++-
 sbin/fsdb/Makefile                                     |   4 +++-
 sbin/newfs/Makefile                                    |   4 +++-
 sbin/newfs_lfs/Makefile                                |   4 +++-
 sbin/newfs_udf/Makefile                                |   3 ++-
 sbin/newfs_v7fs/Makefile                               |   4 +++-
 sbin/nvmectl/Makefile                                  |   4 +++-
 sbin/resize_lfs/Makefile                               |   4 +++-
 sbin/scan_ffs/Makefile                                 |   4 +++-
 share/mk/bsd.kmodule.mk                                |   3 ++-
 share/mk/bsd.own.mk                                    |   4 +++-
 sys/arch/i386/stand/Makefile.booters                   |   4 +++-
 sys/arch/i386/stand/boot/Makefile.boot                 |   4 +++-
 sys/arch/i386/stand/bootxx/Makefile.bootxx             |   4 +++-
 sys/arch/i386/stand/efiboot/Makefile.efiboot           |   4 +++-
 sys/arch/i386/stand/efiboot/bootx64/Makefile           |   3 ++-
 sys/arch/i386/stand/lib/Makefile                       |   3 ++-
 sys/arch/macppc/stand/Makefile.inc                     |   3 ++-
 sys/arch/sparc/stand/boot/Makefile                     |   3 ++-
 sys/arch/sparc/stand/bootxx/Makefile                   |   3 ++-
 sys/arch/sparc/stand/ofwboot/Makefile                  |   3 ++-
 sys/arch/zaurus/stand/Makefile.inc                     |   3 ++-
 sys/conf/Makefile.kern.inc                             |   3 ++-
 sys/modules/udf/Makefile                               |   4 +++-
 sys/rump/fs/lib/libffs/Makefile                        |   5 ++++-
 sys/rump/fs/lib/liblfs/Makefile                        |   3 ++-
 sys/rump/fs/lib/libmsdos/Makefile                      |   4 +++-
 sys/rump/fs/lib/libudf/Makefile                        |   3 ++-
 sys/rump/fs/lib/libv7fs/Makefile                       |   4 +++-
 sys/rump/fs/lib/libzfs/Makefile                        |   4 +++-
 sys/rump/net/lib/Makefile.inc                          |   3 ++-
 sys/stand/efiboot/Makefile.efiboot                     |   3 ++-
 tests/lib/libc/sys/Makefile                            |   5 ++++-
 usr.bin/ftp/Makefile                                   |   4 +++-
 usr.sbin/bootp/common/Makefile                         |   3 ++-
 usr.sbin/dumplfs/Makefile                              |   4 +++-
 usr.sbin/fstyp/Makefile                                |   3 ++-
 usr.sbin/installboot/Makefile                          |   3 ++-
 usr.sbin/makefs/udf/Makefile.inc                       |   3 ++-
 usr.sbin/map-mbone/Makefile                            |   4 +++-
 usr.sbin/mrinfo/Makefile                               |   4 +++-
 usr.sbin/mrouted/Makefile                              |   3 ++-
 usr.sbin/mtrace/Makefile                               |   4 +++-
 usr.sbin/ndbootd/Makefile                              |   3 ++-
 usr.sbin/pf/Makefile.inc                               |   3 ++-
 usr.sbin/puffs/rump_lfs/Makefile                       |   3 ++-
 usr.sbin/racoon/Makefile                               |   4 +++-
 usr.sbin/rtadvd/Makefile                               |   3 ++-
 usr.sbin/traceroute/Makefile                           |   4 +++-
 usr.sbin/traceroute6/Makefile                          |   4 +++-
 62 files changed, 162 insertions(+), 70 deletions(-)

diffs (truncated from 1035 to 300 lines):

diff -r fefb3f2ef819 -r f70dcb4d0915 external/apache2/mDNSResponder/usr.sbin/mdnsd/Makefile
--- a/external/apache2/mDNSResponder/usr.sbin/mdnsd/Makefile    Sun Sep 06 05:31:46 2020 +0000
+++ b/external/apache2/mDNSResponder/usr.sbin/mdnsd/Makefile    Sun Sep 06 07:20:26 2020 +0000
@@ -1,4 +1,4 @@
-#      $NetBSD: Makefile,v 1.12 2018/11/18 18:58:44 christos Exp $
+#      $NetBSD: Makefile,v 1.13 2020/09/06 07:20:26 mrg Exp $
 
 PROG=  mdnsd
 
@@ -18,5 +18,6 @@
 MAN=   mdnsd.8
 
 CWARNFLAGS.clang+=     -Wno-unused-value -Wno-error=address-of-packed-member
+CWARNFLAGS.gcc+=       ${GCC_NO_ADDR_OF_PACKED_MEMBER}
 
 .include <bsd.prog.mk>
diff -r fefb3f2ef819 -r f70dcb4d0915 external/bsd/ipf/Makefile.inc
--- a/external/bsd/ipf/Makefile.inc     Sun Sep 06 05:31:46 2020 +0000
+++ b/external/bsd/ipf/Makefile.inc     Sun Sep 06 07:20:26 2020 +0000
@@ -1,10 +1,11 @@
-#      $NetBSD: Makefile.inc,v 1.4 2019/10/13 07:28:06 mrg Exp $
+#      $NetBSD: Makefile.inc,v 1.5 2020/09/06 07:20:26 mrg Exp $
 
 WARNS?=        1       # XXX -Wcast-qual -Wshadow
 CWARNFLAGS.clang+=     -Wno-format -Wno-tautological-compare \
                        -Wno-self-assign -Wno-array-bounds \
                        -Wno-error=unused-const-variable \
                        -Wno-error=address-of-packed-member
+CWARNFLAGS.gcc+=       ${GCC_NO_ADDR_OF_PACKED_MEMBER}
 
 .include <bsd.own.mk>
 
diff -r fefb3f2ef819 -r f70dcb4d0915 external/bsd/ipf/bin/ipmon/Makefile
--- a/external/bsd/ipf/bin/ipmon/Makefile       Sun Sep 06 05:31:46 2020 +0000
+++ b/external/bsd/ipf/bin/ipmon/Makefile       Sun Sep 06 07:20:26 2020 +0000
@@ -1,4 +1,4 @@
-#      $NetBSD: Makefile,v 1.3 2020/02/12 00:30:48 fox Exp $
+#      $NetBSD: Makefile,v 1.4 2020/09/06 07:20:26 mrg Exp $
 
 PROG=          ipmon
 SRCS=          ipmon.c ipmon_y.c ipmon_l.c
@@ -36,4 +36,6 @@
        ${TOOL_SED} -e 's/yy/ipmon_yy/g' \
            ${.ALLSRC} > ${.TARGET}
 
+COPTS.ipmon.c+=                ${GCC_NO_ADDR_OF_PACKED_MEMBER} ${GCC_NO_FORMAT_OVERFLOW}
+
 .include <bsd.prog.mk>
diff -r fefb3f2ef819 -r f70dcb4d0915 external/bsd/ntp/bin/ntpd/Makefile
--- a/external/bsd/ntp/bin/ntpd/Makefile        Sun Sep 06 05:31:46 2020 +0000
+++ b/external/bsd/ntp/bin/ntpd/Makefile        Sun Sep 06 07:20:26 2020 +0000
@@ -1,4 +1,4 @@
-#      $NetBSD: Makefile,v 1.26 2020/05/29 10:53:02 kamil Exp $
+#      $NetBSD: Makefile,v 1.27 2020/09/06 07:20:26 mrg Exp $
 
 .include <bsd.own.mk>
 
@@ -87,17 +87,14 @@
 .PATH: ${DIST}
 YHEADER=1
 
-# XXXGCC5
-.if defined(HAVE_GCC) && ${HAVE_GCC} >= 5 && ${ACTIVE_CC} == "gcc"
-COPTS.ntp_control.c+=  -Wno-error=maybe-uninitialized
-.endif
-
-COPTS.ntp_loopfilter.c+=       ${${ACTIVE_CC} == "gcc" && ${HAVE_GCC:U0} == 8:? -Wno-error=restrict :}
+COPTS.ntp_control.c+=          ${${ACTIVE_CC} == "gcc" && ${HAVE_GCC:U0} >= 7:? -Wno-error=maybe-uninitialized :}
+COPTS.ntp_loopfilter.c+=       ${${ACTIVE_CC} == "gcc" && ${HAVE_GCC:U0} >= 8:? -Wno-error=restrict :}
 COPTS.ntp_loopfilter.c+=       ${GCC_NO_FORMAT_TRUNCATION}
 COPTS.ntp_crypto.c+=           ${GCC_NO_FORMAT_TRUNCATION}
-COPTS.refclock_jjy.c+=         ${GCC_NO_STRINGOP_TRUNCATION}
+COPTS.refclock_jjy.c+=         ${GCC_NO_STRINGOP_TRUNCATION} ${GCC_NO_FORMAT_TRUNCATION}
 COPTS.refclock_parse.c+=       ${GCC_NO_FORMAT_TRUNCATION}
 COPTS.refclock_leitch.c+=      ${GCC_NO_FORMAT_TRUNCATION}
 COPTS.refclock_neoclock4x.c+=  ${GCC_NO_FORMAT_TRUNCATION}
+COPTS.ntp_io.c+=               ${GCC_NO_ADDR_OF_PACKED_MEMBER}
 
 .include <bsd.prog.mk>
diff -r fefb3f2ef819 -r f70dcb4d0915 external/bsd/tcpdump/bin/Makefile
--- a/external/bsd/tcpdump/bin/Makefile Sun Sep 06 05:31:46 2020 +0000
+++ b/external/bsd/tcpdump/bin/Makefile Sun Sep 06 07:20:26 2020 +0000
@@ -1,4 +1,4 @@
-#      $NetBSD: Makefile,v 1.19 2017/05/21 15:28:39 riastradh Exp $    
+#      $NetBSD: Makefile,v 1.20 2020/09/06 07:20:26 mrg Exp $  
 
 WARNS?=        1       # XXX: need to cleanup later
 
@@ -206,5 +206,6 @@
        cp ${.ALLSRC} ${.TARGET}
 
 CWARNFLAGS.clang+=     -Wno-error=address-of-packed-member
+CWARNFLAGS.gcc+=       ${GCC_NO_ADDR_OF_PACKED_MEMBER}
 
 .include <bsd.prog.mk>
diff -r fefb3f2ef819 -r f70dcb4d0915 external/bsd/wpa/bin/hostapd/Makefile
--- a/external/bsd/wpa/bin/hostapd/Makefile     Sun Sep 06 05:31:46 2020 +0000
+++ b/external/bsd/wpa/bin/hostapd/Makefile     Sun Sep 06 07:20:26 2020 +0000
@@ -1,4 +1,4 @@
-# $NetBSD: Makefile,v 1.13 2020/02/03 09:10:47 roy Exp $
+# $NetBSD: Makefile,v 1.14 2020/09/06 07:20:26 mrg Exp $
 
 .include "${.CURDIR}/../Makefile.inc"
 
@@ -215,4 +215,6 @@
 SRCS+= sha1-pbkdf2.c sha1.c md5.c
 .endif
 
+CWARNFLAGS.gcc+=       ${GCC_NO_ADDR_OF_PACKED_MEMBER}
+
 .include <bsd.prog.mk>
diff -r fefb3f2ef819 -r f70dcb4d0915 libexec/identd/Makefile
--- a/libexec/identd/Makefile   Sun Sep 06 05:31:46 2020 +0000
+++ b/libexec/identd/Makefile   Sun Sep 06 07:20:26 2020 +0000
@@ -1,4 +1,4 @@
-# $NetBSD: Makefile,v 1.18 2019/08/24 13:20:52 rmind Exp $
+# $NetBSD: Makefile,v 1.19 2020/09/06 07:20:26 mrg Exp $
 
 .include <bsd.own.mk>
 
@@ -28,5 +28,6 @@
 
 # XXX
 COPTS+=        -fno-strict-aliasing
+CWARNFLAGS.gcc+=       ${GCC_NO_ADDR_OF_PACKED_MEMBER}
 
 .include <bsd.prog.mk>
diff -r fefb3f2ef819 -r f70dcb4d0915 libexec/lfs_cleanerd/Makefile
--- a/libexec/lfs_cleanerd/Makefile     Sun Sep 06 05:31:46 2020 +0000
+++ b/libexec/lfs_cleanerd/Makefile     Sun Sep 06 07:20:26 2020 +0000
@@ -1,4 +1,4 @@
-#      $NetBSD: Makefile,v 1.19 2015/10/10 22:34:46 dholland Exp $
+#      $NetBSD: Makefile,v 1.20 2020/09/06 07:20:26 mrg Exp $
 #      @(#)Makefile    8.1 (Berkeley) 6/18/93
 
 .include "Makefile.inc"
@@ -11,4 +11,6 @@
 LDSTATIC?=      -static
 .endif
 
+CWARNFLAGS.gcc+=       ${GCC_NO_ADDR_OF_PACKED_MEMBER}
+
 .include <bsd.prog.mk>
diff -r fefb3f2ef819 -r f70dcb4d0915 sbin/dump/Makefile
--- a/sbin/dump/Makefile        Sun Sep 06 05:31:46 2020 +0000
+++ b/sbin/dump/Makefile        Sun Sep 06 07:20:26 2020 +0000
@@ -1,4 +1,4 @@
-#      $NetBSD: Makefile,v 1.40 2019/10/13 07:28:12 mrg Exp $
+#      $NetBSD: Makefile,v 1.41 2020/09/06 07:20:27 mrg Exp $
 #      @(#)Makefile    8.1 (Berkeley) 6/5/93
 
 #      dump.h                  header file
@@ -39,5 +39,7 @@
 
 COPTS.ffs_inode.c+=    -Wno-pointer-sign
 COPTS.traverse.c+=     ${GCC_NO_FORMAT_TRUNCATION}
+CWARNFLAGS.gcc+=       ${GCC_NO_ADDR_OF_PACKED_MEMBER}
+CWARNFLAGS.gcc+=       ${GCC_NO_FORMAT_TRUNCATION}
 
 .include <bsd.prog.mk>
diff -r fefb3f2ef819 -r f70dcb4d0915 sbin/dump_lfs/Makefile
--- a/sbin/dump_lfs/Makefile    Sun Sep 06 05:31:46 2020 +0000
+++ b/sbin/dump_lfs/Makefile    Sun Sep 06 07:20:26 2020 +0000
@@ -1,4 +1,4 @@
-#      $NetBSD: Makefile,v 1.17 2019/10/13 07:28:12 mrg Exp $
+#      $NetBSD: Makefile,v 1.18 2020/09/06 07:20:27 mrg Exp $
 #      @(#)Makefile    8.1 (Berkeley) 6/5/93
 
 #      lfs_inode.c             LFS filestore-specific routines
@@ -30,6 +30,6 @@
 .PATH:  ${NETBSDSRCDIR}/sbin/dump ${NETBSDSRCDIR}/sys/ufs/lfs \
     ${NETBSDSRCDIR}/sys/ufs/ffs
 
-COPTS.traverse.c+=     ${GCC_NO_FORMAT_TRUNCATION}
+CWARNFLAGS.gcc+=       ${GCC_NO_ADDR_OF_PACKED_MEMBER} ${GCC_NO_FORMAT_TRUNCATION}
 
 .include <bsd.prog.mk>
diff -r fefb3f2ef819 -r f70dcb4d0915 sbin/fsck_ffs/Makefile
--- a/sbin/fsck_ffs/Makefile    Sun Sep 06 05:31:46 2020 +0000
+++ b/sbin/fsck_ffs/Makefile    Sun Sep 06 07:20:26 2020 +0000
@@ -1,4 +1,4 @@
-#      $NetBSD: Makefile,v 1.49 2018/06/11 14:18:17 kamil Exp $
+#      $NetBSD: Makefile,v 1.50 2020/09/06 07:20:27 mrg Exp $
 #      @(#)Makefile    8.2 (Berkeley) 4/27/95
 
 # when making a change to this file, please check if the change is
@@ -13,5 +13,7 @@
 
 .include "Makefile.common"
 
+CWARNFLAGS.gcc+=       ${GCC_NO_ADDR_OF_PACKED_MEMBER}
+
 .include <bsd.prog.mk>
 .include <bsd.subdir.mk>
diff -r fefb3f2ef819 -r f70dcb4d0915 sbin/fsck_lfs/Makefile
--- a/sbin/fsck_lfs/Makefile    Sun Sep 06 05:31:46 2020 +0000
+++ b/sbin/fsck_lfs/Makefile    Sun Sep 06 07:20:26 2020 +0000
@@ -1,4 +1,4 @@
-#      $NetBSD: Makefile,v 1.20 2013/06/08 21:13:19 dholland Exp $
+#      $NetBSD: Makefile,v 1.21 2020/09/06 07:20:27 mrg Exp $
 #      @(#)Makefile    8.1 (Berkeley) 6/5/93
 
 WARNS?=        3       # XXX: sign-compare issues
@@ -19,4 +19,6 @@
 LDADD+=-lutil
 DPADD+=${LIBUTIL}
 
+CWARNFLAGS.gcc+=       ${GCC_NO_ADDR_OF_PACKED_MEMBER}
+
 .include <bsd.prog.mk>
diff -r fefb3f2ef819 -r f70dcb4d0915 sbin/fsck_v7fs/Makefile
--- a/sbin/fsck_v7fs/Makefile   Sun Sep 06 05:31:46 2020 +0000
+++ b/sbin/fsck_v7fs/Makefile   Sun Sep 06 07:20:26 2020 +0000
@@ -1,4 +1,4 @@
-#      $NetBSD: Makefile,v 1.2 2012/09/05 22:58:53 riz Exp $
+#      $NetBSD: Makefile,v 1.3 2020/09/06 07:20:27 mrg Exp $
 
 .include <bsd.own.mk>
 
@@ -17,4 +17,6 @@
 LDADD+=-lutil
 DPADD+=${LIBUTIL}
 
+CWARNFLAGS.gcc+=       ${GCC_NO_ADDR_OF_PACKED_MEMBER}
+
 .include <bsd.prog.mk>
diff -r fefb3f2ef819 -r f70dcb4d0915 sbin/fsdb/Makefile
--- a/sbin/fsdb/Makefile        Sun Sep 06 05:31:46 2020 +0000
+++ b/sbin/fsdb/Makefile        Sun Sep 06 07:20:26 2020 +0000
@@ -1,4 +1,4 @@
-#      $NetBSD: Makefile,v 1.39 2019/10/28 22:52:43 christos Exp $
+#      $NetBSD: Makefile,v 1.40 2020/09/06 07:20:27 mrg Exp $
 #      @(#)Makefile    8.1 (Berkeley) 6/5/93
 
 .include <bsd.own.mk>
@@ -44,4 +44,6 @@
 COPTS.inode.c+=        -O0
 .endif
 
+CWARNFLAGS.gcc+=       ${GCC_NO_ADDR_OF_PACKED_MEMBER}
+
 .include <bsd.prog.mk>
diff -r fefb3f2ef819 -r f70dcb4d0915 sbin/newfs/Makefile
--- a/sbin/newfs/Makefile       Sun Sep 06 05:31:46 2020 +0000
+++ b/sbin/newfs/Makefile       Sun Sep 06 07:20:26 2020 +0000
@@ -1,4 +1,4 @@
-#      $NetBSD: Makefile,v 1.40 2017/02/08 16:11:40 rin Exp $
+#      $NetBSD: Makefile,v 1.41 2020/09/06 07:20:27 mrg Exp $
 #      @(#)Makefile    8.2 (Berkeley) 3/27/94
 
 # when making a change to this file, please check if the change is
@@ -15,4 +15,6 @@
 
 .include "Makefile.common"
 
+CWARNFLAGS.gcc+=       ${GCC_NO_ADDR_OF_PACKED_MEMBER}
+
 .include <bsd.prog.mk>
diff -r fefb3f2ef819 -r f70dcb4d0915 sbin/newfs_lfs/Makefile
--- a/sbin/newfs_lfs/Makefile   Sun Sep 06 05:31:46 2020 +0000
+++ b/sbin/newfs_lfs/Makefile   Sun Sep 06 07:20:26 2020 +0000
@@ -1,4 +1,4 @@
-#      $NetBSD: Makefile,v 1.12 2013/06/08 21:13:19 dholland Exp $
+#      $NetBSD: Makefile,v 1.13 2020/09/06 07:20:27 mrg Exp $
 #      @(#)Makefile    8.1 (Berkeley) 6/18/93
 
 WARNS?=        3       # XXX: sign-compare issues
@@ -25,4 +25,6 @@
 
 CPPFLAGS+=-I${FSCK_LFS} -I${FSCK} # -DNDEBUG # -DVERBOSE_BLOCKMAP
 
+CWARNFLAGS.gcc+=       ${GCC_NO_ADDR_OF_PACKED_MEMBER}
+
 .include <bsd.prog.mk>
diff -r fefb3f2ef819 -r f70dcb4d0915 sbin/newfs_udf/Makefile
--- a/sbin/newfs_udf/Makefile   Sun Sep 06 05:31:46 2020 +0000
+++ b/sbin/newfs_udf/Makefile   Sun Sep 06 07:20:26 2020 +0000
@@ -1,4 +1,4 @@
-# $NetBSD: Makefile,v 1.6 2017/01/11 12:19:43 joerg Exp $
+# $NetBSD: Makefile,v 1.7 2020/09/06 07:20:27 mrg Exp $
 
 .include <bsd.own.mk>
 
@@ -15,5 +15,6 @@
 LDADD+=-lutil
 
 CWARNFLAGS.clang+=     -Wno-error=address-of-packed-member
+CWARNFLAGS.gcc+=       ${GCC_NO_ADDR_OF_PACKED_MEMBER}
 
 .include <bsd.prog.mk>
diff -r fefb3f2ef819 -r f70dcb4d0915 sbin/newfs_v7fs/Makefile
--- a/sbin/newfs_v7fs/Makefile  Sun Sep 06 05:31:46 2020 +0000
+++ b/sbin/newfs_v7fs/Makefile  Sun Sep 06 07:20:26 2020 +0000
@@ -1,4 +1,4 @@
-# $NetBSD: Makefile,v 1.4 2012/09/05 23:01:42 riz Exp $
+# $NetBSD: Makefile,v 1.5 2020/09/06 07:20:28 mrg Exp $
 
 .include <bsd.own.mk>
 
@@ -18,4 +18,6 @@



Home | Main Index | Thread Index | Old Index