[PATCH libx11] Plug a memory leak

Peter Hutterer peter.hutterer at who-t.net
Wed Oct 26 05:46:51 UTC 2016


On Tue, Oct 25, 2016 at 09:30:15PM +0200, Emilio Pozuelo Monfort wrote:
> This was introduced in 8ea762f.
> 
> Reported-by: Julien Cristau <jcristau at debian.org>
> Signed-off-by: Emilio Pozuelo Monfort <pochu at 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 at lists.x.org: X.Org development
> Archives: http://lists.x.org/archives/xorg-devel
> Info: https://lists.x.org/mailman/listinfo/xorg-devel
> 


More information about the xorg-devel mailing list