php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Bug #43962 backslashes in session save path are not escaped in error message
Submitted: 2008-01-29 08:05 UTC Modified: 2009-03-10 01:00 UTC
From: php-qa at sebatianmendel dot de Assigned: pajoye (profile)
Status: No Feedback Package: Session related
PHP Version: 5CVS-2008-01-29 (snap) OS: win32 only
Private report: No CVE-ID: None
Have you experienced this issue?
Rate the importance of this bug to you:

 [2008-01-29 08:05 UTC] php-qa at sebatianmendel dot de
Description:
------------
re-submitting this bug, cause: "You can not comment bogus reports or change their status."


[19 Dec 2007 7:02am UTC] php-qa at sebastianmendel dot de

backslashes in the session.save_path are not properly escaped when
printed out in an error message


[29 Jan 12:10am UTC] tony2001@php.net

Printing "C:\" doesn't require any escaping.
It's you who needs to escape the slashes when you enter them in your
php.ini.


[2008-01-29] php-qa at sebastianmendel dot de

since when? and where is this written?

if you are right than at least this is a documentation bug

but i do not think so

from php.ini-recommended:

[...]
; Windows: "\path1;\path2"
;include_path = ".;c:\php\includes"
[...]

and \ in session save path works, it is just the error message which is wrong! and using 'c:\temp' always worked and still works as expected.

the problem is not the error displayed, the problem is in the displayed error. (the error was a local problem already fixed)

Reproduce code:
---------------
php.ini:

session.save_path = "C:\temp"

continue a session without valid session file


Expected result:
----------------
session_start() : open(C:\temp\sess_ifj79uququ2tu2gnhu8ol2cs81, O_RDWR)
failed: Invalid argument (22)


Actual result:
--------------
session_start() : open(C: emp\sess_ifj79uququ2tu2gnhu8ol2cs81, O_RDWR)
failed: Invalid argument (22)


Patches

Add a Patch

Pull Requests

Add a Pull Request

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2008-10-26 23:25 UTC] jani@php.net
Assigned to the windows port maintainer.
 [2009-03-02 20:23 UTC] pajoye@php.net
Please try using this CVS snapshot:

  http://snaps.php.net/php5.3-latest.tar.gz
 
For Windows:

  http://windows.php.net/snapshots/


 [2009-03-10 01:00 UTC] php-bugs at lists dot php dot net
No feedback was provided for this bug for over a week, so it is
being suspended automatically. If you are able to provide the
information that was originally requested, please do so and change
the status of the bug back to "Open".
 
PHP Copyright © 2001-2023 The PHP Group
All rights reserved.
Last updated: Fri Feb 03 13:05:52 2023 UTC