fix a memory leak on qemuProcessReconnect

[libvirt PATCH 0/3] fix a memory leak on qemuProcessReconnect

Ján Tomko posted 3 patches 4 years, 2 months ago

Download series mbox

Test syntax-check failed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/cover.1600084134.git.jtomko@redhat.com

src/qemu/qemu_domain.c    | 2 +-
src/qemu/qemu_domainjob.c | 7 +++++--
src/qemu/qemu_domainjob.h | 3 ++-
src/qemu/qemu_process.c   | 2 +-
4 files changed, 9 insertions(+), 5 deletions(-)

Expand all Fold all

[libvirt PATCH 0/3] fix a memory leak on qemuProcessReconnect

Posted by Ján Tomko 4 years, 2 months ago

The first two patches have no functional impact on current code.

Ján Tomko (3):
  qemu: rename qemuDomainObjFreeJob -> qemuDomainObjClearJob
  qemu: qemuDomainObjClearJob: use g_clear_pointer
  qemuProcessReconnect: clear 'oldjob'

 src/qemu/qemu_domain.c    | 2 +-
 src/qemu/qemu_domainjob.c | 7 +++++--
 src/qemu/qemu_domainjob.h | 3 ++-
 src/qemu/qemu_process.c   | 2 +-
 4 files changed, 9 insertions(+), 5 deletions(-)

-- 
2.26.2

Re: [libvirt PATCH 0/3] fix a memory leak on qemuProcessReconnect

Posted by Michal Privoznik 4 years, 2 months ago

On 9/14/20 1:49 PM, Ján Tomko wrote:
> The first two patches have no functional impact on current code.
> 
> Ján Tomko (3):
>    qemu: rename qemuDomainObjFreeJob -> qemuDomainObjClearJob
>    qemu: qemuDomainObjClearJob: use g_clear_pointer
>    qemuProcessReconnect: clear 'oldjob'
> 
>   src/qemu/qemu_domain.c    | 2 +-
>   src/qemu/qemu_domainjob.c | 7 +++++--
>   src/qemu/qemu_domainjob.h | 3 ++-
>   src/qemu/qemu_process.c   | 2 +-
>   4 files changed, 9 insertions(+), 5 deletions(-)
> 

Reviewed-by: Michal Privoznik <mprivozn@redhat.com>

Michal