From 2a9ea2c9f57aaf8b8bb2cb14102ae82679407d13 Mon Sep 17 00:00:00 2001 From: Niels Dossche <7771979+nielsdos@users.noreply.github.com> Date: Sat, 24 May 2025 14:35:32 +0200 Subject: [PATCH] Drop FIXME in pcntl_signal_dispatch() (#18633) --- ext/pcntl/pcntl.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/ext/pcntl/pcntl.c b/ext/pcntl/pcntl.c index 2189f94c992..e6cba81d988 100644 --- a/ext/pcntl/pcntl.c +++ b/ext/pcntl/pcntl.c @@ -1356,7 +1356,6 @@ void pcntl_signal_dispatch(void) while (queue) { if ((handle = zend_hash_index_find(&PCNTL_G(php_signal_table), queue->signo)) != NULL) { if (Z_TYPE_P(handle) != IS_LONG) { - ZVAL_NULL(&retval); ZVAL_LONG(¶ms[0], queue->signo); #ifdef HAVE_STRUCT_SIGINFO_T array_init(¶ms[1]); @@ -1366,7 +1365,6 @@ void pcntl_signal_dispatch(void) #endif /* Call php signal handler - Note that we do not report errors, and we ignore the return value */ - /* FIXME: this is probably broken when multiple signals are handled in this while loop (retval) */ call_user_function(NULL, NULL, handle, &retval, 2, params); zval_ptr_dtor(&retval); #ifdef HAVE_STRUCT_SIGINFO_T