Re: [PATCH v2 0/7] Resolve Coverity issues

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

 



On 07/11/2013 12:02 PM, Daniel P. Berrange wrote:
> On Thu, Jul 11, 2013 at 11:16:22AM -0400, John Ferlan wrote:
>> v1:
>> https://www.redhat.com/archives/libvir-list/2013-July/msg00691.html
>>
>> Changes over v1:
>>  * Add additional files to patch #2 to include other places were
>>    sysconf(_SC_OPEN_MAX) was used within libvirt sources.
>>  * Change for loop end conditions to use "<variable> != -1"  (should have
>>    looked deeper at other changes - wasn't even thinking of that option)
>>
>> John Ferlan (7):
>>   hellolibvirt: Resolve Coverity issues
>>   testutils: Resolve Coverity issues
>>   virsh-domain-monitor: Resolve Coverity issues
>>   virsh-interface: Resolve Coverity issues
>>   virsh-nodedev: Resolve Coverity issues
>>   storage_backend: Resolve Coverity issue
>>   qemu_hostdev: Resolve Coverity issue
> 
> ACK to all
> 
> 
> Daniel
> 
Pushed

Thanks for the quick reviews!

John

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