Re: [PATCH 0/6] More Coverity changes

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

 



On 01/22/2013 03:09 PM, John Ferlan wrote:
> Like a gift that keeps on giving... More Coverity changes.
> 
> Adjust changes from earlier based on Eric's comments/feedback.
> 
> nodeinfo.c - use sa_assert() instead of comment:
> https://www.redhat.com/archives/libvir-list/2013-January/msg01578.html
> 
> virnetclient.c - remove change - the VIR_FREE change resolves this:
> https://www.redhat.com/archives/libvir-list/2013-January/msg01581.html
> 
> Two new Coverity warnings popped up today (lxc_driver.c & lxc_process.c).
> No rhyme or reason, they just were new with no recent change to blame
> (very strange).

Not entirely - if I understand it, Coverity tries to do some duplicate
and priority suppression to avoid flooding the report with the same
minor error over and over again.  Based on whatever heuristics it is
using, some errors won't show up in reports until enough other errors
have been cleared to force the remaining problems to no longer be in the
noise.

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature

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