Edit report at http://bugs.php.net/bug.php?id=48465&edit=1
ID: 48465
Updated by: paj...@php.net
Reported by: marcel dot esser at gmail dot com
Summary: sys_get_temp_dir() possibly inconsistent when using
TMPDIR
-Status: Assigned
+Status: Closed
Type: Bug
Package: Scripting Engine problem
Operating System: *
PHP Version: 5.*, 6CVS (2009-06-04)
Assigned To: pajoye
Block user comment: N
Private report: N
Previous Comments:
------------------------------------------------------------------------
[2011-03-28 18:43:55] paj...@php.net
Automatic comment from SVN on behalf of pajoye
Revision: http://svn.php.net/viewvc/?view=revision&revision=309792
Log: - Fixed bug #48465 (sys_get_temp_dir() possibly inconsistent,
windows fix
------------------------------------------------------------------------
[2011-03-28 15:35:33] paj...@php.net
Not fixed on windows
------------------------------------------------------------------------
[2009-06-24 12:21:54] il...@php.net
This bug has been fixed in CVS.
Snapshots of the sources are packaged every three hours; this change
will be in the next snapshot. You can grab the snapshot at
http://snaps.php.net/.
Thank you for the report, and for helping us make PHP better.
------------------------------------------------------------------------
[2009-06-04 11:57:05] j...@php.net
It's not MacOSX specific, you can always set TMPDIR whatever you want.
------------------------------------------------------------------------
[2009-06-04 06:23:14] marcel dot esser at gmail dot com
Alternative patch:
--- php52/php5/main/php_open_temporary_file.c 2009-06-03
13:42:44.000000000 -0400
+++ /Users/marcelesser/php_open_temporary_file.c 2009-06-04
02:19:29.000000000 -0400
@@ -200,9 +200,18 @@
{
char* s = getenv("TMPDIR");
if (s) {
- temporary_directory = strdup(s);
+ int last_char_idx = strlen(s) - 1;
+
+ /* on some systems (notably mac os x) TMPDIR has a
DIRECTORY_SEPARATOR appended */
+ if(s[last_char_idx] == DEFAULT_SLASH) {
+ temporary_directory =
tsrm_strndup(s,last_char_idx);
+ } else {
+ temporary_directory = strdup(s);
+ }
+
return temporary_directory;
}
+
}
#ifdef P_tmpdir
/* Use the standard default temporary directory. */
------------------------------------------------------------------------
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/bug.php?id=48465
--
Edit this bug report at http://bugs.php.net/bug.php?id=48465&edit=1
