[Bug 536920] New: [abrt] crash detected in fontforge-20090923-1.fc12

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: [abrt] crash detected in fontforge-20090923-1.fc12

https://bugzilla.redhat.com/show_bug.cgi?id=536920

Summary: [abrt] crash detected in fontforge-20090923-1.fc12
           Product: Fedora
           Version: 12
          Platform: x86_64
        OS/Version: Linux
            Status: NEW
 Status Whiteboard: abrt_hash:b3e2ab72c0df6624350588b46b7b8a6bb6ff35f1
          Severity: medium
          Priority: low
         Component: fontforge
        AssignedTo: [e-mail]
        ReportedBy: [e-mail]
         QAContact: [e-mail]
                CC: [e-mail], [e-mail],
                    [e-mail]
    Classification: Fedora

abrt detected a crash.

Attached file: backtrace
cmdline: /usr/bin/fontforge -lang=ff /usr/bin/fontlint
usr/share/poker3d/data/neurpoli.ttf
component: fontforge
executable: /usr/bin/fontforge
kernel: 2.6.31.5-122.fc12.x86_64
package: fontforge-20090923-1.fc12
rating: 3
reason: Process was terminated by signal 11

[Bug 536920] [abrt] crash detected in fontforge-20090923-1.fc12

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

https://bugzilla.redhat.com/show_bug.cgi?id=536920

--- Comment #1 from Nicolas Mailhot   2009-11-11 14:28:03 EDT ---
Created an attachment (id=369093)
 --> (https://bugzilla.redhat.com/attachment.cgi?id=369093)
File: backtrace

[Bug 536920] [abrt] crash detected in fontforge-20090923-1.fc12

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

https://bugzilla.redhat.com/show_bug.cgi?id=536920

--- Comment #3 from Kevin Fenzi  2010-07-04 15:05:09 EDT ---
I'm not sure if this one was fixed or not.

Do you see it with current rawhide fontforge?

[Bug 536920] [abrt] crash detected in fontforge-20090923-1.fc12

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

https://bugzilla.redhat.com/show_bug.cgi?id=536920

Paul Flo Williams  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]

--- Comment #4 from Paul Flo Williams  2010-07-12 13:04:16 EDT ---
This still segfaults with the scratch build you made on 2010-07-04:

http://koji.fedoraproject.org/koji/taskinfo?taskID=2294313

This is the build with sources from 2010-05-01.

It also crashes with current CVS, sources from 2010-06-25.

[Bug 536920] [abrt] crash detected in fontforge-20090923-1.fc12

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

https://bugzilla.redhat.com/show_bug.cgi?id=536920

--- Comment #5 from Paul Flo Williams  2010-07-12 14:13:26 EDT ---
Created an attachment (id=431233)
 --> (https://bugzilla.redhat.com/attachment.cgi?id=431233)
Smallest subset of original font that produces crash

I have cut the original font down to the smallest subset that reproduces the
original crash, and attached it here.

Validation crashes on the ellipsis character, which is made by reference to two
other characters, full stop and space. The references to space are pointless,
as it has no outlines.

I am about to send a bug report to the upstream mailing list with this
information.

[Bug 536920] [abrt] crash detected in fontforge-20090923-1.fc12

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

https://bugzilla.redhat.com/show_bug.cgi?id=536920

--- Comment #6 from Kevin Fenzi  2010-07-13 00:27:19 EDT ---
Thanks Paul. :) Hopefully upstream will find a fix.

[Bug 536920] [abrt] crash detected in fontforge-20090923-1.fc12

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

https://bugzilla.redhat.com/show_bug.cgi?id=536920

--- Comment #7 from Paul Flo Williams  2010-07-14 07:47:05 EDT ---
Created an attachment (id=431739)
 --> (https://bugzilla.redhat.com/attachment.cgi?id=431739)
Correct LayerUnAllSplines() when refs have nil splinesets

When validation is performed, the set of splines for each character, including
those used by reference (diacritics, for example), are "flattened" to one set
by LayerAllSplines(). LayerUnAllSplines() is supposed to perform the reverse
transformation, but fails if any references have nil splinesets.

This patch applies to 20090923, for the benefit of F13 and rawhide. It will be
supplied upstream for CVS head with more information this evening.

Local testing:

[paul@palant x86_64]$ fontlint /usr/share/poker3d/data/neurpoli.ttf 
Copyright (c) 2000-2009 by George Williams.
 Executable based on sources from 23:48 GMT 23-Sep-2009.
 Library based on sources from 17:32 GMT 14-Sep-2009.
The following table(s) in the font have been ignored by FontForge
  Ignoring 'LTSH' linear threshold table
  Ignoring 'VDMX' vertical device metrics table
  Ignoring 'hdmx' horizontal device metrics table
The glyph named periodcentered is mapped to U+2219.
  But its name indicates it should be mapped to U+00B7.
The glyph named Delta is mapped to U+2206.
  But its name indicates it should be mapped to U+0394.
Validation Neuropolitical ...Failed
  Self Intersecting Glyph
  Missing Points at Extrema

[Bug 536920] [abrt] crash detected in fontforge-20090923-1.fc12

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

https://bugzilla.redhat.com/show_bug.cgi?id=536920

--- Comment #8 from Kevin Fenzi  2010-07-16 16:34:23 EDT ---
I can go ahead and add this patch in rawhide... or did we need to backport it
to the f12/f13 versions?

Typically we don't upgrade fontforge in a stable release to avoid any
regressions with fonts already built with it.

Thoughts?

[Bug 536920] [abrt] crash detected in fontforge-20090923-1.fc12

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

https://bugzilla.redhat.com/show_bug.cgi?id=536920

--- Comment #9 from Paul Flo Williams  2010-07-16 17:43:25 EDT ---
I'd patch it in rawhide. The chances of this occurring with any other font are
extremely slim (and I ran fontlint on 30000 of them yesterday).

If bug 615347 is accepted, this font won't be in Fedora for long anyway ;-)

[Bug 536920] [abrt] crash detected in fontforge-20090923-1.fc12

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

https://bugzilla.redhat.com/show_bug.cgi?id=536920

Kevin Fenzi  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]
          Component|fontforge                   |0xFFFF

--- Comment #10 from Kevin Fenzi  2010-07-16 18:56:37 EDT ---
ok. Built for rawhide: 
http://koji.fedoraproject.org/koji/taskinfo?taskID=2324995

Shall we close this bug now?

[Bug 536920] [abrt] crash detected in fontforge-20090923-1.fc12

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

https://bugzilla.redhat.com/show_bug.cgi?id=536920

--- Comment #11 from Paul Flo Williams  2010-07-18 07:31:39 EDT ---
(In reply to comment #10)
> Shall we close this bug now?

I vote yes.

[Bug 536920] [abrt] crash detected in fontforge-20090923-1.fc12

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

https://bugzilla.redhat.com/show_bug.cgi?id=536920

Kevin Fenzi  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |CLOSED
         Resolution|                            |RAWHIDE