using _SC_NPROCESSORS_CONF instead in case of cpu put offline.

addressing other remarks.

Author: devnexen

ext/pcntl/config.m4

@@ -7,18 +7,7 @@ if test "$PHP_PCNTL" != "no"; then
   AC_CHECK_FUNCS([fork], [], [AC_MSG_ERROR([pcntl: fork() not supported by this platform])])
   AC_CHECK_FUNCS([waitpid], [], [AC_MSG_ERROR([pcntl: waitpid() not supported by this platform])])
   AC_CHECK_FUNCS([sigaction], [], [AC_MSG_ERROR([pcntl: sigaction() not supported by this platform])])
-  AC_CHECK_FUNCS([
-	getpriority
-	setpriority
-	wait3
-	wait4
-	sigwaitinfo
-	sigtimedwait
-	unshare
-	rfork 
-	forkx
-	pidfd_open
-	sched_setaffinity])
+  AC_CHECK_FUNCS([getpriority setpriority wait3 wait4 sigwaitinfo sigtimedwait unshare rfork forkx pidfd_open sched_setaffinity])
 
   AC_CHECK_TYPE([siginfo_t],[PCNTL_CFLAGS="-DHAVE_STRUCT_SIGINFO_T"],,[#include <signal.h>])
 

ext/pcntl/pcntl.c

@@ -1514,7 +1514,7 @@ PHP_FUNCTION(pcntl_getcpuaffinity)
 		RETURN_FALSE;
 	}
 
-	zend_ulong maxcpus = (zend_ulong)sysconf(_SC_NPROCESSORS_ONLN);
+	zend_ulong maxcpus = (zend_ulong)sysconf(_SC_NPROCESSORS_CONF);
 	array_init(return_value);
 
 	for (zend_ulong i = 0; i < maxcpus; i ++) {
@@ -1544,18 +1544,25 @@ PHP_FUNCTION(pcntl_setcpuaffinity)
 
 	// 0 == getpid in this context, we're just saving a syscall
 	pid = pid_is_null ? 0 : pid;
-	zend_ulong maxcpus = (zend_ulong)sysconf(_SC_NPROCESSORS_ONLN);
+	zend_ulong maxcpus = (zend_ulong)sysconf(_SC_NPROCESSORS_CONF);
 	CPU_ZERO(&mask);
 
 	ZEND_HASH_FOREACH_VAL(Z_ARRVAL_P(hmask), ncpu) {
 		ZVAL_DEREF(ncpu);
 		zend_long cpu;
 		if (Z_TYPE_P(ncpu) != IS_LONG) {
 			if (Z_TYPE_P(ncpu) == IS_STRING) {
-				cpu = zval_get_long(ncpu);
+				char *str = Z_STRVAL_P(ncpu);
+				if (EXPECTED(strlen(str) && isdigit(str[0]))) {
+					cpu = zval_get_long(ncpu);
+				} else {
+					zend_value_error("cpu id invalid type (%s)", str);
+					RETURN_THROWS();
+				}
 			} else {
 				zend_string *wcpu = zval_get_string_func(ncpu);
 				zend_value_error("cpu id invalid type (%s)", ZSTR_VAL(wcpu));
+				zend_string_release(wcpu);
 				RETURN_THROWS();
 			}
 		} else {

ext/pcntl/tests/pcntl_cpuaffinity.phpt

@@ -60,6 +60,7 @@ array(0) {
 }
 bool(true)
 pcntl_setcpuaffinity(): Argument #2 ($cpu_ids) must not be empty
+cpu id invalid type (def)
 cpu id must be between 0 and %d (%d)
 cpu id must be between 0 and %d (-1024)
 pcntl_getcpuaffinity(): Argument #1 ($process_id) invalid process (-1024)