[Pkg-fonts-devel] Bug#569548: Bug#569548: fontforge: fix for the segfault

Christian PERRIER bubulle at debian.org
Fri Feb 19 06:16:38 UTC 2010

Quoting Rogério Brito (rbrito at ime.usp.br):
> Package: fontforge
> Version: 0.0.20090923-1+b1
> Severity: normal
> At line scstyles.c:2676, we test if sc_sc == NULL and then jump to
> end_loop2.
> But at end_loop2 (sctyles.c:2725), we dereference sc and this gives us a
> nice segfault. Skipping the dereference at that point means that we
> don't update the UI for that point (which should not be updated anyway,
> since nothing was converted to small caps).
> This patch only allows the dereference of the pointer if it is non-NULL.
> This has been applied, compiled, tested and successfully used to
> generate a preliminary version of Utopia with small caps.
> Please, be warned that the result of running fontforge under valgrind is
> quite scary regarding the memory management.

Thanks, Rogério, this is much appreciated.

Kęstutis, who is the main maintainer for fontforge, did not follow up
on your patch proposal. He's maybe busy in some other tasks, so I take
the liberty to answer you so that you don't feel ignored.

I would tend to have some confidence in your patch but I'm lacking the
required skills for this. Kęstutis, would you mind confirming whether
you're OK to apply Rogério's patch (and probably forward it upstream)?

Rogério, regarding your mentions of probably bad memory management in
FF, it would probably be more efficient if you can get in touch with
George Williams or the BTS at http://fontforge.sourceforge.net/

Of course, if needed, we can act as proxies...

-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.alioth.debian.org/pipermail/pkg-fonts-devel/attachments/20100219/dd4fa309/attachment.pgp>

More information about the Pkg-fonts-devel mailing list