modules/libimg/png contains outdated pngconf.h

VERIFIED FIXED

Status

()

Core
ImageLib
P3
normal
VERIFIED FIXED
19 years ago
19 years ago

People

(Reporter: Greg Roelofs, Assigned: pnunn)

Tracking

Trunk
x86
Linux
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

19 years ago
Mozilla's libpng is mostly 1.0.2, but pngconf.h is 0.95.  Because Mozilla never
includes system PNG headers even if it links against the system libpng, this
results in inconsistent data structures when attempting to use native (system)
PNG libraries for the build.

Fix:  replace modules/libimg/png/pngconf.h with the stock version from 1.0.2.

Note that this also eliminates the requirement to include any sort of libpng.so
or libpng.so.2 symlink in dist/bin, which I thought was an issue for bug 5459.
There is also no apparent problem with libc5 vs. glibc2 (or ld.so.1 vs ld.so.2).
(Assignee)

Updated

19 years ago
Status: NEW → ASSIGNED
(Assignee)

Comment 1

19 years ago
on it. Thanks for catching this one, G.
-pn
(Assignee)

Comment 2

19 years ago
I've got to sort out if any of the diffs are still needed.
-pn
(Reporter)

Comment 3

19 years ago
> I've got to sort out if any of the diffs are still needed.

If they are, then MOZ_NATIVE_PNG compiles are also broken.  I'd rather go with a
completely stock libpng and fix any new breakage (preferably in Mozilla itself)
than continue carrying along old fixes of questionable validity.

The only problems I expect to see have to do with setjmp/longjmp and
inconsistent notions of the size of jmp_buf between library and application;
those should go away as soon as I install a custom libpng error handler (which
will be required in some future version of libpng anyway).
(Assignee)

Comment 4

19 years ago
I agree that a stock png implementation makes sense.
but I'm not going to check in the 'new' pngconf.h without
having an idea what it will do on several platforms.
So far, its ok on my NT and Linux. Hopefully will be able to
check Mac and solaris soon.
-pn
(Assignee)

Updated

19 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 19 years ago
Resolution: --- → FIXED
(Assignee)

Comment 5

19 years ago
Builds ok on Mac, linux, NT.
Checked in 1.02 pngconf.h file.
This check in will result in bad displays of colormapped png images
with alpha or transparency. This problem will be addressed in bug#917.
There is no need to file another bug.
-pnunn
(Reporter)

Comment 6

19 years ago
Thanks, Pamela!  I'll contact you separately about 917 and a PNG-flavored
gif_struct with line buffers.  (I guess 917 is M6 now. :-) )

Updated

19 years ago
QA Contact: 1698 → 2910

Comment 7

19 years ago
[Per e-mail, QA assigning to Greg, since build stuff is out of my domain of clue.
Thanks, Greg!]
(Reporter)

Updated

19 years ago
Status: RESOLVED → VERIFIED
(Reporter)

Comment 8

19 years ago
Marking as verified...
You need to log in before you can comment on or make changes to this bug.