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
Attachments:
- signature.asc (application/pgp-signature — 621 bytes)