Skip to content

Commit 3069ad8

Browse files
weltlingweltling
committed
Fixed bug #75063
1 parent 207105b commit 3069ad8

File tree

6 files changed

+196
-4
lines changed

6 files changed

+196
-4
lines changed

‎Zend/zend.c

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -660,6 +660,10 @@ int zend_startup(zend_utility_functions *utility_functions, char **extensions) /
660660
extern zend_php_scanner_globals language_scanner_globals;
661661
#endif
662662

663+
#ifdef ZEND_WIN32
664+
php_win32_cp_set_by_id(65001);
665+
#endif
666+
663667
start_memory_manager();
664668

665669
virtual_cwd_startup(); /* Could use shutdown to free the main cwd but it would just slow it down for CGI */

‎Zend/zend_virtual_cwd.c

Lines changed: 13 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -445,11 +445,15 @@ static void cwd_globals_dtor(virtual_cwd_globals *cwd_g) /* {{{ */
445445
}
446446
/* }}} */
447447

448-
CWD_API void virtual_cwd_startup(void) /* {{{ */
448+
void virtual_cwd_main_cwd_init(uint8_t reinit) /* {{{ */
449449
{
450450
char cwd[MAXPATHLEN];
451451
char *result;
452452

453+
if (reinit) {
454+
free(main_cwd_state.cwd);
455+
}
456+
453457
#ifdef NETWARE
454458
result = getcwdpath(cwd, NULL, 1);
455459
if(result)
@@ -461,10 +465,10 @@ CWD_API void virtual_cwd_startup(void) /* {{{ */
461465
++c;
462466
}
463467
}
464-
#else
465-
#ifdef ZEND_WIN32
468+
#elif defined(ZEND_WIN32)
466469
ZeroMemory(&cwd, sizeof(cwd));
467-
#endif
470+
result = php_win32_ioutil_getcwd(cwd, sizeof(cwd));
471+
#else
468472
result = getcwd(cwd, sizeof(cwd));
469473
#endif
470474
if (!result) {
@@ -478,7 +482,12 @@ CWD_API void virtual_cwd_startup(void) /* {{{ */
478482
}
479483
#endif
480484
main_cwd_state.cwd = strdup(cwd);
485+
}
486+
/* }}} */
481487

488+
CWD_API void virtual_cwd_startup(void) /* {{{ */
489+
{
490+
virtual_cwd_main_cwd_init(0);
482491
#ifdef ZTS
483492
ts_allocate_id(&cwd_globals_id, sizeof(virtual_cwd_globals), (ts_allocate_ctor) cwd_globals_ctor, (ts_allocate_dtor) cwd_globals_dtor);
484493
#else

‎Zend/zend_virtual_cwd.h

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -251,6 +251,10 @@ CWD_API zend_long realpath_cache_size(void);
251251
CWD_API zend_long realpath_cache_max_buckets(void);
252252
CWD_API realpath_cache_bucket** realpath_cache_get_buckets(void);
253253

254+
#ifdef CWD_EXPORTS
255+
extern void virtual_cwd_main_cwd_init(uint8_t);
256+
#endif
257+
254258
/* The actual macros to be used in programs using TSRM
255259
* If the program defines VIRTUAL_DIR it will use the
256260
* virtual_* functions

‎ext/standard/tests/file/windows_mb_path/bug75063_cp1251.phpt

Lines changed: 80 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,80 @@
1+
--TEST--
2+
Bug #75063 Many filesystem-related functions do not work with multibyte file names, cp1251
3+
--SKIPIF--
4+
<?php
5+
include dirname(__FILE__) . DIRECTORY_SEPARATOR . "util.inc";
6+
7+
skip_if_not_win();
8+
if (getenv("SKIP_SLOW_TESTS")) die("skip slow test");
9+
skip_if_no_required_exts();
10+
11+
?>
12+
--INI--
13+
default_charset=cp1251
14+
--FILE--
15+
<?php
16+
17+
/* This file is in cp1251. */
18+
19+
include dirname(__FILE__) . DIRECTORY_SEPARATOR . "util.inc";
20+
21+
$dir_basename = "ňĺńň";
22+
$prefix = dirname(__FILE__) . DIRECTORY_SEPARATOR . "bug75063-cp1251";
23+
$d0 = $prefix . DIRECTORY_SEPARATOR . $dir_basename;
24+
25+
mkdir($prefix);
26+
create_verify_dir($prefix, $dir_basename, 1251);
27+
28+
var_dump(get_basename_with_cp($d0, 1251, false));
29+
30+
$old_cwd = getcwd();
31+
var_dump(chdir($d0));
32+
33+
$code = <<<CODE
34+
<?php
35+
36+
foreach(["test", "ňŕńň"] as \$fn) {
37+
file_put_contents("\$fn.txt", "");
38+
}
39+
40+
var_dump(getcwd());
41+
if (\$dh = opendir(getcwd())) {
42+
while ((\$file = readdir(\$dh)) !== false) {
43+
if ("." == \$file || ".." == \$file) continue;
44+
var_dump(\$file);
45+
}
46+
closedir(\$dh);
47+
}
48+
CODE;
49+
$code_fn = "code.php";
50+
file_put_contents($code_fn, $code);
51+
52+
print(shell_exec(getenv('TEST_PHP_EXECUTABLE') . " -n -d default_charset=cp1251 -f code.php"));
53+
54+
chdir($old_cwd);
55+
56+
/* --CLEAN-- section were the right place, but it won't accept default_charset ATM, it seems. */
57+
$dir_basename = "ňĺńň";
58+
$prefix = dirname(__FILE__) . DIRECTORY_SEPARATOR . "bug75063-cp1251";
59+
$d0 = $prefix . DIRECTORY_SEPARATOR . $dir_basename;
60+
61+
$obj = scandir($d0);
62+
foreach ($obj as $file) {
63+
if ("." == $file || ".." == $file) continue;
64+
unlink($d0 . DIRECTORY_SEPARATOR . $file);
65+
}
66+
67+
rmdir($d0);
68+
rmdir($prefix);
69+
?>
70+
===DONE===
71+
72+
--EXPECTF--
73+
string(4) "ňĺńň"
74+
bool(true)
75+
string(%d) "%sbug75063-cp1251%eňĺńň"
76+
string(8) "code.php"
77+
string(8) "test.txt"
78+
string(8) "ňŕńň.txt"
79+
===DONE===
80+

‎ext/standard/tests/file/windows_mb_path/bug75063_utf8.phpt

Lines changed: 81 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,81 @@
1+
--TEST--
2+
Bug #75063 Many filesystem-related functions do not work with multibyte file names, UTF-8
3+
--SKIPIF--
4+
<?php
5+
include dirname(__FILE__) . DIRECTORY_SEPARATOR . "util.inc";
6+
7+
skip_if_not_win();
8+
if (getenv("SKIP_SLOW_TESTS")) die("skip slow test");
9+
skip_if_no_required_exts();
10+
11+
?>
12+
--FILE--
13+
<?php
14+
15+
/* This file is in UTF-8. */
16+
17+
include dirname(__FILE__) . DIRECTORY_SEPARATOR . "util.inc";
18+
19+
$dir_basename = "тест";
20+
$prefix = dirname(__FILE__) . DIRECTORY_SEPARATOR . "bug75063-utf8";
21+
$d0 = $prefix . DIRECTORY_SEPARATOR . $dir_basename;
22+
23+
mkdir($prefix);
24+
create_verify_dir($prefix, $dir_basename);
25+
26+
var_dump(get_basename_with_cp($d0, 65001, false));
27+
28+
$old_cwd = getcwd();
29+
var_dump(chdir($d0));
30+
31+
$code = <<<CODE
32+
<?php
33+
34+
foreach(["test", "таст"] as \$fn) {
35+
file_put_contents("\$fn.txt", "");
36+
}
37+
38+
var_dump(getcwd());
39+
if (\$dh = opendir(getcwd())) {
40+
while ((\$file = readdir(\$dh)) !== false) {
41+
if ("." == \$file || ".." == \$file) continue;
42+
var_dump(\$file);
43+
}
44+
closedir(\$dh);
45+
}
46+
CODE;
47+
$code_fn = "code.php";
48+
file_put_contents($code_fn, $code);
49+
50+
print(shell_exec(getenv('TEST_PHP_EXECUTABLE') . " -nf code.php"));
51+
52+
chdir($old_cwd);
53+
54+
?>
55+
===DONE===
56+
--CLEAN--
57+
<?php
58+
$dir_basename = "тест";
59+
$prefix = dirname(__FILE__) . DIRECTORY_SEPARATOR . "bug75063-utf8";
60+
$d0 = $prefix . DIRECTORY_SEPARATOR . $dir_basename;
61+
62+
$obj = scandir($d0);
63+
foreach ($obj as $file) {
64+
if ("." == $file || ".." == $file) continue;
65+
unlink($d0 . DIRECTORY_SEPARATOR . $file);
66+
}
67+
68+
rmdir($d0);
69+
rmdir($prefix);
70+
71+
?>
72+
73+
--EXPECTF--
74+
string(8) "тест"
75+
bool(true)
76+
string(%d) "%sbug75063-utf8%eтест"
77+
string(8) "code.php"
78+
string(8) "test.txt"
79+
string(12) "таст.txt"
80+
===DONE===
81+

‎main/main.c

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2221,6 +2221,20 @@ int php_module_startup(sapi_module_struct *sf, zend_module_entry *additional_mod
22212221
/* Register Zend ini entries */
22222222
zend_register_standard_ini_entries();
22232223

2224+
#ifdef ZEND_WIN32
2225+
/* Until the current ini values was setup, the current cp is 65001.
2226+
If the actual ini vaues are different, some stuff needs to be updated.
2227+
It concerns at least main_cwd_state and there might be more. As we're
2228+
still in the startup phase, lets use the chance and reinit the relevant
2229+
item according to the current codepage. Still, if ini_set() is used
2230+
later on, a more intelligent way to update such stuff is needed.
2231+
Startup/shutdown routines could involve touching globals and thus
2232+
can't always be used on demand. */
2233+
if (!php_win32_cp_use_unicode()) {
2234+
virtual_cwd_main_cwd_init(1);
2235+
}
2236+
#endif
2237+
22242238
/* Disable realpath cache if an open_basedir is set */
22252239
if (PG(open_basedir) && *PG(open_basedir)) {
22262240
CWDG(realpath_cache_size_limit) = 0;

0 commit comments

Comments
 (0)