[prev in list] [next in list] [prev in thread] [next in thread] 

List:       freedesktop-xorg-devel
Subject:    Re: [PATCH libx11] Plug a memory leak
From:       Peter Hutterer <peter.hutterer () who-t ! net>
Date:       2016-10-26 5:46:51
Message-ID: 20161026054651.GA20838 () jelly ! local
[Download RAW message or body]

On Tue, Oct 25, 2016 at 09:30:15PM +0200, Emilio Pozuelo Monfort wrote:
> This was introduced in 8ea762f.
> 
> Reported-by: Julien Cristau <jcristau@debian.org>
> Signed-off-by: Emilio Pozuelo Monfort <pochu@debian.org>

pushed, thanks.

   8f349fe..20a3f99  master -> master

Cheers,
   Peter

> ---
>  src/FontNames.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/src/FontNames.c b/src/FontNames.c
> index e55f338..3e23b5f 100644
> --- a/src/FontNames.c
> +++ b/src/FontNames.c
> @@ -98,12 +98,14 @@ int *actualCount)	/* RETURN */
>  		    *ch = '\0';  /* and replace with null-termination */
>  		    count++;
>  		} else {
> +                    Xfree(ch);
>                      Xfree(flist);
>                      flist = NULL;
>                      count = 0;
>                      break;
>  		}
>  	    } else {
> +                Xfree(ch);
>                  Xfree(flist);
>                  flist = NULL;
>                  count = 0;
> -- 
> 2.9.3
> 
> _______________________________________________
> xorg-devel@lists.x.org: X.Org development
> Archives: http://lists.x.org/archives/xorg-devel
> Info: https://lists.x.org/mailman/listinfo/xorg-devel
> 
_______________________________________________
xorg-devel@lists.x.org: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: https://lists.x.org/mailman/listinfo/xorg-devel
[prev in list] [next in list] [prev in thread] [next in thread]