From: Antony Dovgal <tony2001@php.net>
Date: Tue, 20 Feb 2007 19:19:14 +0000 (+0000)
Subject: don't call php_module_startup() directly, use startup functions
X-Git-Tag: RELEASE_1_0_1~205
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=2ad6c07ca170ae5720f201e0ffae6ca56c2854e3;p=php

don't call php_module_startup() directly, use startup functions
patch by Andrei Nigmatulin
---

diff --git a/sapi/cgi/cgi_main.c b/sapi/cgi/cgi_main.c
index 4805f2ca48..4598ddb1f1 100644
--- a/sapi/cgi/cgi_main.c
+++ b/sapi/cgi/cgi_main.c
@@ -116,6 +116,7 @@ static pid_t pgroup;
 
 static char *php_optarg = NULL;
 static int php_optind = 1;
+static zend_module_entry cgi_module_entry;
 
 static const opt_struct OPTIONS[] = {
 	{'a', 0, "interactive"},
@@ -542,7 +543,7 @@ static int sapi_cgi_deactivate(TSRMLS_D)
 
 static int php_cgi_startup(sapi_module_struct *sapi_module)
 {
-	if (php_module_startup(sapi_module, NULL, 0) == FAILURE) {
+	if (php_module_startup(sapi_module, &cgi_module_entry, 1) == FAILURE) {
 		return FAILURE;
 	}
 	return SUCCESS;
@@ -1211,7 +1212,7 @@ int main(int argc, char *argv[])
 	cgi_sapi_module.additional_functions = additional_functions;
 
 	/* startup after we get the above ini override se we get things right */
-	if (php_module_startup(&cgi_sapi_module, &cgi_module_entry, 1) == FAILURE) {
+	if (cgi_sapi_module.startup(&cgi_sapi_module) == FAILURE) {
 #ifdef ZTS
 		tsrm_shutdown();
 #endif
diff --git a/sapi/cli/php_cli.c b/sapi/cli/php_cli.c
index f51185a6a0..996a5674a1 100644
--- a/sapi/cli/php_cli.c
+++ b/sapi/cli/php_cli.c
@@ -710,7 +710,7 @@ int main(int argc, char *argv[])
 #endif
 
 	/* startup after we get the above ini override se we get things right */
-	if (php_module_startup(&cli_sapi_module, NULL, 0)==FAILURE) {
+	if (cli_sapi_module.startup(&cli_sapi_module)==FAILURE) {
 		/* there is no way to see if we must call zend_ini_deactivate()
 		 * since we cannot check if EG(ini_directives) has been initialised
 		 * because the executor's constructor does not set initialize it.