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

Re: [Libvirt-announce] [libvirt] Entering freeze for libvirt-2.4.0



On Mon, Oct 31, 2016 at 03:09:05AM +0800, Michal Privoznik wrote:
> On 28.10.2016 05:59, John Ferlan wrote:
> > 
> > 
> > On 10/27/2016 05:04 PM, Daniel Veillard wrote:
> >>    As planned I just tagged the release in git and pushed signed tarball
> >> and rpms to the usual place:
> >>
> >>     ftp://libvirt.org/libvirt/
> >>
> >>   Not in my normal environment this week so I hadn't had a chance to test
> >> the build before pushing, sorry about this but things went smoothly, I noticed
> >> a warning in virsh however:
> >>
> >>   CC       virsh-virsh-host.o
> >> vsh.c: In function 'vshReadlineParse':
> >> vsh.c:516:9: warning: 'opts_need_arg' may be used uninitialized in this function [-Wmaybe-uninitialized]
> >>      i = ffsl(*opts_need_arg) - 1;
> >>          ^~~~~~~~~~~~~~~~~~~~
> >> vsh.c:2658:14: note: 'opts_need_arg' was declared here
> >>      uint64_t opts_need_arg, opts_seen;
> >>               ^~~~~~~~~~~~~
> > 
> 
> What's the compiler version you're using? I'm running gcc 5.4.0 and I
> don't see this message. And I don't use any special CFLAGS. I've even
> tried compiling on Fedora and still nothing.

  Normal Fedora 24 on x86-64,
 gcc version 6.2.1 20160916 (Red Hat 6.2.1-2) (GCC) 

> > 
> > Ewww... In code no one really wants to touch from commit id 'dcfdf341'.
> > 
> > Whatever compiler you have is the first to notice it I think. I haven't
> > looked closely at the Jenkins output lately and usually when someone
> > notices something like this with their new compiler, they post a patch.
> 
> I've taken look at Jenkins and don't see this problem there either.
> 
> > 
> > Anwyay I wonder if the parameter should have been "const_opts_need_arg"
> > (and similarly for the opts_seen argument to be const_opts_seen) -
> > similar to the call to vshCmddefOptParse earlier in that awful loop...
> 
> I think we can just initialize the opts_need_arg with const_opts_need_arg.

Daniel

-- 
Daniel Veillard      | Open Source and Standards, Red Hat
veillard redhat com  | libxml Gnome XML XSLT toolkit  http://xmlsoft.org/
http://veillard.com/ | virtualization library  http://libvirt.org/


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