[libvirt] [PATCH 0/5] Resolve parallels and virCommandPtr resource leaks
Michal Privoznik
mprivozn at redhat.com
Thu Jan 24 11:40:36 UTC 2013
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
More information about the libvir-list
mailing list