[libpng16] Fixed an intermittent SEGV in pngstest.

This commit is contained in:
John Bowler 2012-11-02 14:08:29 -05:00 committed by Glenn Randers-Pehrson
parent 7b3eefbcfd
commit 4314d63792
3 changed files with 14 additions and 30 deletions

View File

@ -1,5 +1,5 @@
Libpng 1.6.0beta32 - November 1, 2012 Libpng 1.6.0beta32 - November 2, 2012
This is not intended to be a public release. It will be replaced This is not intended to be a public release. It will be replaced
within a few weeks by a public version or by another test version. within a few weeks by a public version or by another test version.
@ -528,16 +528,6 @@ Version 1.6.0beta30 [October 24, 2012]
Version 1.6.0beta31 [November 1, 2012] Version 1.6.0beta31 [November 1, 2012]
Undid the erroneous change to vstudio/pngvalid build in libpng-1.6.0beta30. Undid the erroneous change to vstudio/pngvalid build in libpng-1.6.0beta30.
Made pngvalid so that it will build outside the libpng source tree. Made pngvalid so that it will build outside the libpng source tree.
Changed ICC profile support to allow use of an external color management
system (CMS). In practice it is not possible to obtain cHRM information
reliably using just the end-point tags because they do not exist in the
vast majority of profiles. Instead it is necessary to run the endpoints
through the ICC colorimetric intent transform (as described in the v4
spec). Since this is likely to be too much code inside libpng for too
little gain (it implies a fairly complete CMS implementation) the code
has been changed to allow an external CMS to be used. This code is
temporarily disabled until a suitable set of test cases using one or more
external CMS implementations have been implemented.
Made builds -DPNG_NO_READ_GAMMA compile (the unit tests still fail). Made builds -DPNG_NO_READ_GAMMA compile (the unit tests still fail).
Made PNG_NO_READ_GAMMA switch off interfaces that depend on READ_GAMMA. Made PNG_NO_READ_GAMMA switch off interfaces that depend on READ_GAMMA.
Prior to 1.6.0 switching off READ_GAMMA did unpredictable things to the Prior to 1.6.0 switching off READ_GAMMA did unpredictable things to the
@ -550,11 +540,13 @@ Version 1.6.0beta31 [November 1, 2012]
question (typically most users who disable it won't). question (typically most users who disable it won't).
Fixed GUIDs in projects/vstudio. Some were duplicated or missing, Fixed GUIDs in projects/vstudio. Some were duplicated or missing,
resulting in VS2010 having to update the files. resulting in VS2010 having to update the files.
Removed non-working ICC profile support code. There was too much code Removed non-working ICC profile support code that was mostly added to
for too little gain; implementing full ICC color correction is maybe libpng-1.6.0beta29 and beta30. There was too much code for too little
desireable but is left up to applications. gain; implementing full ICC color correction may be desireable but is left
up to applications.
Version 1.6.0beta32 [November 1, 2012] Version 1.6.0beta32 [November 2, 2012]
Fixed an intermittent SEGV in pngstest.
Send comments/corrections/commendations to png-mng-implement at lists.sf.net Send comments/corrections/commendations to png-mng-implement at lists.sf.net
(subscription required; visit (subscription required; visit

20
CHANGES
View File

@ -4280,16 +4280,6 @@ Version 1.6.0beta30 [October 24, 2012]
Version 1.6.0beta31 [November 1, 2012] Version 1.6.0beta31 [November 1, 2012]
Undid the erroneous change to vstudio/pngvalid build in libpng-1.6.0beta30. Undid the erroneous change to vstudio/pngvalid build in libpng-1.6.0beta30.
Made pngvalid so that it will build outside the libpng source tree. Made pngvalid so that it will build outside the libpng source tree.
Changed ICC profile support to allow use of an external color management
system (CMS). In practice it is not possible to obtain cHRM information
reliably using just the end-point tags because they do not exist in the
vast majority of profiles. Instead it is necessary to run the endpoints
through the ICC colorimetric intent transform (as described in the v4
spec). Since this is likely to be too much code inside libpng for too
little gain (it implies a fairly complete CMS implementation) the code
has been changed to allow an external CMS to be used. This code is
temporarily disabled until a suitable set of test cases using one or more
external CMS implementations have been implemented.
Made builds -DPNG_NO_READ_GAMMA compile (the unit tests still fail). Made builds -DPNG_NO_READ_GAMMA compile (the unit tests still fail).
Made PNG_NO_READ_GAMMA switch off interfaces that depend on READ_GAMMA. Made PNG_NO_READ_GAMMA switch off interfaces that depend on READ_GAMMA.
Prior to 1.6.0 switching off READ_GAMMA did unpredictable things to the Prior to 1.6.0 switching off READ_GAMMA did unpredictable things to the
@ -4302,11 +4292,13 @@ Version 1.6.0beta31 [November 1, 2012]
question (typically most users who disable it won't). question (typically most users who disable it won't).
Fixed GUIDs in projects/vstudio. Some were duplicated or missing, Fixed GUIDs in projects/vstudio. Some were duplicated or missing,
resulting in VS2010 having to update the files. resulting in VS2010 having to update the files.
Removed non-working ICC profile support code. There was too much code Removed non-working ICC profile support code that was mostly added to
for too little gain; implementing full ICC color correction is maybe libpng-1.6.0beta29 and beta30. There was too much code for too little
desireable but is left up to applications. gain; implementing full ICC color correction may be desireable but is left
up to applications.
Version 1.6.0beta32 [November 1, 2012] Version 1.6.0beta32 [November 2, 2012]
Fixed an intermittent SEGV in pngstest.
Send comments/corrections/commendations to png-mng-implement at lists.sf.net Send comments/corrections/commendations to png-mng-implement at lists.sf.net
(subscription required; visit (subscription required; visit

View File

@ -2625,7 +2625,7 @@ component_loc(png_byte loc[4], png_uint_32 format)
else else
{ {
channels = 1; channels = 1;
loc[0] = loc[1] = loc[2] = 0; loc[1] = loc[2] = loc[3] = 0;
} }
if (format & PNG_FORMAT_FLAG_ALPHA) if (format & PNG_FORMAT_FLAG_ALPHA)