[libvirt] [PATCH 0/5] Resolve parallels and virCommandPtr resource leaks

John Ferlan jferlan at redhat.com
Wed Jan 23 22:04:43 UTC 2013


This is v3 of the parallels_driver.c changes. The most recent review pointed
out that virCommandPtr's need to be free'd. I found a few other instances
where they weren't free'd.  I also found a few instances where the status
check from virCommandRun was not < 0 and cleaned them to be more consistent.

John Ferlan (5):
  parallels: Resolve some resource leaks
  security: Need to add virCommandFree()
  storage: Need to add virCommandFree()
  util: Need to add virCommandFree()
  parallels_utils: Check return status properly from virCommandRun()

 src/parallels/parallels_driver.c | 47 ++++++++++++++++++++++++----------------
 src/parallels/parallels_utils.c  |  2 +-
 src/security/security_apparmor.c |  1 +
 src/storage/storage_backend_fs.c |  2 ++
 src/util/virnetdevopenvswitch.c  |  2 ++
 5 files changed, 34 insertions(+), 20 deletions(-)

-- 
1.7.11.7




More information about the libvir-list mailing list