cvs commit: ports/x11-fonts/libXft Makefile distinfo
chinsan
chinsan.tw at gmail.com
Fri Jul 15 18:56:42 GMT 2005
2005/7/16, MANTANI Nobutaka <nobutaka at freebsd.org>:
> At Fri, 15 Jul 2005 23:42:53 +0800,
> Clive Lin wrote:
> >
> > Uhm, stay tuned. It's not yet proven right.
Ya ... :(
(sorry,cjk-patch-libXft-20050715.diff.gz was made by me)
>
> I have found a null pointer dereference bug in the CJK patch.
> It causes segmentation fault in the startup of firefox (aagh!).
> The following patch fixes it.
Yes! It works well for me. Thank to nobutaka for your new patch :)
Could this patch be included in such as
cjk-patch-libXft-20050716.diff.gz(changed filename)?
>
>
> Index: Makefile
> ===================================================================
> RCS file: /home/pcvs/ports/x11-fonts/libXft/Makefile,v
> retrieving revision 1.32
> diff -u -r1.32 Makefile
> --- Makefile 15 Jul 2005 15:40:37 -0000 1.32
> +++ Makefile 15 Jul 2005 18:25:27 -0000
> @@ -30,6 +30,7 @@
> PATCH_SITES= ftp://local-distfiles.freebsd.org.cn/pub/china-ports/hamigua/ \
> http://bsdchat.com/dist/firefly-cjk-patchset/
> PATCHFILES+= cjk-patch-libXft-20050715.diff.gz
> +EXTRA_PATCHES= ${PATCHDIR}/extra-patch-xftglyphs.c
> .endif
>
> .include <bsd.port.pre.mk>
> Index: files/extra-patch-xftglyphs.c
> ===================================================================
> RCS file: files/extra-patch-xftglyphs.c
> diff -N files/extra-patch-xftglyphs.c
> --- /dev/null 1 Jan 1970 00:00:00 -0000
> +++ files/extra-patch-xftglyphs.c 15 Jul 2005 18:25:27 -0000
> @@ -0,0 +1,26 @@
> +--- xftglyphs.c.orig Sat Jul 16 02:57:30 2005
> ++++ xftglyphs.c Sat Jul 16 02:57:44 2005
> +@@ -127,6 +127,11 @@
> + if (!info)
> + return;
> +
> ++ face = XftLockFace (&font->public);
> ++
> ++ if (!face)
> ++ return;
> ++
> + /*-------------------------------------------------------
> + * Add by Firefly (firefly at firefly.idv.tw)
> + * Initialize gamma table
> +@@ -141,11 +146,6 @@
> + if (font->info.weight >= FC_WEIGHT_BOLD &&
> + !(face->style_flags & FT_STYLE_FLAG_BOLD))
> + bold_advance_width = 64;
> +-
> +- face = XftLockFace (&font->public);
> +-
> +- if (!face)
> +- return;
> +
> + matrix.xx = matrix.yy = 0x10000L;
> + matrix.xy = matrix.yx = 0;
>
>
>
> --
> MANTANI Nobutaka
> nobutaka at nobutaka.org, nobutaka at FreeBSD.org
>
More information about the cvs-all
mailing list