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

Ján Tomko posted 3 patches 1 week ago
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(-)

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

Posted by Ján Tomko 1 week 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 1 week 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