Memory leak fix and some refactoring

[libvirt] [PATCH 0/2] Memory leak fix and some refactoring

Marc Hartmayer posted 2 patches 6 years, 2 months ago

Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/20180202121346.7991-1-mhartmay@linux.vnet.ibm.com

src/qemu/qemu_domain.c  | 23 +++++++++++++++++++++++
src/qemu/qemu_domain.h  |  2 ++
src/qemu/qemu_driver.c  | 12 ++----------
src/qemu/qemu_process.c | 41 ++++++++++++++---------------------------
4 files changed, 41 insertions(+), 37 deletions(-)

Expand all Fold all

[libvirt] [PATCH 0/2] Memory leak fix and some refactoring

Posted by Marc Hartmayer 6 years, 2 months ago

The second patch fixes the memory leak.

Marc Hartmayer (2):
  qemu: Use the return value of virObjectRef directly
  qemu: Add and use qemuProcessEventFree for freeing qemuProcessEvents

 src/qemu/qemu_domain.c  | 23 +++++++++++++++++++++++
 src/qemu/qemu_domain.h  |  2 ++
 src/qemu/qemu_driver.c  | 12 ++----------
 src/qemu/qemu_process.c | 41 ++++++++++++++---------------------------
 4 files changed, 41 insertions(+), 37 deletions(-)

-- 
2.13.4

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

Re: [libvirt] [PATCH 0/2] Memory leak fix and some refactoring

Posted by Michal Privoznik 6 years, 2 months ago

On 02/02/2018 01:13 PM, Marc Hartmayer wrote:
> The second patch fixes the memory leak.
> 
> Marc Hartmayer (2):
>   qemu: Use the return value of virObjectRef directly
>   qemu: Add and use qemuProcessEventFree for freeing qemuProcessEvents
> 
>  src/qemu/qemu_domain.c  | 23 +++++++++++++++++++++++
>  src/qemu/qemu_domain.h  |  2 ++
>  src/qemu/qemu_driver.c  | 12 ++----------
>  src/qemu/qemu_process.c | 41 ++++++++++++++---------------------------
>  4 files changed, 41 insertions(+), 37 deletions(-)
> 

I've fixed all the issues raised and pushed.

Thanks,
Michal

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

Re: [libvirt] [PATCH 0/2] Memory leak fix and some refactoring

Posted by Marc Hartmayer 6 years, 2 months ago

On Mon, Feb 05, 2018 at 10:35 AM +0100, Michal Privoznik <mprivozn@redhat.com> wrote:
> On 02/02/2018 01:13 PM, Marc Hartmayer wrote:
>> The second patch fixes the memory leak.
>> 
>> Marc Hartmayer (2):
>>   qemu: Use the return value of virObjectRef directly
>>   qemu: Add and use qemuProcessEventFree for freeing qemuProcessEvents
>> 
>>  src/qemu/qemu_domain.c  | 23 +++++++++++++++++++++++
>>  src/qemu/qemu_domain.h  |  2 ++
>>  src/qemu/qemu_driver.c  | 12 ++----------
>>  src/qemu/qemu_process.c | 41 ++++++++++++++---------------------------
>>  4 files changed, 41 insertions(+), 37 deletions(-)
>> 
>
> I've fixed all the issues raised and pushed.

Thanks!

>
> Thanks,
> Michal
>
-- 
Beste Grüße / Kind regards
   Marc Hartmayer

IBM Deutschland Research & Development GmbH
Vorsitzende des Aufsichtsrats: Martina Koederitz
Geschäftsführung: Dirk Wittkopp
Sitz der Gesellschaft: Böblingen
Registergericht: Amtsgericht Stuttgart, HRB 243294


--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list