ID: 26757 Updated by: [EMAIL PROTECTED] Reported By: unknown at simplemachines dot org -Status: Open +Status: Closed Bug Type: Session related Operating System: Windows PHP Version: 4CVS, 5CVS New Comment:
Fixed in CVS. The new default for save_path in upcoming releaess will be the empty string, which causes the temporary directory to be probed. Thanks for your patch. Previous Comments: ------------------------------------------------------------------------ [2004-03-29 09:56:43] unknown at simplemachines dot org After I applied that patch, and changed session.save_path in my php.ini to nothing, it did indeed work without trouble. However, if I renamed php.ini to php_.ini, and tried it... it of course went back to the default /tmp - but if the default is changed to NULL without handling it, crashes will result... My purpose here is to make it easier for people, mainly, to test PHP on their own computers without running into funny errors. That means default values, because some people will never read the manual.... which is why I did it as I did in my patch. Thanks for replying, -[Unknown] ------------------------------------------------------------------------ [2004-03-29 07:25:14] [EMAIL PROTECTED] Please try this patch against latest stable CVS: http://www.php.net/~wez/session-files-fix-4.3.diff I also have an equivalent for PHP 5. This is slightly different from your patch, as it doesn't obliterate non file-based session save paths. ------------------------------------------------------------------------ [2004-03-29 04:58:24] unknown at simplemachines dot org I'm sorry, I just feel that with 47 votes (as of now) and even a provided working patch, a comment from a developer would be nice. Changing the category to "Session related" in the hopes that it not only will be looked at there, but also will receive some notice. Was previously a "Feature/Change Request". Thanks again, -[Unknown] ------------------------------------------------------------------------ [2004-02-17 03:14:45] unknown at simplemachines dot org Maybe someone will read this if I say 4CVS, 5CVS instead of Irrelevant. ------------------------------------------------------------------------ [2004-01-14 00:44:48] unknown at simplemachines dot org A diff for a recent cvs version can be found here: http://unknown.network32.net/php_session_save_path_diff.txt This diff has the following affects: - any call to session_start() or session_save_path() will set the setting to the Windows/*nix temporary directory, unless it's otherwise specified. - if ini_get('session.save_path') is called before this happens, it will return an empty string; afterward, it will return the path used. I have tested this on Windows 2003 Server, in ISAPI/CGI, with and without using user-defined session functions. It should work on Linux/Unix as well, although I haven't had a chance to recompile PHP on any Linux/Unix machine with this patch. It uses the same method to get the path that upload_tmp_dir uses, however. Thank you, -[Unknown] ------------------------------------------------------------------------ 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 http://bugs.php.net/26757 -- Edit this bug report at http://bugs.php.net/?id=26757&edit=1
