Bug 788246

Summary: GraphicsMagick 1.3.13 update breaks some PNGs
Product: [Fedora] Fedora Reporter: Ville Skyttä <ville.skytta>
Component: GraphicsMagickAssignee: Andreas Thienemann <andreas>
Status: CLOSED ERRATA QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: high Docs Contact:
Priority: unspecified    
Version: 16CC: andreas, bfriesen, cn6uw7d02, m.amazirh, om, rdieter
Target Milestone: ---Keywords: Regression
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-12-20 16:20:38 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
Reproducer PNG broken by 1.3.13 update
none
Reproducer PNG as shown by "gm display" from 1.3.13 none

Description Ville Skyttä 2012-02-07 20:49:53 UTC
The GraphicsMagick 1.3.13-3 F-16 update breaks some, but not all PNGs.  The broken ones appear as if some vertical empty space/interlacing was added to them.  No such problems with earlier GraphicsMagick versions.

Some reproducer files attached.

Comment 1 Ville Skyttä 2012-02-07 20:50:36 UTC
Created attachment 560057 [details]
Reproducer PNG broken by 1.3.13 update

Comment 2 Ville Skyttä 2012-02-07 20:52:10 UTC
Created attachment 560058 [details]
Reproducer PNG as shown by "gm display" from 1.3.13

Comment 3 Rex Dieter 2012-02-07 20:54:40 UTC
So, the png in comment #1 was created by gm or is the source used in comment #2?

Comment 4 Ville Skyttä 2012-02-07 21:01:29 UTC
The PNG in comment 1 is just a random PNG that GM 1.3.13 has problems with, no idea what it was created with.

The PNG in comment 2 is a screenshot of how the PNG from comment 1 looks with GM 1.3.13's "gm display" (I should probably have attached this one as a BMP or something else than PNG to reduce confusion).

Comment 5 Rex Dieter 2012-02-07 21:24:25 UTC
ok, thanks.

I'll see if the impact is limited to or goes beyond the use of 'gm display'

Comment 6 Rex Dieter 2012-02-07 21:53:58 UTC
OK, GM-1.1.13 does not like this image alright, pretty much any operation botches it.

Bug opened upstream,
https://sourceforge.net/tracker/?func=detail&aid=3485509&group_id=73485&atid=537937

Comment 7 Rex Dieter 2012-02-07 21:55:54 UTC
I'll see about diff'ing gm-1.1.12 -> 1.1.13 coders/png.c for anything obviously wrong.  it was updated to support libpng-1.5 (as in rawhide, but apparently there's some regression going on here with f16/libpng-1.2.x)

Comment 8 Bob Friesenhahn 2012-02-08 03:18:47 UTC
I am able to reproduce the problem here as well.  I see the same problem with libpng 1.5.6 and libpng 1.2.46.  While there were a number of known problems with the PNG writer, this is the first issue I am aware of with the reader.

Comment 9 Rex Dieter 2012-03-01 19:59:48 UTC
confirmed still an issue with 1.3.14

Comment 10 Ville Skyttä 2012-03-17 19:57:55 UTC
*** Bug 803578 has been marked as a duplicate of this bug. ***

Comment 11 Ville Skyttä 2012-03-17 19:58:52 UTC
Any progress on this or thoughts when it might be fixed?

Comment 12 Rex Dieter 2012-03-19 01:51:59 UTC
Ville, 
this has been reported upstream,
https://sourceforge.net/tracker/?func=detail&aid=3485509&group_id=73485&atid=537937

and it's been acknowledged/confirmed (see Bob's comment #8 as well as that referenced in the upstream report).

Comment 13 Bob Friesenhahn 2012-03-19 14:07:37 UTC
I am hoping that Glenn Randers-Pehrson will be able to complete the fixes to (or rewrite of) GM's png.c.  Progress has been slow since he would like to do the job correctly and has other responsibilities.  It seems likely that the issues were introduced due to the port to libpng 1.4.X and then libpng 1.5.X.

Comment 14 Ville Skyttä 2012-04-15 18:20:38 UTC
*** Bug 812645 has been marked as a duplicate of this bug. ***

Comment 15 Ville Skyttä 2012-07-08 14:24:50 UTC
Confirmed still an issue with 1.3.16.

Comment 16 Fedora Update System 2012-10-16 20:00:50 UTC
GraphicsMagick-1.3.17-1.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/GraphicsMagick-1.3.17-1.fc17

Comment 17 Fedora Update System 2012-10-16 20:02:01 UTC
GraphicsMagick-1.3.17-1.fc16 has been submitted as an update for Fedora 16.
https://admin.fedoraproject.org/updates/GraphicsMagick-1.3.17-1.fc16

Comment 18 Fedora Update System 2012-10-16 20:02:50 UTC
GraphicsMagick-1.3.17-1.el6 has been submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/GraphicsMagick-1.3.17-1.el6

Comment 19 Fedora Update System 2012-10-17 13:05:58 UTC
GraphicsMagick-1.3.17-1.el5 has been submitted as an update for Fedora EPEL 5.
https://admin.fedoraproject.org/updates/GraphicsMagick-1.3.17-1.el5

Comment 20 Fedora Update System 2012-10-18 00:21:04 UTC
Package GraphicsMagick-1.3.17-1.fc16:
* should fix your issue,
* was pushed to the Fedora 16 testing repository,
* should be available at your local mirror within two days.
Update it with:
# su -c 'yum update --enablerepo=updates-testing GraphicsMagick-1.3.17-1.fc16'
as soon as you are able to.
Please go to the following url:
https://admin.fedoraproject.org/updates/FEDORA-2012-16336/GraphicsMagick-1.3.17-1.fc16
then log in and leave karma (feedback).

Comment 21 Otto J. Makela 2012-10-20 16:16:28 UTC
Quick testing seems to indicate this new testing version 1.3.17-1 does not cause the problems that were associated with 1.3.13-3, will leave feedback.

Comment 22 Fedora Update System 2012-12-20 16:20:41 UTC
GraphicsMagick-1.3.17-1.fc17 has been pushed to the Fedora 17 stable repository.  If problems still persist, please make note of it in this bug report.