make exceeding arglists behave the same for both native and userland functions

Author: marcioAlmada

Zend/zend_API.c

@@ -208,8 +208,8 @@ ZEND_API ZEND_COLD void ZEND_FASTCALL zend_wrong_parameters_count_error(int num_
 		class_name, \
 		class_name[0] ? "::" : "", \
 		ZSTR_VAL(active_function->common.function_name),
-		min_num_args == max_num_args ? "exactly" : num_args < min_num_args ? "at least" : "at most",
-		num_args < min_num_args ? min_num_args : max_num_args,
+		min_num_args == max_num_args ? "exactly" : "at least",
+		min_num_args,
 		(num_args < min_num_args ? min_num_args : max_num_args) == 1 ? "" : "s",
 		num_args);
 }
@@ -738,7 +738,7 @@ static const char *zend_parse_arg_impl(int arg_num, zval *arg, va_list *va, cons
 			/* 'Z' iz not supported anymore and should be replaced with 'z' */
 			ZEND_ASSERT(c != 'Z');
 		default:
-			return "unknown";
+			return NULL;
 	}
 
 	*spec = spec_walk;
@@ -872,7 +872,7 @@ static int zend_parse_va_args(int num_args, const char *type_spec, va_list *va,
 		max_num_args = -1;
 	}
 
-	if (num_args < min_num_args || (num_args > max_num_args && max_num_args >= 0)) {
+	if (num_args < min_num_args) {
 		if (!(flags & ZEND_PARSE_PARAMS_QUIET)) {
 			zend_function *active_function = EG(current_execute_data)->func;
 			const char *class_name = active_function->common.scope ? ZSTR_VAL(active_function->common.scope->name) : "";
@@ -881,8 +881,8 @@ static int zend_parse_va_args(int num_args, const char *type_spec, va_list *va,
 					class_name,
 					class_name[0] ? "::" : "",
 					ZSTR_VAL(active_function->common.function_name),
-					min_num_args == max_num_args ? "exactly" : num_args < min_num_args ? "at least" : "at most",
-					num_args < min_num_args ? min_num_args : max_num_args,
+					min_num_args == max_num_args ? "exactly" : "at least",
+					min_num_args,
 					(num_args < min_num_args ? min_num_args : max_num_args) == 1 ? "" : "s",
 					num_args);
 		}

Zend/zend_API.h

@@ -725,9 +725,7 @@ ZEND_API ZEND_COLD void ZEND_FASTCALL zend_wrong_callback_error(int severity, in
 		((void)_optional); \
 		\
 		do { \
-			if (UNEXPECTED(_num_args < _min_num_args) || \
-			    (UNEXPECTED(_num_args > _max_num_args) && \
-			     EXPECTED(_max_num_args >= 0))) { \
+			if (UNEXPECTED(_num_args < _min_num_args)) { \
 				if (!(_flags & ZEND_PARSE_PARAMS_QUIET)) { \
 					zend_wrong_parameters_count_error(_num_args, _min_num_args, _max_num_args); \
 				} \