From: Bob Weinand <bobwei9@hotmail.com>
Date: Tue, 12 Jul 2016 23:05:32 +0000 (+0200)
Subject: Remove bogus zend_signal_shutdown symbol from header
X-Git-Tag: php-7.1.0beta1~82
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=7a125a44099fefbb1017c7b939385152cae71846;p=php

Remove bogus zend_signal_shutdown symbol from header
---

diff --git a/Zend/zend_signal.h b/Zend/zend_signal.h
index f9662b8d94..b58115b197 100644
--- a/Zend/zend_signal.h
+++ b/Zend/zend_signal.h
@@ -90,7 +90,6 @@ void zend_signal_activate(void);
 void zend_signal_deactivate(void);
 void zend_signal_startup(void);
 void zend_signal_init(void);
-void zend_signal_shutdown(void);
 
 ZEND_API int zend_signal(int signo, void (*handler)(int));
 ZEND_API int zend_sigaction(int signo, const struct sigaction *act, struct sigaction *oldact);
@@ -104,7 +103,6 @@ ZEND_API int zend_sigaction(int signo, const struct sigaction *act, struct sigac
 # define zend_signal_deactivate()
 # define zend_signal_startup()
 # define zend_signal_init()
-# define zend_signal_shutdown()
 
 # define zend_signal(signo, handler)           signal(signo, handler)
 # define zend_sigaction(signo, act, oldact)    sigaction(signo, act, oldact)
diff --git a/sapi/phpdbg/phpdbg.c b/sapi/phpdbg/phpdbg.c
index d851e0b082..1e0b214a23 100644
--- a/sapi/phpdbg/phpdbg.c
+++ b/sapi/phpdbg/phpdbg.c
@@ -1738,7 +1738,7 @@ phpdbg_main:
 		/* setup io here */
 		if (remote) {
 			PHPDBG_G(flags) |= PHPDBG_IS_REMOTE;
-			signal(SIGPIPE, SIG_IGN);
+			zend_signal(SIGPIPE, SIG_IGN);
 		}
 		PHPDBG_G(io)[PHPDBG_STDIN].ptr = stdin;
 		PHPDBG_G(io)[PHPDBG_STDIN].fd = fileno(stdin);
@@ -1968,6 +1968,10 @@ phpdbg_out:
 			zend_objects_store_mark_destructed(&EG(objects_store));
 		}
 
+		zend_try {
+			php_request_shutdown(NULL);
+		} zend_end_try();
+
 		/* backup globals when cleaning */
 		if ((cleaning > 0 || remote) && !quit_immediately) {
 			settings = calloc(1, sizeof(zend_phpdbg_globals));
@@ -1995,10 +1999,6 @@ phpdbg_out:
 			}
 		}
 
-		zend_try {
-			php_request_shutdown(NULL);
-		} zend_end_try();
-
 		if (exit_status == 0) {
 			exit_status = EG(exit_status);
 		}
@@ -2020,6 +2020,7 @@ phpdbg_out:
 		} zend_end_try();
 
 #ifndef _WIN32
+		/* force override (no zend_signals) to prevent crashes due to signal recursion in SIGSEGV/SIGBUS handlers */
 		signal(SIGSEGV, SIG_DFL);
 		signal(SIGBUS, SIG_DFL);
 
@@ -2038,8 +2039,6 @@ phpdbg_out:
 		free(sapi_name);
 	}
 
-	zend_signal_shutdown();
-
 #ifdef ZTS
 	tsrm_shutdown();
 #endif