On 23.01.2013 23:04, John Ferlan wrote:
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(-)
ACKed and pushed.
Michal