Bug #66048 [Csd->Fbk]: temp. directory is cached during multiple requests

Edit report at https://bugs.php.net/bug.php?id=66048&edit=1 ID: 66048 Updated by: jpauli@php.net Reported by: manuel-php at mausz dot at Summary: temp. directory is cached during multiple requests -Status: Closed +Status: Feedback Type: Bug Package: *Configuration Issues PHP Version: Irrelevant Block user comment: N Private report: N New Comment: Please try using this snapshot: http://snaps.php.net/php5.5-latest.tar.gz For Windows: http://windows.php.net/snapshots/ Fixed in PHP-5.5 and up Previous Comments: ------------------------------------------------------------------------ [2015-05-12 14:17:35] jpauli@php.net Automatic comment on behalf of jpauli Revision: http://git.php.net/?p=php-src.git;a=commit;h=468e37fe027259fd0a61ac3a84cac1336f768005 Log: fix #66048 for PHP7 ------------------------------------------------------------------------ [2015-04-29 19:24:47] php at spam dot lifeforms dot nl The fix resolves the issue completely! First built php7 master from source on Ubuntu 14.04 LTS to reproduce the problem on PHP7: yep! Then built fix-66048: problem resolved! :) I tried to apply the diff to 5.6.8 but wasn't immediately successful as it's quite different (error_clear_last and such), do you think it can be backported? ------------------------------------------------------------------------ [2015-04-29 12:37:08] jpauli@php.net Hello. Can you test https://github.com/jpauli/php-src/tree/fix-66048 please ? This should fix your problem. Thx. ------------------------------------------------------------------------ [2015-04-05 14:43:12] php at spam dot lifeforms dot nl We are affected by this issue on mod_php 5.6.7 on FreeBSD and Ubuntu since we have started to use a separate temporary directory for every separate VirtualHost, such as: php_admin_value sys_temp_dir /home/foo/tmp Unfortunately the sys_get_temp_dir() return value gets mixed up between virtual hosts. It's easy to reproduce, just use Apache with mod_prefork and mod_php5. Configure a different 'sys_temp_dir' value for two virtualhosts, and put a script such as: <?php echo sys_get_temp_dir(); Reload this script on both virtualhosts a few times in two browser tabs, and you will see the value frequently switch from one vhost's sys_temp_dir value to another one's (on both OS). On FreeBSD even /tmp is returned sometimes. It appears that the cause of this mixup is a cache in main/php_open_temporary_file.c: PHPAPI const char* php_get_temporary_directory(TSRMLS_D) { /* Did we determine the temporary directory already? */ if (temporary_directory) { return temporary_directory; } This is supposed to be cleaned by php_shutdown_temporary_directory(). However, that is called at module shutdown and not at request shutdown. Therefore I understand it's liable not to be cleaned between requests, meaning that different VirtualHosts will re-use each other's temporary directory. If in Apache configuration MaxConnectionsPerChild is set to 1, so that Apache processes are never reused, the problem goes away (but this has serious performance implications). However this reinforces the idea that per-request cleanup of the value is not done. In this PR https://github.com/php/php-src/pull/524 the PR author argued php_shutdown_temporary_directory() should be moved to php_request_shutdown(), however jpauli has registered some doubts. For virtual hosting environments it's extremely useful to set the temp directory on a VirtualHost basis, and its malfunction causes issues for instance with Wordpress auto-updating, so it'd really be awesome if this bug could get some love and would like to help if we can. ------------------------------------------------------------------------ [2013-12-30 17:33:45] manuel-php at mausz dot at Just for reference: https://github.com/php/php-src/pull/524 (still open) ------------------------------------------------------------------------ The remainder of the comments for this report are too long. To view the rest of the comments, please view the bug report online at https://bugs.php.net/bug.php?id=66048 -- Edit this bug report at https://bugs.php.net/bug.php?id=66048&edit=1