Better fix for bug #41655

diff --git a/ext/standard/dir.c b/ext/standard/dir.c
index a26e0c644040678d0c8e5284cf4ec47aabfa0732..8c0f0fb3dc45ab6eb867a48123d5f96668ddbcaf 100644 (file)
--- a/ext/standard/dir.c
+++ b/ext/standard/dir.c
@@ -423,16 +423,15 @@ PHP_FUNCTION(glob)
 #endif
 
        if (PG(open_basedir) && *PG(open_basedir)) {
-               size_t base_len = php_dirname(pattern, strlen(pattern));
-               char pos = pattern[base_len];
-
-               pattern[base_len] = '\0';
-
+               int pattern_len = strlen(pattern);
+               char *basename = estrndup(pattern, pattern_len);
+               
+               php_dirname(basename, pattern_len);
                if (php_check_open_basedir(pattern TSRMLS_CC)) {
+                       efree(basename);
                        RETURN_FALSE;
                }
-
-               pattern[base_len] = pos;
+               efree(basename);
        }
 
        globbuf.gl_offs = 0;

diff --git a/ext/standard/tests/file/bug41655_1.phpt b/ext/standard/tests/file/bug41655_1.phpt
index 9b047bcace5081096c8ba5ad1c4218e9d20eddf7..9338ebd817d0b9137c1359fa39aefb80bca3d2ae 100644 (file)
--- a/ext/standard/tests/file/bug41655_1.phpt
+++ b/ext/standard/tests/file/bug41655_1.phpt
@@ -7,4 +7,4 @@ open_basedir=/tmp
        $a=glob("./*.jpeg");
 ?>
 --EXPECTF--
-Warning: glob() [%s]: open_basedir restriction in effect. File(.) is not within the allowed path(s): (/tmp) in %s on line %d
+Warning: glob(): open_basedir restriction in effect. File(.) is not within the allowed path(s): (/tmp) in %s on line %d