From owner-freebsd-ports-bugs@freebsd.org Tue Jul 12 08:38:20 2016 Return-Path: Delivered-To: freebsd-ports-bugs@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 0F67DB937B7 for ; Tue, 12 Jul 2016 08:38:20 +0000 (UTC) (envelope-from bugzilla-noreply@freebsd.org) Received: from kenobi.freebsd.org (kenobi.freebsd.org [IPv6:2001:1900:2254:206a::16:76]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id E71431ABB for ; Tue, 12 Jul 2016 08:38:19 +0000 (UTC) (envelope-from bugzilla-noreply@freebsd.org) Received: from bugs.freebsd.org ([127.0.1.118]) by kenobi.freebsd.org (8.15.2/8.15.2) with ESMTP id u6C8cJYv058430 for ; Tue, 12 Jul 2016 08:38:19 GMT (envelope-from bugzilla-noreply@freebsd.org) From: bugzilla-noreply@freebsd.org To: freebsd-ports-bugs@FreeBSD.org Subject: [Bug 210798] devel/gdb: Uninitialized variables found presenting possible security issues or bugs Date: Tue, 12 Jul 2016 08:38:19 +0000 X-Bugzilla-Reason: AssignedTo X-Bugzilla-Type: changed X-Bugzilla-Watch-Reason: None X-Bugzilla-Product: Ports & Packages X-Bugzilla-Component: Individual Port(s) X-Bugzilla-Version: Latest X-Bugzilla-Keywords: needs-patch, needs-qa X-Bugzilla-Severity: Affects Only Me X-Bugzilla-Who: markmi@dsl-only.net X-Bugzilla-Status: Open X-Bugzilla-Resolution: X-Bugzilla-Priority: --- X-Bugzilla-Assigned-To: freebsd-ports-bugs@FreeBSD.org X-Bugzilla-Flags: maintainer-feedback+ X-Bugzilla-Changed-Fields: Message-ID: In-Reply-To: References: Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Bugzilla-URL: https://bugs.freebsd.org/bugzilla/ Auto-Submitted: auto-generated MIME-Version: 1.0 X-BeenThere: freebsd-ports-bugs@freebsd.org X-Mailman-Version: 2.1.22 Precedence: list List-Id: Ports bug reports List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 12 Jul 2016 08:38:20 -0000 https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=3D210798 --- Comment #7 from Mark Millard --- [Other things have taken my time but here are the results of one test of the patch: amd64 context.] I applied your patch on an amd64 11.0 -r302457 context with /usr/ports at -r4178253 and rebuilt gdb, generating a typescript file via script. grep'ing that typescript file for "warning:" shows: # grep warning: ~/ports_typescripts/gdb_patch_typescript checking for memmem... inflate.c:1507:61: warning: shifting a negative sign= ed value is undefined [-Wshift-negative-value] ./simple-object-xcoff.c:330:12: warning: using extended field designator is= an extension [-Wextended-offsetof] ./simple-object-xcoff.c:332:39: warning: using extended field designator is= an extension [-Wextended-offsetof] ./simple-object-xcoff.c:335:29: warning: using extended field designator is= an extension [-Wextended-offsetof] ./simple-object-xcoff.c:342:12: warning: using extended field designator is= an extension [-Wextended-offsetof] ./simple-object-xcoff.c:344:39: warning: using extended field designator is= an extension [-Wextended-offsetof] ./simple-object-xcoff.c:347:29: warning: using extended field designator is= an extension [-Wextended-offsetof] ./simple-object-xcoff.c:475:32: warning: using extended field designator is= an extension [-Wextended-offsetof] ./simple-object-xcoff.c:477:30: warning: using extended field designator is= an extension [-Wextended-offsetof] ./simple-object-xcoff.c:482:32: warning: using extended field designator is= an extension [-Wextended-offsetof] ./simple-object-xcoff.c:484:30: warning: using extended field designator is= an extension [-Wextended-offsetof] ./simple-object-xcoff.c:598:21: warning: using extended field designator is= an extension [-Wextended-offsetof] ./simple-object-xcoff.c:600:21: warning: using extended field designator is= an extension [-Wextended-offsetof] ./simple-object-xcoff.c:603:21: warning: using extended field designator is= an extension [-Wextended-offsetof] ./simple-object-xcoff.c:608:21: warning: using extended field designator is= an extension [-Wextended-offsetof] ./simple-object-xcoff.c:610:21: warning: using extended field designator is= an extension [-Wextended-offsetof] ./simple-object-xcoff.c:613:21: warning: using extended field designator is= an extension [-Wextended-offsetof] ./simple-object-xcoff.c:663:21: warning: using extended field designator is= an extension [-Wextended-offsetof] ./simple-object-xcoff.c:665:21: warning: using extended field designator is= an extension [-Wextended-offsetof] ./simple-object-xcoff.c:670:21: warning: using extended field designator is= an extension [-Wextended-offsetof] ./simple-object-xcoff.c:672:21: warning: using extended field designator is= an extension [-Wextended-offsetof] ./simple-object-xcoff.c:683:19: warning: using extended field designator is= an extension [-Wextended-offsetof] ./simple-object-xcoff.c:685:19: warning: using extended field designator is= an extension [-Wextended-offsetof] ./stack-limit.c:54:24: warning: comparison of integers of different signs: 'rlim_t' (aka 'long') and 'unsigned long' [-Wsign-compare] checking whether logf is declared without a macro... rl78-dis.c:232:44: warning: use of logical '||' with constant operand [-Wconstant-logical-oper= and] microblaze-tdep.c:94:28: warning: format string is not a string literal [-Wformat-nonliteral] msp430-tdep.c:401:24: warning: comparison of constant 16 with expression of type 'MSP430_Size' is always false [-Wtautological-constant-out-of-range-compare] score-tdep.c:819:12: warning: comparison of unsigned expression < 0 is alwa= ys false [-Wtautological-compare] sh64-tdep.c:977:42: warning: comparison of unsigned expression >=3D 0 is al= ways true [-Wtautological-compare] v850-tdep.c:562:35: warning: comparison of unsigned expression >=3D 0 is al= ways true [-Wtautological-compare] v850-tdep.c:562:66: warning: comparison of unsigned expression >=3D 0 is al= ways true [-Wtautological-compare] remote-mips.c:500:37: warning: format string is not a string literal [-Wformat-nonliteral] xcoffread.c:1027:41: warning: variable 'main_aux' is uninitialized when used here [-Wuninitialized] ./nat/x86-dregs.c:209:7: warning: variable 'i' is incremented both in the l= oop header and in the loop body [-Wfor-loop-analysis] ./tui/tui-stack.c:419:16: warning: expression which evaluates to zero treat= ed as a null pointer constant of type 'CORE_ADDR *' (aka 'unsigned long *') [-Wnon-literal-null-conversion] dtrace-probe.c:424:52: warning: while loop has empty body [-Wempty-body] main.c:229:56: warning: adding 'int' to a string does not append to the str= ing [-Wstring-plus-int] ada-lang.c:2489:50: warning: shifting a negative signed value is undefined [-Wshift-negative-value] ada-lang.c:2503:46: warning: shifting a negative signed value is undefined [-Wshift-negative-value] /usr/bin/ld: warning: libncurses.so.8, needed by /usr/local/lib/libreadline= .so, may conflict with libncurses.so.6 /usr/bin/ld: warning: libncurses.so.8, needed by /usr/local/lib/libreadline= .so, may conflict with libncurses.so.6 (libncurses.so.* issues are tied to /usr/lib and /lib vs. /usr/local/lib bindings.) Most of that is C source code being reported on, which is the focus here. I list them all simply because you indicated you are starting to attempt to eliminate many types of compiler reports. Certainly the various things that I reported earlier are not listed in the above: now gone, at least for amd64. As for the make.conf in use: # more /etc/make.conf WANT_QT_VERBOSE_CONFIGURE=3D1 # DEFAULT_VERSIONS+=3Dperl5=3D5.22 WRKDIRPREFIX=3D/usr/obj/portswork WITH_DEBUG=3D WITH_DEBUG_FILES=3D MALLOC_PRODUCTION=3D An armv6 build might be somewhat different but I've not done that test yet.= I will. [I will not have access to powerpc64 or powerpc again for weeks or months y= et. Currently I'm limited to amd64 and armv6 examples. So only little endian.] --=20 You are receiving this mail because: You are the assignee for the bug.=