Feature Request #28790 Add php.ini option to disable stat cache

Zend/zend_virtual_cwd.c

@@ -146,6 +146,7 @@ static void cwd_globals_ctor(virtual_cwd_globals *cwd_g) /* {{{ */
 	cwd_g->realpath_cache_size = 0;
 	cwd_g->realpath_cache_size_limit = REALPATH_CACHE_SIZE;
 	cwd_g->realpath_cache_ttl = REALPATH_CACHE_TTL;
+	cwd_g->disable_stat_cache = 0;
 	memset(cwd_g->realpath_cache, 0, sizeof(cwd_g->realpath_cache));
 }
 /* }}} */

Zend/zend_virtual_cwd.h

@@ -222,6 +222,7 @@ typedef struct _virtual_cwd_globals {
 	zend_long                   realpath_cache_size;
 	zend_long                   realpath_cache_size_limit;
 	zend_long                   realpath_cache_ttl;
+	zend_bool                   disable_stat_cache;
 	realpath_cache_bucket *realpath_cache[1024];
 } virtual_cwd_globals;
 

ext/standard/tests/file/bug28790.ini

@@ -0,0 +1 @@
+disable_stat_cache = True

ext/standard/tests/file/bug28790.phpt

@@ -0,0 +1,77 @@
+--TEST--
+Bug #28790: Add php.ini option to disable stat cache
+--CREDITS--
+jnoll <jnoll [at] prim [dot] hu>
+Kevin Lyda <kevin [at] lyda [dot] ie>
+--FILE--
+<?php
+
+    $inifile = __DIR__.DIRECTORY_SEPARATOR.'bug28790.ini';
+    $php = '"'.getenv('TEST_PHP_EXECUTABLE').'"';
+    $phpfile = getenv('TEST_PHP_EXECUTABLE');
+    $impossiblefile = $phpfile.DIRECTORY_SEPARATOR.'bug28790.impossible';
+    $testfile = __DIR__.DIRECTORY_SEPARATOR.'bug28790.file';
+    $codefile = __DIR__.DIRECTORY_SEPARATOR.'bug28790.test.php';
+    $code = <<<EOC
+<?php
+
+function all_the_stats(\$filename, \$message) {
+    if (@lstat(\$filename)) {
+        print("lstat: \$message.\n");
+    }
+    if (@stat(\$filename)) {
+        print("stat: \$message.\n");
+    }
+}
+
+\$testfile = str_replace("\\\\", "/", "$testfile");
+passthru('$php -n -r \'touch("\$testfile");\'');
+all_the_stats("$testfile", "testfile exists");
+passthru('$php -n -r \'unlink("\$testfile");\'');
+all_the_stats("$testfile", "testfile exists (it shouldn't)");
+if (!@stat("$impossiblefile")) {
+    print("stat impossiblefile does not exist.\n");
+}
+all_the_stats("$testfile", "testfile exists (it shouldn't)");
+if (is_file("$phpfile")) {
+    print("is_file(stat): php binary exists.\n");
+}
+all_the_stats("$testfile", "testfile exists (it shouldn't)");
+if (lstat("$phpfile")) {
+    print("lstat: php binary exists.\n");
+}
+all_the_stats("$testfile", "testfile exists (it shouldn't)");
+EOC;
+    $f = fopen($codefile, "w");
+    fwrite($f, $code);
+    fclose($f);
+
+    $code = '"var_dump(php_ini_loaded_file());"';
+
+    // No ini file
+    passthru($php.' -n '.$codefile);
+
+    print("---running with disable_stat_cache = True---\n");
+    // Specified ini file
+    passthru($php.' -c "'.$inifile.'" '.$codefile);
+
+    unlink($codefile);
+
+?>
+--EXPECT--
+lstat: testfile exists.
+stat: testfile exists.
+lstat: testfile exists (it shouldn't).
+stat: testfile exists (it shouldn't).
+stat impossiblefile does not exist.
+lstat: testfile exists (it shouldn't).
+stat: testfile exists (it shouldn't).
+is_file(stat): php binary exists.
+lstat: testfile exists (it shouldn't).
+lstat: php binary exists.
+---running with disable_stat_cache = True---
+lstat: testfile exists.
+stat: testfile exists.
+stat impossiblefile does not exist.
+is_file(stat): php binary exists.
+lstat: php binary exists.

main/main.c

@@ -758,6 +758,7 @@ PHP_INI_BEGIN()
 
 	STD_PHP_INI_ENTRY("realpath_cache_size",	"4096K",	PHP_INI_SYSTEM,		OnUpdateLong,	realpath_cache_size_limit,	virtual_cwd_globals,	cwd_globals)
 	STD_PHP_INI_ENTRY("realpath_cache_ttl",		"120",		PHP_INI_SYSTEM,		OnUpdateLong,	realpath_cache_ttl,			virtual_cwd_globals,	cwd_globals)
+	STD_PHP_INI_BOOLEAN("disable_stat_cache",	"0",		PHP_INI_SYSTEM,		OnUpdateBool,	disable_stat_cache,			virtual_cwd_globals,	cwd_globals)
 
 	STD_PHP_INI_ENTRY("user_ini.filename",		".user.ini",	PHP_INI_SYSTEM,		OnUpdateString,		user_ini_filename,	php_core_globals,		core_globals)
 	STD_PHP_INI_ENTRY("user_ini.cache_ttl",		"300",			PHP_INI_SYSTEM,		OnUpdateLong,		user_ini_cache_ttl,	php_core_globals,		core_globals)

main/streams/streams.c

@@ -1959,7 +1959,7 @@ PHPAPI int _php_stream_stat_path(const char *path, int flags, php_stream_statbuf
 
 	memset(ssb, 0, sizeof(*ssb));
 
-	if (!(flags & PHP_STREAM_URL_STAT_NOCACHE)) {
+	if (!CWDG(disable_stat_cache) && !(flags & PHP_STREAM_URL_STAT_NOCACHE)) {
 		/* Try to hit the cache first */
 		if (flags & PHP_STREAM_URL_STAT_LINK) {
 			if (BG(CurrentLStatFile) && strcmp(path, BG(CurrentLStatFile)) == 0) {
@@ -1978,7 +1978,7 @@ PHPAPI int _php_stream_stat_path(const char *path, int flags, php_stream_statbuf
 	if (wrapper && wrapper->wops->url_stat) {
 		ret = wrapper->wops->url_stat(wrapper, path_to_open, flags, ssb, context);
 		if (ret == 0) {
-		        if (!(flags & PHP_STREAM_URL_STAT_NOCACHE)) {
+			if (!CWDG(disable_stat_cache) && !(flags & PHP_STREAM_URL_STAT_NOCACHE)) {
 				/* Drop into cache */
 				if (flags & PHP_STREAM_URL_STAT_LINK) {
 					if (BG(CurrentLStatFile)) {

php.ini-development

@@ -356,6 +356,12 @@ disable_classes =
 ; http://php.net/realpath-cache-ttl
 ;realpath_cache_ttl = 120
 
+; Allows user to disable the "stat cache." By default php remembers the
+; results of the last call to stat() and lstat() and functions that call
+; it underneath.  Setting this to True disables that behaviour.
+; http://php.net/disable-stat-cache
+;disable_stat_cache = False
+
 ; Enables or disables the circular reference collector.
 ; http://php.net/zend.enable-gc
 zend.enable_gc = On

php.ini-production

@@ -356,6 +356,12 @@ disable_classes =
 ; http://php.net/realpath-cache-ttl
 ;realpath_cache_ttl = 120
 
+; Allows user to disable the "stat cache." By default php remembers the
+; results of the last call to stat() and lstat() and functions that call
+; it underneath.  Setting this to True disables that behaviour.
+; http://php.net/disable-stat-cache
+;disable_stat_cache = False
+
 ; Enables or disables the circular reference collector.
 ; http://php.net/zend.enable-gc
 zend.enable_gc = On