[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