[libvirt] [PATCH 0/2] Resolve Coverity issues

John Ferlan jferlan at redhat.com
Wed May 1 15:53:27 UTC 2013


The following patches resolve a couple of Coverity issues.  The 'lxc' change
is merely a work-around by adding a Coverity tag to avoid what I consider to
be a false positive.  Another option to resolving is to change the code in
lxcContainerPrepareRoot() to assign 'tmp = root->src' and then VIR_FREE(tmp)
instead.  I have submitted a bug to Coverity for this.  The issue first 
showed up earlier this month and I've tried various ways to resolve without
much luck.  The 'virsh' change is from a more recent change where the ':'
option processing was changed in such a way that Coverity's analysis found
a missing break. The loop in question should find a match with optopt, but
if it didn't it would have fallen into the '?' code.

John Ferlan (2):
  lxc: Coverity false positive USE_AFTER_FREE
  virsh: Resolve Coverity 'MISSING_BREAK'

 src/lxc/lxc_container.c |  5 +++++
 tools/virsh.c           | 10 +++++++---
 2 files changed, 12 insertions(+), 3 deletions(-)

-- 
1.8.1.4




More information about the libvir-list mailing list