[libvirt] [PATCH] bridge: Fix uninitialized variable
Daniel P. Berrange
berrange at redhat.com
Tue Jan 4 16:10:22 UTC 2011
On Mon, Jan 03, 2011 at 08:11:57AM -0500, Laine Stump wrote:
> On 01/03/2011 04:19 AM, Jiri Denemark wrote:
> >>>@@ -715,7 +715,7 @@ brDelInetAddress(brControl *ctl ATTRIBUTE_UNUSED,
> >>> virSocketAddr *addr,
> >>> unsigned int prefix)
> >>> {
> >>>- virCommandPtr cmd;
> >>>+ virCommandPtr cmd = NULL;
> >>> char *addrstr;
> >>> int ret = -1;
> >>>
> >>Wow! How did I miss *that*?!?
> >>
> >>Thanks for catching it!
> >>
> >>(Beyond that - what options are you using that it didn't build?
> >I always build with --enable-compile-warnings=error so any compiler warning
> >results in build failure. No special options beyond that.
>
> Right. I do that too, and never saw an error. Your compiler must be
> warning on more things than mine. Here's my build commandline
> (copied from Jim nearly 2 years ago, and pretty much unchanged
> since):
>
> ./autogen.sh --with-qemu-user=qemu --with-qemu-group=qemu
> --enable-compile-warnings=error --prefix=/usr --libdir=/usr/lib64
> --disable-nls CFLAGS="-g -Wformat -Wformat-security -Winit-self
> -Wall -Wpointer-arith -Wextra -Wshadow -Wcast-align -Wwrite-strings
> -Winline -Wno-sign-compare -Wp,-D_FORTIFY_SOURCE=2 -Dlint"
All those CFLAGS except for '-Dlint' are set by --enable-compile-warnings
so is redundant. You can also use --system instead of --prefix and
--libdir, which ensures you should build with a RPM-compatible prefix
Daniel
More information about the libvir-list
mailing list