[libvirt] [PATCH] qemu: Avoid memory leaks on qemuParseRBDString

2012-01-05 Thread ajia
From: Alex Jia a...@redhat.com Detected by valgrind. Leak introduced in commit 5745dc1. * src/qemu/qemu_command.c: fix memory leak on failure and successful path. * How to reproduce? % valgrind -v --leak-check=full ./qemuargv2xmltest * Actual result: ==2196== 80 bytes in 1 blocks are

Re: [libvirt] [PATCH] qemu: Avoid memory leaks on qemuParseRBDString

2012-01-05 Thread Osier Yang
On 2012年01月06日 14:36, a...@redhat.com wrote: From: Alex Jiaa...@redhat.com Detected by valgrind. Leak introduced in commit 5745dc1. * src/qemu/qemu_command.c: fix memory leak on failure and successful path. * How to reproduce? % valgrind -v --leak-check=full ./qemuargv2xmltest * Actual

[libvirt] [PATCH] qemu: avoid memory leaks

2011-08-02 Thread Eric Blake
Quite a few leaks detected by coverity. For chr, the leaks were close enough to the allocations to plug in place; for disk, the leaks were separated from the allocation by enough other lines with intermediate failure cases that I refactored the cleanup instead. * src/qemu/qemu_command.c

Re: [libvirt] [PATCH] qemu: avoid memory leaks

2011-08-02 Thread Laine Stump
On 08/02/2011 04:10 PM, Eric Blake wrote: Quite a few leaks detected by coverity. For chr, the leaks were close enough to the allocations to plug in place; for disk, the leaks were separated from the allocation by enough other lines with intermediate failure cases that I refactored the cleanup

Re: [libvirt] [PATCH] qemu: avoid memory leaks

2011-08-02 Thread Eric Blake
On 08/02/2011 02:48 PM, Laine Stump wrote: On 08/02/2011 04:10 PM, Eric Blake wrote: Quite a few leaks detected by coverity. For chr, the leaks were close enough to the allocations to plug in place; for disk, the leaks were separated from the allocation by enough other lines with intermediate

Re: [libvirt] [PATCH] qemu: avoid memory leaks

2011-08-02 Thread Laine Stump
On 08/02/2011 05:46 PM, Eric Blake wrote: On 08/02/2011 02:48 PM, Laine Stump wrote: On 08/02/2011 04:10 PM, Eric Blake wrote: Quite a few leaks detected by coverity. For chr, the leaks were close enough to the allocations to plug in place; for disk, the leaks were separated from the