Re: fontconfig: Branch 'master'

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

 



Funny thing.  For the first time I compiled fontconfig on my Macbook Air last
night and saw it crashing.  This morning on the metro ride to the office I
tracked it down and committed a fixed.  Just got to the office and tried to
git push, which failed.  So I git pulled and got your commit in, and my commit
become a null, not a conflict.  It was byte by byte identical to yours! :))

On 14-07-04 05:05 AM, Akira TAGOH wrote:
>  src/fccfg.c |   11 +++++++----
>  1 file changed, 7 insertions(+), 4 deletions(-)
> 
> New commits:
> commit 5b22776999b6052afe0e829b1a0c0935bbe1e9f7
> Author: Akira TAGOH <akira@xxxxxxxxx>
> Date:   Fri Jul 4 18:04:52 2014 +0900
> 
>     Fix a crash when no sysroot is given and failed to load the default fonts.conf
> 
> diff --git a/src/fccfg.c b/src/fccfg.c
> index e2555f0..6d32595 100644
> --- a/src/fccfg.c
> +++ b/src/fccfg.c
> @@ -2342,7 +2342,7 @@ void
>  FcConfigSetSysRoot (FcConfig      *config,
>  		    const FcChar8 *sysroot)
>  {
> -    FcChar8 *s;
> +    FcChar8 *s = NULL;
>      FcBool init = FcFalse;
>  
>      if (!config)
> @@ -2362,9 +2362,12 @@ FcConfigSetSysRoot (FcConfig      *config,
>  	}
>      }
>  
> -    s = FcStrCopyFilename (sysroot);
> -    if (!s)
> -	return;
> +    if (sysroot)
> +    {
> +	s = FcStrCopyFilename (sysroot);
> +	if (!s)
> +	    return;
> +    }
>  
>      if (config->sysRoot)
>  	FcStrFree (config->sysRoot);
> _______________________________________________
> Fontconfig mailing list
> Fontconfig@xxxxxxxxxxxxxxxxxxxxx
> http://lists.freedesktop.org/mailman/listinfo/fontconfig
> 

-- 
behdad
http://behdad.org/
_______________________________________________
Fontconfig mailing list
Fontconfig@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/fontconfig




[Index of Archives]     [Fedora Fonts]     [Fedora Users]     [Fedora Cloud]     [Kernel]     [Fedora Packaging]     [Fedora Desktop]     [PAM]     [Gimp Graphics Editor]     [Yosemite News]

  Powered by Linux