1452735 - Unable to decode particular PNG file

Status: RESOLVED DUPLICATE of bug 1530222

(Core :: Graphics: ImageLib, defect, P3)

Description

[scroggo]

Attachment: PARTNER DELIVERED FAIL.png

User Agent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/65.0.3325.181 Safari/537.36

Steps to reproduce:

Try to open the attached file in FireFox


Actual results:

"The image "..." cannot be displayed because it contains errors."


Expected results:

Display the image. This seems to be a recent failure. FireFox 52.6.0 (64-bit) displayed the image, but when I tried on trunk (specifically 61.0a1) it failed.

I'm guessing this is due to a libpng update. It seems the image has a large iTXt chunk with XMP data - larger than the PNG_USER_CHUNK_MALLOC_MAX of 4000000L. This used to be okay, but in 1.6.32, libpng was updated [1] to check all chunks (except IDAT) against PNG_USER_CHUNK_MALLOC_MAX.

Maybe this is working as intended? FireFox set a maximum size for a chunk, and this chunk is bigger than that. It does seem like maybe it's too bad, since an image that used to work no longer does. Also, FireFox was going to ignore the chunk anyway, so maybe it was an unnecessary check? (Same bug occurs in Chromium: crbug.com/827754

[1] https://github.com/glennrp/libpng/commit/347538efbdc21b8df684ebd92d37400b3ce85d55

Comment 1

[Gingerbread Man]

Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:61.0) Gecko/20100101 Firefox/61.0
20180409220118

https://hg.mozilla.org/integration/mozilla-inbound/json-pushes?changeset=add5e417d0528189831b5d7431f6e264c3dc3ecf&full=1

Comment 2

[Glenn Randers-Pehrson]

Not a compete solution but try changing png_error() to png_chunk_benign_error() in pngrutil.c around line 3186

Comment 3

[Jonathan Watt [:jwatt]]

Mass bug change to replace various 'parity' whiteboard flags with the new canonical keywords. (See bug 1443764 comment 13.)

Comment 4

[Timothy Nikkel (:tnikkel)]

Fixed by bug 1530222.