Re: [PATCH] Fix uninitialized variables

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

 



On Fri, Nov 30, 2012 at 06:38:31PM +0000, Daniel P. Berrange wrote:
> On Fri, Nov 30, 2012 at 07:13:01PM +0100, Guido Günther wrote:
> > detecet by
> > 
> > 	http://honk.sigxcpu.org:8001/job/libvirt-build/348/console
> > ---
> > Probably o.k. to push under the build breaker rule but I'd better check.
> > Cheers,
> >  -- Guido
> > 
> > 
> >  src/qemu/qemu_monitor.c |    4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> > 
> > diff --git a/src/qemu/qemu_monitor.c b/src/qemu/qemu_monitor.c
> > index c1f7c41..f85bb76 100644
> > --- a/src/qemu/qemu_monitor.c
> > +++ b/src/qemu/qemu_monitor.c
> > @@ -2408,7 +2408,7 @@ int qemuMonitorRemoveHostNetwork(qemuMonitorPtr mon,
> >                                   int vlan,
> >                                   const char *netname)
> >  {
> > -    int ret;
> > +    int ret = -1;
> >      VIR_DEBUG("mon=%p netname=%s",
> >            mon, netname);
> >  
> > @@ -2541,7 +2541,7 @@ int qemuMonitorAttachDrive(qemuMonitorPtr mon,
> >            mon, drivestr,
> >            controllerAddr->domain, controllerAddr->bus,
> >            controllerAddr->slot, controllerAddr->function);
> > -    int ret;
> > +    int ret = 1;
> >  
> >      if (!mon) {
> >          virReportError(VIR_ERR_INVALID_ARG, "%s",
> 
> ACK

Build back to green. Thanks,
 -- Guido

> 
> 
> Daniel
> -- 
> |: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
> |: http://libvirt.org              -o-             http://virt-manager.org :|
> |: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
> |: http://entangle-photo.org       -o-       http://live.gnome.org/gtk-vnc :|
> 

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]