On 10/08/2014 10:29 AM, Eric Blake wrote:
> The function hypervEnumAndPull consumes query on success, but leaked
> it on failure.  Rather than having to change all callers (many of
> them indirect callers through the generated
> hypervGetMsvmComputerSystemList), it was easier to just guarantee
> that the buffer is cleaned on return from the function.
> 
> Reported by Yves Vinter.
> 
> * src/hyperv/hyperv_wmi.c (hypervEnumAndPull): Don't leak query on
> failure.
> 
> Signed-off-by: Eric Blake <[email protected]>
> ---
>  src/hyperv/hyperv_wmi.c | 37 +++++++++++++++++++++----------------
>  1 file changed, 21 insertions(+), 16 deletions(-)

Yves gave me an off-list ACK for this patch, so I've pushed it.  With
this in place, it would be easier if you rebase your remaining patches
on top of this and repost them for review.

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature

--
libvir-list mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/libvir-list

Reply via email to