[libvirt] [PATCH 00/11] Revisit xen driver Coverity cleanup changes

John Ferlan jferlan at redhat.com
Wed Jan 30 18:51:35 UTC 2013


This is a rework of a couple of patches. Initially the changes were to
just remove some Coverity warnings; however, it was pointed out that each
of the xen drivers could use some cleanup since they were duplicating 
checks in libvirt.c, see:

https://www.redhat.com/archives/libvir-list/2013-January/msg01038.html
https://www.redhat.com/archives/libvir-list/2013-January/msg01057.html

While I was at it - I cleaned up some of the function headers.

Additionally, a revisit of a change to add Coverity false positive tags
to the v0 & v1 xen_hypervisor cpumap manipulation, see:

https://www.redhat.com/archives/libvir-list/2013-January/msg01527.html

and followups. 


John Ferlan (11):
  xs:  Remove redundant validity checks
  xs: Code cleanup
  inotify: Code cleanup
  xm: Remove redundant validity checks
  xm: Clean up
  xend: Remove redundant validity checks
  xend: Fix a memory leak found by Coverity
  xend: Clean up
  hypervisor: Remove redundant validity checks
  hypervisor: Clean up
  hypervisor: Revisit Coverity issues regarding cpumap

 src/xen/xen_hypervisor.c | 220 ++++++++--------------------------
 src/xen/xen_inotify.c    |  48 ++++----
 src/xen/xend_internal.c  | 303 ++++++++++++++---------------------------------
 src/xen/xm_internal.c    | 195 ++++++++++++++----------------
 src/xen/xs_internal.c    | 239 +++++++++++--------------------------
 5 files changed, 325 insertions(+), 680 deletions(-)

-- 
1.7.11.7




More information about the libvir-list mailing list