Re: [PATCHv5 1/9] Fix a bug related to VIR_TYPED_PARAM_STRING

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 12/20/2011 01:34 AM, Hu Tao wrote:
> From: Eric Blake <eblake@xxxxxxxxxx>
> 
> https://www.redhat.com/archives/libvir-list/2011-December/msg00854.html
> ---
>  daemon/remote.c |   22 ++++++++++------------
>  1 files changed, 10 insertions(+), 12 deletions(-)

I'll take your re-posting of my patch as an ACK, and pushing this along
with my genprotocol.pl fix squashed in.  The commit message is now:

commit 4e394dea1f3a1cd4f4f6637ed91c9f7b367800c0
Author: Eric Blake <eblake@xxxxxxxxxx>
Date:   Tue Dec 20 08:22:25 2011 -0700

    rpc: handle param_int, plug memory leaks

    The RPC code had several latent memory leaks and an attempt to
    free the wrong string, but thankfully nothing triggered them
    (blkiotune was the only one returning a string, and always as
    the last parameter).  Also, our cleanups for rpcgen ended up
    nuking a line of code that renders VIR_TYPED_PARAM_INT broken,
    because it was the only use of 'i' in a function, even though
    it was a member usage rather than a standalone declaration.

    * daemon/remote.c (remoteSerializeTypedParameters): Free the
    correct array element.
    (remoteDispatchDomainGetSchedulerParameters)
    (remoteDispatchDomainGetSchedulerParametersFlags)
    (remoteDispatchDomainBlockStatsFlags)
    (remoteDispatchDomainGetMemoryParameters): Don't leak strings.
    * src/rpc/genprotocol.pl: Don't nuke member-usage of 'buf' or 'i'.

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

Attachment: signature.asc
Description: OpenPGP digital signature

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

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]