diff options
author | Nikita Popov <nikita.ppv@gmail.com> | 2019-07-29 16:58:29 +0200 |
---|---|---|
committer | Nikita Popov <nikita.ppv@gmail.com> | 2019-07-29 17:14:40 +0200 |
commit | 6df1665bb3d5b5d8f799ec40b27463df2b630c51 (patch) | |
tree | 1d0f81af278bafa82425e12eb4a00bc2f14e61c1 /Zend/zend_signal.c | |
parent | 94924841ac42b9ef4445abfbe7d5d77c93e181f0 (diff) | |
parent | bbed5564eb8155efb14146e11ef0ad67edb20e28 (diff) | |
download | php-git-6df1665bb3d5b5d8f799ec40b27463df2b630c51.tar.gz |
Merge branch 'PHP-7.3' into PHP-7.4
Diffstat (limited to 'Zend/zend_signal.c')
-rw-r--r-- | Zend/zend_signal.c | 21 |
1 files changed, 16 insertions, 5 deletions
diff --git a/Zend/zend_signal.c b/Zend/zend_signal.c index 703097eeb5..96870f8d5c 100644 --- a/Zend/zend_signal.c +++ b/Zend/zend_signal.c @@ -328,13 +328,13 @@ void zend_signal_activate(void) SIGG(active) = 1; SIGG(depth) = 0; + SIGG(check) = ZEND_DEBUG; } /* }}} */ /* {{{ zend_signal_deactivate * */ void zend_signal_deactivate(void) { - if (SIGG(check)) { size_t x; struct sigaction sa; @@ -342,21 +342,32 @@ void zend_signal_deactivate(void) if (SIGG(depth) != 0) { zend_error(E_CORE_WARNING, "zend_signal: shutdown with non-zero blocking depth (%d)", SIGG(depth)); } + /* did anyone steal our installed handler */ for (x = 0; x < sizeof(zend_sigs) / sizeof(*zend_sigs); x++) { sigaction(zend_sigs[x], NULL, &sa); - if (sa.sa_sigaction != zend_signal_handler_defer) { + if (sa.sa_sigaction != zend_signal_handler_defer && + sa.sa_sigaction != (void *) SIG_IGN) { zend_error(E_CORE_WARNING, "zend_signal: handler was replaced for signal (%d) after startup", zend_sigs[x]); } } } - SIGNAL_BEGIN_CRITICAL(); - SIGG(active) = 0; + /* After active=0 is set, signal handlers will be called directly and other + * state that is reset below will not be accessed. */ + *((volatile int *) &SIGG(active)) = 0; + SIGG(running) = 0; SIGG(blocked) = 0; SIGG(depth) = 0; - SIGNAL_END_CRITICAL(); + + /* If there are any queued signals because of a missed unblock, drop them. */ + if (SIGG(phead) && SIGG(ptail)) { + SIGG(ptail)->next = SIGG(pavail); + SIGG(pavail) = SIGG(phead); + SIGG(phead) = NULL; + SIGG(ptail) = NULL; + } } /* }}} */ |