Re: [PHP-DEV] Critical Bug #20887

2002-12-12 Thread Jani Taskinen
But unfortunately neither of these fix the bug. If there is php.ini in /, it's still used. --Jani On Thu, 12 Dec 2002, Moriyoshi Koizumi wrote: +1 for applying this patch. and attached is yet another fix as my suggestion. (a bit dirty, and not tested enough). Moriyoshi

Re: [PHP-DEV] Critical Bug #20887

2002-12-12 Thread Moriyoshi Koizumi
Hmm... it's natural that when apache is launched at /, it should read /php.ini because of the current implementation shown below. 274 #ifdef INI_CHECK_CWD 275 if (strcmp(sapi_module.name, cli)!=0) { 276 if (*php_ini_search_path) { 277 strcat(php_ini_search_path,

Re: [PHP-DEV] Critical Bug #20887

2002-12-12 Thread Jani Taskinen
On Thu, 12 Dec 2002, Moriyoshi Koizumi wrote: Hmm... it's natural that when apache is launched at /, it should read /php.ini because of the current implementation shown below. 274 #ifdef INI_CHECK_CWD 275 if (strcmp(sapi_module.name, cli)!=0) { 276if (*php_ini_search_path) { 277

Re: [PHP-DEV] Critical Bug #20887

2002-12-12 Thread Derick Rethans
On Thu, 12 Dec 2002, Jani Taskinen wrote: On Thu, 12 Dec 2002, Moriyoshi Koizumi wrote: Hmm... it's natural that when apache is launched at /, it should read /php.ini because of the current implementation shown below. 274 #ifdef INI_CHECK_CWD 275 if (strcmp(sapi_module.name, cli)!=0) {

Re: [PHP-DEV] Critical Bug #20887

2002-12-12 Thread Moriyoshi Koizumi
You are right. I verified Apache changes the cwd to / unless it's been launched in single process mode. And I found results could be different by cases, using DSO or using CGI executable. When you run your script with CGI executable and php.ini is also present in that directory, the PHP binary

Re: [PHP-DEV] Critical Bug #20887

2002-12-12 Thread Edin Kadribasic
You are right. I verified Apache changes the cwd to / unless it's been launched in single process mode. And I found results could be different by cases, using DSO or using CGI executable. When you run your script with CGI executable and php.ini is also present in that directory, the PHP

Re: [PHP-DEV] Critical Bug #20887

2002-12-12 Thread Moriyoshi Koizumi
At the time CLI was introduced I argued to remove . from php.ini search path, but that was not accepted because some people apparently use this feature for having different configurations for different virtual hosts. Therefore . was removed only from CLI's php.ini search path. This feature

Re: [PHP-DEV] Critical Bug #20887

2002-12-12 Thread Edin Kadribasic
PROTECTED] Cc: Derick Rethans [EMAIL PROTECTED]; Jani Taskinen [EMAIL PROTECTED]; [EMAIL PROTECTED]; [EMAIL PROTECTED] Sent: Thursday, December 12, 2002 12:44 PM Subject: Re: [PHP-DEV] Critical Bug #20887 At the time CLI was introduced I argued to remove . from php.ini search path

Re: [PHP-DEV] Critical Bug #20887

2002-12-12 Thread Moriyoshi Koizumi
[EMAIL PROTECTED]; Jani Taskinen [EMAIL PROTECTED]; [EMAIL PROTECTED]; [EMAIL PROTECTED] Sent: Thursday, December 12, 2002 12:44 PM Subject: Re: [PHP-DEV] Critical Bug #20887 At the time CLI was introduced I argued to remove . from php.ini search path, but that was not accepted because

Re: [PHP-DEV] Critical Bug #20887

2002-12-11 Thread Moriyoshi Koizumi
+1 for applying this patch. and attached is yet another fix as my suggestion. (a bit dirty, and not tested enough). Moriyoshi Sara Golemon [EMAIL PROTECTED] wrote: I THINK the patch below will fix critical bug #20887, but it's late and I've had a long day so I havn't begun to make sure