Re: [PATCH] Fix a memory leak in cmdSchedInfoUpdateOne

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

 



On 08/22/2013 09:40 PM, hwbi2008@xxxxxxxxx wrote:
> From: hwbi <hwbi2008@xxxxxxxxx>
> 
> The param needs to be virTypedParamsFree()'d in cmdSchedInfoUpdateOne().
> ---
>  tools/virsh-domain.c |    3 ++-
>  1 files changed, 2 insertions(+), 1 deletions(-)

[You used git send-email incorrectly - when it asks what to use as
in-reply-to, you should hit 'enter' with blank text to create a new
thread, rather than attempting to answer 'y' which threads it to any
other thread that also used the incorrect 'y' as a message id.  Newer
git prevents the use of invalid message ids - so another solution is to
upgrade your git.]

> 
> diff --git a/tools/virsh-domain.c b/tools/virsh-domain.c
> index b29f934..d704053 100644
> --- a/tools/virsh-domain.c
> +++ b/tools/virsh-domain.c
> @@ -4085,7 +4085,7 @@ cmdSchedInfoUpdateOne(vshControl *ctl,
>                        int *nparams, int *maxparams,
>                        const char *field, const char *value)
>  {
> -    virTypedParameterPtr param;
> +    virTypedParameterPtr param = NULL;
>      int ret = -1;
>      size_t i;
>  
> @@ -4109,6 +4109,7 @@ cmdSchedInfoUpdateOne(vshControl *ctl,
>          vshError(ctl, _("invalid scheduler option: %s"), field);
>  
>   cleanup:
> +    virTypedParamsFree(param, *nparams);
>      return ret;
>  }

Unfortunately, your patch is incorrect.  When applying it, and
re-running 'make check', I get a failure:

FAIL: virsh-schedinfo
=====================

./virsh-schedinfo: line 46: 29400 Aborted                 (core dumped)
virsh -c $test_url schedinfo 1 --set j=k > out 2> err
--- exp-out     2013-08-26 09:52:15.318517471 -0600
+++ out 2013-08-26 09:52:15.341517394 -0600
@@ -1,2 +1 @@
 Scheduler      : fair
-

Please fix and resubmit.

-- 
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
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]