From: Tony Cook Date: Mon, 7 Jan 2019 12:18:56 +0000 (+1100) Subject: on't check if the unsigned size supplied to im_set_image_file_limits() is negative X-Git-Tag: v1.009~4 X-Git-Url: http://git.imager.perl.org/imager.git/commitdiff_plain/748bfdfc4d4f60631ff5d06f2f954a36f2ea8ee5 on't check if the unsigned size supplied to im_set_image_file_limits() is negative --- diff --git a/Changes b/Changes index 8df28b93..c5fccd25 100644 --- 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. + - 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. diff --git a/limits.c b/limits.c index 36ae45b5..e9f68bec 100644 --- 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; } - if (bytes < 0) { - i_push_error(0, "bytes must be non-negative"); - return 0; - } aIMCTX->max_width = width; aIMCTX->max_height = height;