summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorGlenn Randers-Pehrson <glennrp at users.sourceforge.net>2016-01-13 09:55:15 -0600
committerGlenn Randers-Pehrson <glennrp at users.sourceforge.net>2016-01-13 09:55:15 -0600
commit7190c4f5af18bdc470a04353d11eff74aea630b1 (patch)
tree9f083bdf930564107f34037b475865eb78e03eb3
parentd189b54e30824ac9f9b4159475b6d0b3d0985de1 (diff)
downloadlibpng-7190c4f5af18bdc470a04353d11eff74aea630b1.tar.gz
[libpng15] Worked around a false-positive Coverity issue in pngvalid.c.
-rw-r--r--ANNOUNCE5
-rw-r--r--CHANGES3
-rw-r--r--contrib/libtests/pngvalid.c21
3 files changed, 22 insertions, 7 deletions
diff --git a/ANNOUNCE b/ANNOUNCE
index 56374827f..4cf7944c7 100644
--- a/ANNOUNCE
+++ b/ANNOUNCE
@@ -1,5 +1,5 @@
-Libpng 1.5.27beta01 - December 29, 2015
+Libpng 1.5.27beta01 - January 13, 2016
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.
@@ -26,7 +26,7 @@ Other information:
Changes since the last public release (1.5.26):
-version 1.5.27beta01 [December 29, 2015]
+version 1.5.27beta01 [January 13, 2016]
Removed LE/BE dependencies in pngvalid, to 'fix' the current problem
in the BigEndian tests by not testing it, making the BE code the same
as the LE version.
@@ -45,6 +45,7 @@ version 1.5.27beta01 [December 29, 2015]
This changes pngvalid to use BE random numbers; this used to produce
errors but these should not be fixed as a result of the previous changes.
In projects/vstudio, combined readme.txt and WARNING into README.txt
+ Worked around a false-positive Coverity issue in pngvalid.c.
Send comments/corrections/commendations to png-mng-implement at lists.sf.net
(subscription required; visit
diff --git a/CHANGES b/CHANGES
index 7fb358216..fe6cee9f2 100644
--- a/CHANGES
+++ b/CHANGES
@@ -4454,7 +4454,7 @@ version 1.5.26rc01 [December 14, 2015]
version 1.5.26 [December 17, 2015]
No changes.
-version 1.5.27beta01 [December 29, 2015]
+version 1.5.27beta01 [January 13, 2016]
Removed LE/BE dependencies in pngvalid, to 'fix' the current problem
in the BigEndian tests by not testing it, making the BE code the same
as the LE version.
@@ -4473,6 +4473,7 @@ version 1.5.27beta01 [December 29, 2015]
This changes pngvalid to use BE random numbers; this used to produce
errors but these should not be fixed as a result of the previous changes.
In projects/vstudio, combined readme.txt and WARNING into README.txt
+ Worked around a false-positive Coverity issue in pngvalid.c.
Send comments/corrections/commendations to png-mng-implement at lists.sf.net
(subscription required; visit
diff --git a/contrib/libtests/pngvalid.c b/contrib/libtests/pngvalid.c
index f5b11de6c..9063b69fe 100644
--- a/contrib/libtests/pngvalid.c
+++ b/contrib/libtests/pngvalid.c
@@ -1,8 +1,8 @@
/* pngvalid.c - validate libpng by constructing then reading png files.
*
- * Last changed in libpng 1.5.27 [(PENDING RELEASE)]
- * Copyright (c) 2014-2015 Glenn Randers-Pehrson
+ * Last changed in libpng 1.6.21 [(PENDING RELEASE)]
+ * Copyright (c) 2014-2016 Glenn Randers-Pehrson
* Written by John Cunningham Bowler
*
* This code is released under the libpng license.
@@ -309,8 +309,13 @@ static void r16(png_uint_16p p16, size_t count)
}
}
-#define R16(this)\
+#ifdef __COVERITY__
+# define R16(this)\
+ r16(&(this), (sizeof (this))/2U/*(sizeof (png_uint_16))*/)
+#else
+# define R16(this)\
r16(&(this), (sizeof (this))/(sizeof (png_uint_16)))
+#endif
#if defined PNG_READ_RGB_TO_GRAY_SUPPORTED ||\
defined PNG_READ_FILLER_SUPPORTED
@@ -326,8 +331,14 @@ static void r32(png_uint_32p p32, size_t count)
}
}
-#define R32(this)\
+#ifdef __COVERITY__
+# define R32(this)\
+ r32(&(this), (sizeof (this))/4U/*(sizeof (png_uint_32))*/)
+#else
+# define R32(this)\
r32(&(this), (sizeof (this))/(sizeof (png_uint_32)))
+#endif
+
#endif /* READ_FILLER || READ_RGB_TO_GRAY */
#endif /* READ || WRITE_tRNS || WRITE_FILTER */
@@ -6370,6 +6381,8 @@ transform_range_check(png_const_structp pp, unsigned int r, unsigned int g,
png_error(pp, message);
}
+
+ UNUSED(limit)
}
static void