fontforge crashed with SIGSEGV in GImageRead_Png() due to optipng

StacktraceTop:
 GImageRead_Png (fp=0x18d3620) at gimagereadpng.c:287
 GImageReadPng (filename=<value optimized out>) at gimagereadpng.c:341
 _GGadgetImageCache (filename=0x57d0d3 "fileclose.png", foundname=0x0) at gtextinfo.c:468
 GTextInfoImageLookup (ti=<value optimized out>) at gtextinfo.c:560
 GTextInfoImageLookup (ti=<value optimized out>) at gtextinfo.c:539

Thanks for your report. This is reproducible in Oneiric by opening any ttf file.

Rebuilding the package fixes the issue.

the bug still persists on oneiric

I tried to open an OTF font and it crashed the same way it does with TTF

Freezing then crashing when creating a new font under oneiric.

I have tested with no-change rebuild (apt-get build-dep; apt-get source fontforge; cd fontforge*; debuild) at latest oneiric, the bug goes away.

So, we need no-change rebuild at this time.

It appears to be either buggy PNG images, or a buggy PNG reader. I worked around it a few days ago by deleting the problematic files:

  $ dpkg -L fontforge | grep png | xargs ls -1 > /dev/null
  ls: cannot access /usr/share/fontforge/pixmaps/viewfit.png: No such file or directory
  ls: cannot access /usr/share/fontforge/pixmaps/viewzoomin.png: No such file or directory
  ls: cannot access /usr/share/fontforge/pixmaps/viewzoomout.png: No such file or directory

Confirms; rebuild fixes it.

This bug was fixed in the package fontforge - 0.0.20110222-1build1

---------------
fontforge (0.0.20110222-1build1) oneiric; urgency=low

  * No change rebuild to prevent crash-on-startup (LP: #805752)
 -- Paul Sladen <email address hidden> Fri, 30 Sep 2011 15:42:14 +0100

0.0.20110222-1build1 and related(libfontforge,libgdraw) are distributed, but, the bugs still exists in my test environments (...why?).

Anyone can repro this behavior?

I can confirm this. Upgraded Maverick to latest Oneiric Daily build today.

My Fontforge version fontforge is 0.0.20110222-1build1

I open an SFD file and it segfaults.

[sorry, upgraded Natty to Oneiric, pardon me!]

Okay, I tested with clean installed oneiric Beta2, problem still exists. The suspect is pkgbinarymangler::pkgstripfiles(::optipng). pkgbinarymangler kicked at build-process in only archive-build[1]. We built at PPA/private pbuilder, pkgstripfiles does not kicked[2] and that works fine.

Collateral evidence:
When you install good fontforge packegs from my PPA[3], exec below command...problem reproduced! Breaker is optipng (of cource, fontforges PNG reader is still suspicous, and we need workaround).
  <<<WARNING: Before exec, please backup *png files>>>
 $ cd /usr/share/fontforge/pixmaps/
 $ sudo optipng -o4 -preserve *png

So, we have to one-more-rebuild with NO_PNG_PKG_MANGLE for good fontforge packages.

# But, we are in FinalFreeze :(

[1] https://launchpadlibrarian.net/81558874/buildlog_ubuntu-oneiric-i386.fontforge_0.0.20110222-1build1_BUILDING.txt.gz
[2] https://launchpadlibrarian.net/81653514/buildlog_ubuntu-oneiric-i386.fontforge_0.0.20110222-1build2%7Eppa1_BUILDING.txt.gz
[3] https://launchpad.net/~hito/+archive/ppa/+packages

hramrach or sladen, can you please post the command-line sequence for rebuilding the package?
As a non-tech user, I fear it could take me hours to figure out how to get ForntForge to work.
This page comes up right near the top with the Google string "Ubuntu 11.10 FontForge," so this is where us noobs will find the fix.
Thanks.

Dear bug triagers,

This bug still exist in Oneiric. Could you please revert "Fix Released" to "Confirmed"?
(Or, we have to re-file as new bug?)

I confirm it is still an issue and it crashes in Precise as well.

On Oneiric it still crashes. I also tried uninstalling and reinstalling Fontforge. Not sure if that is the same a rebuilding. It never gets beyond asking what file to open. So I guess I am on vacation till that is fixed.

Pitti: looks like Optipng is producing duff PNGs that are causing the crashes.

I can confirm that using the NO_PNG_PKG_MANGLE will workaround this bug. But I suspect that the bug is actually in fontforge's built-in image reader code.

This bug was fixed in the package fontforge - 0.0.20110222-3ubuntu1

---------------
fontforge (0.0.20110222-3ubuntu1) precise; urgency=low

  * debian/patches/902_fix_optipng_reads.diff:
    - Don't crash when reading optimized png files (LP: #805752)
 -- Michael Terry <email address hidden> Tue, 25 Oct 2011 14:26:03 -0400

I've uploaded a fixed version in oneiric-proposed. This is a good SRU candidate because the bug causes a crash regression that makes fontforge unusable.

To reproduce, run "fontforge /usr/share/fonts/truetype/ubuntu-font-family/Ubuntu-R.ttf"

Subscribing ubuntu-sru.

You've also merged a couple of Debian revisions into the package you've uploaded to oneiric-proposed. While they look like harmless changes, you haven't mentioned this in the changelog, nor are there any LP bug references.

Those other changes look reasonable - assuming that it's possible to hit those bugs in Oneiric - but I'd just like to check they're intentional before accepting into -proposed.

Chris, good catch. Totally my fault, in using the precise package as a base for the oneiric. I had forgotten that they diverged.

If you'd like to reject, I can upload one based just on oneiric's packaging.

@Michael: rejected, please reupload. Thanks!

Re-uploaded.

Hello Jorge, or anyone else affected,

Accepted fontforge into oneiric-proposed, the package will build now and be available in a few hours. Please test and give feedback here. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you in advance!

The proposed package no longer segfaults.

I can confirm it now works with the new proposed update.

The proposed fontforge=0.0.20110222-1ubuntu1 works for me, using the testcase:

  fontforge /usr/share/fonts/truetype/ubuntu-font-family/Ubuntu-R.ttf

Confirming that the proposed update is working for me - Ubuntu 11.04 / Unity Desktop

Hi, now working for me also (I was one of those affected by this bug) for creating a new font project and opening an existing .TTF file.
Xubuntu 11.10 64-bit

I might as well add to my comment on #34 that the distro I tested on is 32 bit the Fontforge is i386 archi.

This bug was fixed in the package fontforge - 0.0.20110222-1ubuntu1

---------------
fontforge (0.0.20110222-1ubuntu1) oneiric-proposed; urgency=low

  * debian/patches/902_fix_optipng_reads.diff:
    - Don't crash when reading optimized png files (LP: #805752)
 -- Michael Terry <email address hidden> Thu, 27 Oct 2011 09:05:00 -0400

I am still suffering this bug on Ubuntu 11.10 amd64. Fontforge gives segmentation fault whenever I try to edit (e.g. transform) a glyph.

Blogcrawler: is your font transformation bug definitely showing GImageRead_Png() as the course?

  (a) If it is, can you attach a stacktrace/screenshot of the error message

  (b) If it's something else causing the crash; can you open a new bug against Fontforge and we'll try to debug it there so that we don't confuse two bugs (one of which is hopefully fixed).

Paul: I noticed that the bug has already been reported in #817290