ext/sockets: socket_strerror follow-up on GH-16267 fix.

boundaries should be INT_MIN <= val < INT_MAX in fact.

Author: devnexen

ext/sockets/sockets.c

@@ -1248,8 +1248,8 @@ PHP_FUNCTION(socket_strerror)
 		Z_PARAM_LONG(arg1)
 	ZEND_PARSE_PARAMETERS_END();
 
-	if (ZEND_LONG_EXCEEDS_INT(arg1)) {
-		zend_argument_value_error(1, "must be between %d and %d", INT_MIN, INT_MAX);
+	if (arg1 <= INT_MIN || arg1 > INT_MAX) {
+		zend_argument_value_error(1, "must be greater than %d and lower than %d", INT_MIN, INT_MAX);
 		RETURN_THROWS();
 	}
 

ext/sockets/tests/gh16267.phpt

@@ -7,16 +7,16 @@ sockets
 --FILE--
 <?php
 try {
-	socket_strerror(PHP_INT_MIN);
+	socket_strerror(-2147483648);
 } catch (\ValueError $e) {
 	echo $e->getMessage() . PHP_EOL;
 }
 try {
-	socket_strerror(PHP_INT_MAX);
+	socket_strerror(2147483648);
 } catch (\ValueError $e) {
 	echo $e->getMessage() . PHP_EOL;
 }
 ?>
 --EXPECTF--
-socket_strerror(): Argument #1 ($error_code) must be between %s and %s
-socket_strerror(): Argument #1 ($error_code) must be between %s and %s
+socket_strerror(): Argument #1 ($error_code) must be greater than %s and lower than %s
+socket_strerror(): Argument #1 ($error_code) must be greater than %s and lower than %s