ext/gd: report $size with the argument number of the function called

php_imagettftext_common() serves the bbox functions, where $size is
argument #1, and the drawing ones, where it is #2, but its two size
checks hardcode 2. imageftbbox(NAN, ...) therefore blamed $angle.

Author: lacatoire

ext/gd/gd.c

@@ -3422,14 +3422,16 @@ static void php_imagettftext_common(INTERNAL_FUNCTION_PARAMETERS, int mode)
 		im = php_gd_libgdimageptr_from_zval_p(IM);
 	}
 
+	uint32_t ptsize_arg_num = mode == TTFTEXT_BBOX ? 1 : 2;
+
 	// FT_F26Dot6 is a signed long alias
 	if (ptsize < (double)LONG_MIN / 64 || ptsize > (double)LONG_MAX / 64) {
-		zend_argument_value_error(2, "must be between " ZEND_LONG_FMT " and " ZEND_LONG_FMT, (zend_long)((double)LONG_MIN / 64), (zend_long)((double)LONG_MAX / 64));
+		zend_argument_value_error(ptsize_arg_num, "must be between " ZEND_LONG_FMT " and " ZEND_LONG_FMT, (zend_long)((double)LONG_MIN / 64), (zend_long)((double)LONG_MAX / 64));
 		RETURN_THROWS();
 	}
 
 	if (UNEXPECTED(!zend_finite(ptsize))) {
-		zend_argument_value_error(2, "must be finite");
+		zend_argument_value_error(ptsize_arg_num, "must be finite");
 		RETURN_THROWS();
 	}
 

ext/gd/tests/imageftbbox_size_arg_num.phpt

@@ -0,0 +1,62 @@
+--TEST--
+The $size errors name the argument of the function that was called
+--EXTENSIONS--
+gd
+--SKIPIF--
+<?php
+if (!function_exists('imageftbbox')) die('skip imageftbbox() not available');
+?>
+--FILE--
+<?php
+$font = __DIR__ . '/Rochester-Regular.otf';
+$image = imagecreatetruecolor(100, 80);
+
+/* $size is argument #1 here */
+foreach ([NAN, INF, PHP_INT_MAX, PHP_INT_MIN] as $size) {
+    try {
+        imageftbbox($size, 0.0, $font, 'A');
+    } catch (ValueError $e) {
+        echo $e->getMessage(), "\n";
+    }
+    try {
+        imagettfbbox($size, 0.0, $font, 'A');
+    } catch (ValueError $e) {
+        echo $e->getMessage(), "\n";
+    }
+}
+
+/* and argument #2 here */
+foreach ([NAN, INF] as $size) {
+    try {
+        imagefttext($image, $size, 0.0, 15, 60, 0, $font, 'A');
+    } catch (ValueError $e) {
+        echo $e->getMessage(), "\n";
+    }
+    try {
+        imagettftext($image, $size, 0.0, 15, 60, 0, $font, 'A');
+    } catch (ValueError $e) {
+        echo $e->getMessage(), "\n";
+    }
+}
+
+/* the type error already agreed with the signature and still does */
+try {
+    imageftbbox('x', 0.0, $font, 'A');
+} catch (TypeError $e) {
+    echo $e->getMessage(), "\n";
+}
+?>
+--EXPECTF--
+imageftbbox(): Argument #1 ($size) must be finite
+imagettfbbox(): Argument #1 ($size) must be finite
+imageftbbox(): Argument #1 ($size) must be between %i and %d
+imagettfbbox(): Argument #1 ($size) must be between %i and %d
+imageftbbox(): Argument #1 ($size) must be between %i and %d
+imagettfbbox(): Argument #1 ($size) must be between %i and %d
+imageftbbox(): Argument #1 ($size) must be between %i and %d
+imagettfbbox(): Argument #1 ($size) must be between %i and %d
+imagefttext(): Argument #2 ($size) must be finite
+imagettftext(): Argument #2 ($size) must be finite
+imagefttext(): Argument #2 ($size) must be between %i and %d
+imagettftext(): Argument #2 ($size) must be between %i and %d
+imageftbbox(): Argument #1 ($size) must be of type float, string given