Set SA_ONSTACK too when zend signals are disabled and in pcntl

Closes GH-9758.

Author: dunglas

NEWS

@@ -11,6 +11,7 @@ PHP                                                                        NEWS
     error message). (ilutov)
   . SA_ONSTACK is now set for signal handlers to be friendlier to other
     in-process code such as Go's cgo. (Kévin Dunglas)
+  . SA_ONSTACK is now set when signals are disabled. (Kévin Dunglas)
 
 - Fileinfo:
   . Upgrade bundled libmagic to 5.43. (Anatol)
@@ -27,6 +28,9 @@ PHP                                                                        NEWS
   . Made opcache.preload_user always optional in the cli and phpdbg SAPIs.
     (Arnaud)
 
+- PCNTL:
+  . SA_ONSTACK is now set for pcntl_signal. (Kévin Dunglas)
+
 - Posix:
   . Added posix_sysconf. (David Carlier)
 

Zend/zend_execute_API.c

@@ -1483,7 +1483,7 @@ static void zend_set_timeout_ex(zend_long seconds, bool reset_signals) /* {{{ */
 
 			act.sa_handler = zend_timeout_handler;
 			sigemptyset(&act.sa_mask);
-			act.sa_flags = SA_RESETHAND | SA_NODEFER;
+			act.sa_flags = SA_ONSTACK | SA_RESETHAND | SA_NODEFER;
 			sigaction(signo, &act, NULL);
 #  else
 			signal(signo, zend_timeout_handler);

ext/pcntl/php_signal.c

@@ -35,7 +35,7 @@ Sigfunc *php_signal4(int signo, Sigfunc *func, int restart, int mask_all)
 	} else {
 		sigemptyset(&act.sa_mask);
 	}
-	act.sa_flags = 0;
+	act.sa_flags = SA_ONSTACK;
 #ifdef HAVE_STRUCT_SIGINFO_T
 	act.sa_flags |= SA_SIGINFO;
 #endif