Minor refactoring of basic functions

ext/standard/basic_functions.c

@@ -596,7 +596,12 @@ PHP_FUNCTION(ip2long)
 		Z_PARAM_PATH(addr, addr_len)
 	ZEND_PARSE_PARAMETERS_END();
 
-	if (addr_len == 0 || inet_pton(AF_INET, addr, &ip) != 1) {
+	if (addr_len == 0) {
+		zend_argument_must_not_be_empty_error(1);
+		RETURN_THROWS();
+	}
+
+	if (inet_pton(AF_INET, addr, &ip) != 1) {
 		RETURN_FALSE;
 	}
 #ifdef _AIX
@@ -1177,8 +1182,8 @@ PHP_FUNCTION(time_nanosleep)
 		zend_argument_value_error(1, "must be greater than or equal to 0");
 		RETURN_THROWS();
 	}
-	if (tv_nsec < 0) {
-		zend_argument_value_error(2, "must be greater than or equal to 0");
+	if (tv_nsec < 0 || tv_nsec > 999999999) {
+		zend_argument_value_error(2, "must be between 0 and 999999999");
 		RETURN_THROWS();
 	}
 
@@ -1192,10 +1197,8 @@ PHP_FUNCTION(time_nanosleep)
 		add_assoc_long_ex(return_value, "seconds", sizeof("seconds")-1, php_rem.tv_sec);
 		add_assoc_long_ex(return_value, "nanoseconds", sizeof("nanoseconds")-1, php_rem.tv_nsec);
 		return;
-	} else if (errno == EINVAL) {
-		zend_value_error("Nanoseconds was not in the range 0 to 999 999 999 or seconds was negative");
-		RETURN_THROWS();
 	}
+	ZEND_ASSERT(errno != EINVAL);
 
 	RETURN_FALSE;
 }
@@ -2119,7 +2122,7 @@ PHP_FUNCTION(connection_aborted)
 {
 	ZEND_PARSE_PARAMETERS_NONE();
 
-	RETURN_LONG(PG(connection_status) & PHP_CONNECTION_ABORTED);
+	RETURN_BOOL(PG(connection_status) & PHP_CONNECTION_ABORTED);
 }
 /* }}} */
 
@@ -2137,22 +2140,22 @@ PHP_FUNCTION(ignore_user_abort)
 {
 	bool arg = 0;
 	bool arg_is_null = 1;
-	int old_setting;
+	bool old_setting;
 
 	ZEND_PARSE_PARAMETERS_START(0, 1)
 		Z_PARAM_OPTIONAL
 		Z_PARAM_BOOL_OR_NULL(arg, arg_is_null)
 	ZEND_PARSE_PARAMETERS_END();
 
-	old_setting = (unsigned short)PG(ignore_user_abort);
+	old_setting = PG(ignore_user_abort);
 
 	if (!arg_is_null) {
 		zend_string *key = ZSTR_INIT_LITERAL("ignore_user_abort", 0);
 		zend_alter_ini_entry_chars(key, arg ? "1" : "0", 1, PHP_INI_USER, PHP_INI_STAGE_RUNTIME);
 		zend_string_release_ex(key, 0);
 	}
 
-	RETURN_LONG(old_setting);
+	RETURN_BOOL(old_setting);
 }
 /* }}} */
 

ext/standard/basic_functions.stub.php

@@ -2039,11 +2039,11 @@ function get_include_path(): string|false {}
 /** @refcount 1 */
 function print_r(mixed $value, bool $return = false): string|true {}
 
-function connection_aborted(): int {}
+function connection_aborted(): bool {}
 
 function connection_status(): int {}
 
-function ignore_user_abort(?bool $enable = null): int {}
+function ignore_user_abort(?bool $enable = null): bool {}
 
 #ifdef HAVE_GETSERVBYNAME
 function getservbyname(string $service, string $protocol): int|false {}

ext/standard/tests/general_functions/bug72300.phpt

@@ -14,9 +14,9 @@ var_dump(ini_get("ignore_user_abort"));
 
 ?>
 --EXPECT--
-int(0)
-int(1)
+bool(false)
+bool(true)
 string(1) "1"
-int(1)
-int(0)
+bool(true)
+bool(false)
 string(1) "0"

ext/standard/tests/general_functions/connection_aborted.phpt

@@ -1,10 +1,10 @@
 --TEST--
-int connection_aborted ( void );
+Basic test for connection_aborted()
 --CREDITS--
 marcosptf - <marcosptf@yahoo.com.br> - #phparty7 - @phpsp - novatec/2015 - sao paulo - br
 --FILE--
 <?php
 var_dump(connection_aborted());
 ?>
 --EXPECT--
-int(0)
+bool(false)

ext/standard/tests/misc/time_nanosleep_error4.phpt

@@ -10,12 +10,12 @@ if (!function_exists('time_nanosleep')) die("skip");
 --FILE--
 <?php
 
-time_nanosleep(0, -10);
+try {
+	time_nanosleep(0, -10);
+} catch (ValueError $exception) {
+    echo $exception->getMessage(), "\n";
+}
 
 ?>
---EXPECTF--
-Fatal error: Uncaught ValueError: time_nanosleep(): Argument #2 ($nanoseconds) must be greater than or equal to 0 in %s:%d
-Stack trace:
-#0 %s(%d): time_nanosleep(0, -10)
-#1 {main}
-  thrown in %s on line %d
+--EXPECT--
+time_nanosleep(): Argument #2 ($nanoseconds) must be between 0 and 999999999

ext/standard/tests/misc/time_nanosleep_error5.phpt

@@ -7,12 +7,12 @@ time_nanosleep — Delay for a number of seconds and nanoseconds
 --FILE--
 <?php
 
-time_nanosleep(0, 1000000000);
+try {
+	time_nanosleep(0, 1000000000);
+} catch (ValueError $exception) {
+    echo $exception->getMessage(), "\n";
+}
 
 ?>
---EXPECTF--
-Fatal error: Uncaught ValueError: Nanoseconds was not in the range 0 to 999 999 999 or seconds was negative in %s:%d
-Stack trace:
-#0 %s(%d): time_nanosleep(0, 1000000000)
-#1 {main}
-  thrown in %s on line %d
+--EXPECT--
+time_nanosleep(): Argument #2 ($nanoseconds) must be between 0 and 999999999

ext/standard/tests/network/ip.phpt

@@ -7,21 +7,20 @@ if (PHP_INT_SIZE != 4) die("skip this test is for 32bit platform only");
 --FILE--
 <?php
 
-$array = array(
+$array = [
     "127.0.0.1",
     "10.0.0.1",
     "255.255.255.255",
     "255.255.255.0",
     "0.0.0.0",
     "66.163.161.116",
-);
+];
 
 foreach ($array as $ip) {
     var_dump($long = ip2long($ip));
     var_dump(long2ip($long));
 }
 
-var_dump(ip2long(""));
 var_dump(ip2long("777.777.777.777"));
 var_dump(ip2long("111.111.111.111"));
 
@@ -43,7 +42,6 @@ string(7) "0.0.0.0"
 int(1118019956)
 string(14) "66.163.161.116"
 bool(false)
-bool(false)
 int(1869573999)
 string(13) "255.254.82.80"
 Done

ext/standard/tests/network/ip2long_errors.phpt

@@ -0,0 +1,20 @@
+--TEST--
+ip2long() error conditions
+--FILE--
+<?php
+
+try {
+	var_dump(ip2long(""));
+} catch (Throwable $e) {
+	echo $e::class, ': ', $e->getMessage(), PHP_EOL;
+}
+try {
+	var_dump(ip2long("127\00.0.1"));
+} catch (Throwable $e) {
+	echo $e::class, ': ', $e->getMessage(), PHP_EOL;
+}
+?>
+--EXPECT--
+ValueError: ip2long(): Argument #1 ($ip) must not be empty
+ValueError: ip2long(): Argument #1 ($ip) must not contain any null bytes
+

ext/standard/tests/network/ip_x86_64.phpt

@@ -7,21 +7,19 @@ if (PHP_INT_SIZE == 4) die("skip this test is for >32bit platform only");
 --FILE--
 <?php
 
-$array = array(
+$array = [
     "127.0.0.1",
     "10.0.0.1",
     "255.255.255.255",
     "255.255.255.0",
     "0.0.0.0",
     "66.163.161.116",
-);
+];
 
 foreach ($array as $ip) {
     var_dump($long = ip2long($ip));
     var_dump(long2ip($long));
 }
-
-var_dump(ip2long(""));
 var_dump(ip2long("777.777.777.777"));
 var_dump(ip2long("111.111.111.111"));
 
@@ -45,7 +43,6 @@ string(7) "0.0.0.0"
 int(1118019956)
 string(14) "66.163.161.116"
 bool(false)
-bool(false)
 int(1869573999)
 string(13) "255.254.82.80"
 string(15) "255.255.255.255"

ext/standard/tests/time/bug60222.phpt

@@ -2,18 +2,18 @@
 Bug #60222 (time_nanosleep() does validate input params)
 --FILE--
 <?php
-    try {
-        time_nanosleep(-1, 0);
-    } catch (Throwable $exception) {
-        echo $exception::class, ': ', $exception->getMessage(), "\n";
-    }
+try {
+    time_nanosleep(-1, 0);
+} catch (Throwable $exception) {
+    echo $exception::class, ': ', $exception->getMessage(), "\n";
+}
 
-    try {
-        time_nanosleep(0, -1);
-    } catch (Throwable $exception) {
-        echo $exception::class, ': ', $exception->getMessage(), "\n";
-    }
+try {
+    time_nanosleep(0, -1);
+} catch (Throwable $exception) {
+    echo $exception::class, ': ', $exception->getMessage(), "\n";
+}
 ?>
 --EXPECT--
 ValueError: time_nanosleep(): Argument #1 ($seconds) must be greater than or equal to 0
-ValueError: time_nanosleep(): Argument #2 ($nanoseconds) must be greater than or equal to 0
+ValueError: time_nanosleep(): Argument #2 ($nanoseconds) must be between 0 and 999999999