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

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



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


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