Current-Users archive

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

Re: Build failure - port amd64 - sys/kern/subr_autoconf.c



On Wed, 5 Mar 2008, Paul Goyette wrote:

From sources updated on 2008-03-05 at 12:21:24 UTC

#   compile  ZIPPY/subr_autoconf.o
/usr/obj/tooldir/x86_64/amd64/bin/x86_64--netbsd-gcc -mcmodel=kernel -mno-red-zone -ffreestanding -fno-zero-initialized-in-bss -g -O2 -fno-omit-frame-pointer -std=gnu99 -Werror -Wall -Wno-main -Wno-format-zero-length -Wpointer-arith -Wmissing-prototypes -Wstrict-prototypes -Wswitch -Wshadow -Wcast-qual -Wwrite-strings -Wno-sign-compare -Wno-pointer-sign -Wno-attributes -fno-strict-aliasing -Werror -Wall -Wno-main -Wno-format-zero-length -Wpointer-arith -Wmissing-prototypes -Wstrict-prototypes -Wswitch -Wshadow -Wcast-qual -Wwrite-strings -Wno-sign-compare -Wno-pointer-sign -Wno-attributes -Werror -Damd64 -Dx86_64 -I. -I/build/src/sys/../common/include -I/build/src/sys/arch -I/build/src/sys -nostdinc -DLKM -DDIAGNOSTIC -DMAXUSERS=64 -D_KERNEL -D_KERNEL_OPT -I/build/src/sys/lib/libkern/../../../common/lib/libc/quad -I/build/src/sys/lib/libkern/../../../common/lib/libc/string -I/build/src/sys/lib/libkern/../../../common/lib/libc/arch/x86_64/string -I/build/src/sys/dist/ipf -c /build/src/sys/kern/subr_autoconf.c
cc1: warnings being treated as errors
/build/src/sys/kern/subr_autoconf.c: In function 'deviter_init':
/build/src/sys/kern/subr_autoconf.c:2255: warning: 'rw' is used uninitialized in this function
1 error

This is a custom kernel config, but there's really nothing special about it. It's just a very stripped-down version of GENERIC. The config file is attached to this message just in case it does make a difference.

The following patch should take care of this error:

Index: subr_autoconf.c
===================================================================
RCS file: /cvsroot/src/sys/kern/subr_autoconf.c,v
retrieving revision 1.136
diff -u -p -r1.136 subr_autoconf.c
--- subr_autoconf.c     5 Mar 2008 07:09:18 -0000       1.136
+++ subr_autoconf.c     5 Mar 2008 14:03:18 -0000
@@ -2250,7 +2250,7 @@ void
 deviter_init(deviter_t *di, deviter_flags_t flags)
 {
        device_t dv;
-       bool rw;
+       bool rw = (flags & (DEVITER_F_RW | DEVITER_F_SHUTDOWN)) != 0;

        KASSERT(!rw || curlwp != NULL);

Note that I've initialized rw based on both the RW and SHUTDOWN flags since the comments state that SHUTDOWN implies RW.


----------------------------------------------------------------------
|   Paul Goyette   | PGP DSS Key fingerprint: |  E-mail addresses:   |
| Customer Service | FA29 0E3B 35AF E8AE 6651 |  paul%whooppee.com@localhost   |
| Network Engineer | 0786 F758 55DE 53BA 7731 | pgoyette%juniper.net@localhost |
----------------------------------------------------------------------


Home | Main Index | Thread Index | Old Index