]> git.imager.perl.org - imager.git/commitdiff
on't check if the unsigned size supplied to im_set_image_file_limits() is negative
authorTony Cook <tony@develop-help.com>
Mon, 7 Jan 2019 12:18:56 +0000 (23:18 +1100)
committerTony Cook <tony@develop-help.com>
Mon, 7 Jan 2019 12:18:56 +0000 (23:18 +1100)
Changes
limits.c

diff --git a/Changes b/Changes
index 8df28b935232452abb7ff8c3bd5e23035edc4f4b..c5fccd25ce419d5503464bfa4ab40d509f24ebf0 100644 (file)
--- a/Changes
+++ b/Changes
@@ -129,6 +129,9 @@ Lower severity (according to Coverity):
    when calculating a palette with median cut for extraordinarily
    large images.  CID 185306.
 
    when calculating a palette with median cut for extraordinarily
    large images.  CID 185306.
 
+ - don't check if the unsigned size supplied to
+   im_set_image_file_limits() is negative.  CID 185307.
+
 [1] The first two build submissions ended up at the end of a ~400
 build queue, and seemed to have been cancelled by Coverity.  A build
 submitted on NYE went through in minutes.
 [1] The first two build submissions ended up at the end of a ~400
 build queue, and seemed to have been cancelled by Coverity.  A build
 submitted on NYE went through in minutes.
index 36ae45b5c9afcbe6b3ca98c8ea3b7afa81ffb2ff..e9f68bec3f0fad096d48100fb374301fcfdc163e 100644 (file)
--- a/limits.c
+++ b/limits.c
@@ -77,10 +77,6 @@ im_set_image_file_limits(pIMCTX, i_img_dim width, i_img_dim height, size_t bytes
     i_push_error(0, "height must be non-negative");
     return 0;
   }
     i_push_error(0, "height must be non-negative");
     return 0;
   }
-  if (bytes < 0) {
-    i_push_error(0, "bytes must be non-negative");
-    return 0;
-  }
 
   aIMCTX->max_width = width;
   aIMCTX->max_height = height;
 
   aIMCTX->max_width = width;
   aIMCTX->max_height = height;