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 -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list