Re: Entering freeze for libvirt-2.4.0

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

 



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.

> 
> 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.

Michal

--
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]