Warnings to Errors imagetruecolortopalette

Author: marandall

ext/gd/gd.c

@@ -885,9 +885,10 @@ PHP_FUNCTION(imagetruecolortopalette)
 	}
 
 	if (ncolors <= 0 || ZEND_LONG_INT_OVFL(ncolors)) {
-		php_error_docref(NULL, E_WARNING, "Number of colors has to be greater than zero and no more than %d", INT_MAX);
-		RETURN_FALSE;
+		zend_throw_error(NULL, "Number of colors has to be greater than zero and no more than %d", INT_MAX);
+		return;
 	}
+
 	if (gdImageTrueColorToPalette(im, dither, (int)ncolors)) {
 		RETURN_TRUE;
 	} else {

ext/gd/tests/bug72697.phpt

@@ -8,10 +8,16 @@ if (PHP_INT_MAX !== 9223372036854775807) die("skip for 64-bit long systems only"
 --FILE--
 <?php
 
+require __DIR__ . '/func.inc';
+
 $img=imagecreatetruecolor(10, 10);
-imagetruecolortopalette($img, false, PHP_INT_MAX / 8);
+
+trycatch_dump(
+    fn() => imagetruecolortopalette($img, false, PHP_INT_MAX / 8)
+);
+
 ?>
 DONE
---EXPECTF--
-Warning: imagetruecolortopalette(): Number of colors has to be greater than zero and no more than 2147483647 in %sbug72697.php on line %d
+--EXPECT--
+!! [Error] Number of colors has to be greater than zero and no more than 2147483647
 DONE

ext/gd/tests/imagetruecolortopalette_error3.phpt

@@ -9,8 +9,14 @@ Rafael Dohms <rdohms [at] gmail [dot] com>
 ?>
 --FILE--
 <?php
+require __DIR__  . '/func.inc';
+
 $image = imagecreatetruecolor(50, 50);
-imagetruecolortopalette($image, true, null);
+
+trycatch_dump(
+    fn() => imagetruecolortopalette($image, true, null)
+);
+
 ?>
---EXPECTF--
-Warning: imagetruecolortopalette(): Number of colors has to be greater than zero and no more than %d in %s on line %d
+--EXPECT--
+!! [Error] Number of colors has to be greater than zero and no more than 2147483647

ext/gd/tests/imagetruecolortopalette_error4.phpt

@@ -9,13 +9,16 @@ Rafael Dohms <rdohms [at] gmail [dot] com>
 ?>
 --FILE--
 <?php
+require __DIR__  . '/func.inc';
+
 $image = imagecreatetruecolor(50, 50);
 
-imagetruecolortopalette($image, true, 0);
-imagetruecolortopalette($image, true, -1);
+trycatch_dump(
+    fn() => imagetruecolortopalette($image, true, 0),
+    fn() => imagetruecolortopalette($image, true, -1)
+);
 
 ?>
---EXPECTF--
-Warning: imagetruecolortopalette(): Number of colors has to be greater than zero and no more than %d in %s line %d
-
-Warning: imagetruecolortopalette(): Number of colors has to be greater than zero and no more than %d in %s line %d
+--EXPECT--
+!! [Error] Number of colors has to be greater than zero and no more than 2147483647
+!! [Error] Number of colors has to be greater than zero and no more than 2147483647