[libvirt] [PATCH 0/6] More Coverity changes

Eric Blake eblake at redhat.com
Tue Jan 22 22:29:36 UTC 2013


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

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 621 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20130122/545b4e24/attachment-0001.sig>


More information about the libvir-list mailing list