ID: 10960
Updated by: sniper
Reported By: [EMAIL PROTECTED]
Old-Status: Open
Status: Closed
Bug Type: PHP options/info functions
Operating system: 
PHP Version: 4.0.5
Assigned To: 
Comments:

patch applied.

--Jani


Previous Comments:
---------------------------------------------------------------------------

[2001-05-23 02:36:54] [EMAIL PROTECTED]
patch applied.

--Jani


---------------------------------------------------------------------------

[2001-05-18 16:29:10] [EMAIL PROTECTED]
Sorry, left in an extra paren.  The fix should be:


+       if ((strcmp(basedir, ".") == 0) && 
 

---------------------------------------------------------------------------

[2001-05-18 14:12:38] [EMAIL PROTECTED]
>From previous bug report 10322 submitted by Paul Gregg:

In main/fopen_wrappers.c I see that there is a function:
PHPAPI int php_check_specific_open_basedir(char *basedir, char *path PLS_DC)

However "basedir" is never used in this function at all,
only PG(open_basedir).  Surely this negates the point of the function being > called 
individually for each tokenised entry on  open_basedir/php.ini?

--- end of bug report 10322

Although Paul was correct, "Jason Greene" <[EMAIL PROTECTED]> marked his report as bogus, 
apparently because it was included as part of a larger patch which Jason claimed was 
plagarizing his (Jason's) earlier patch, and Jason did not notice the one-line 
correction dealing with the logical error.

Paul's report of the logical error was not bogus, and his one-line correction was 
obviously correct.  Here it is again:

--- fopen_wrappers.c.orig       Mon Feb 26 00:07:31 2001
+++ fopen_wrappers.c    Fri May 18 12:40:54 2001
@@ -147,7 +147,7 @@
        SLS_FETCH();
        
        /* Special case basedir==".": Use script-directory */
-       if ((strcmp(PG(open_basedir), ".") == 0) && 
+       if ((strcmp(basedir), ".") == 0) && 
                SG(request_info).path_translated &&
                *SG(request_info).path_translated
                ) {




---------------------------------------------------------------------------



ATTENTION! Do NOT reply to this email!
To reply, use the web interface found at http://bugs.php.net/?id=10960&edit=2


-- 
PHP Development Mailing List <http://www.php.net/>
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
To contact the list administrators, e-mail: [EMAIL PROTECTED]

Reply via email to