Skip to content

Commit 2ddb560

Browse files
author
Julien Pauli
committed
Merge branch 'PHP-5.5' into PHP-5.6
* PHP-5.5: Fix for #66048
2 parents e1cb22a + c117548 commit 2ddb560

File tree

2 files changed

+11
-11
lines changed

2 files changed

+11
-11
lines changed

main/main.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1868,7 +1868,7 @@ void php_request_shutdown(void *dummy)
18681868
}
18691869
} zend_end_try();
18701870

1871-
/* 7.5 free last error information */
1871+
/* 7.5 free last error information and temp dir */
18721872
if (PG(last_error_message)) {
18731873
free(PG(last_error_message));
18741874
PG(last_error_message) = NULL;
@@ -1877,6 +1877,7 @@ void php_request_shutdown(void *dummy)
18771877
free(PG(last_error_file));
18781878
PG(last_error_file) = NULL;
18791879
}
1880+
php_shutdown_temporary_directory();
18801881

18811882
/* 7. Shutdown scanner/executor/compiler and restore ini entries */
18821883
zend_deactivate(TSRMLS_C);
@@ -2475,7 +2476,6 @@ void php_module_shutdown(TSRMLS_D)
24752476
#endif
24762477

24772478
php_output_shutdown();
2478-
php_shutdown_temporary_directory();
24792479

24802480
module_initialized = 0;
24812481

main/php_open_temporary_file.c

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -181,7 +181,7 @@ static char* temporary_directory;
181181
PHPAPI void php_shutdown_temporary_directory(void)
182182
{
183183
if (temporary_directory) {
184-
free(temporary_directory);
184+
efree(temporary_directory);
185185
temporary_directory = NULL;
186186
}
187187
}
@@ -202,10 +202,10 @@ PHPAPI const char* php_get_temporary_directory(TSRMLS_D)
202202
if (sys_temp_dir) {
203203
int len = strlen(sys_temp_dir);
204204
if (len >= 2 && sys_temp_dir[len - 1] == DEFAULT_SLASH) {
205-
temporary_directory = zend_strndup(sys_temp_dir, len - 1);
205+
temporary_directory = estrndup(sys_temp_dir, len - 1);
206206
return temporary_directory;
207207
} else if (len >= 1 && sys_temp_dir[len - 1] != DEFAULT_SLASH) {
208-
temporary_directory = zend_strndup(sys_temp_dir, len);
208+
temporary_directory = estrndup(sys_temp_dir, len);
209209
return temporary_directory;
210210
}
211211
}
@@ -222,9 +222,9 @@ PHPAPI const char* php_get_temporary_directory(TSRMLS_D)
222222
DWORD len = GetTempPath(sizeof(sTemp),sTemp);
223223
assert(0 < len); /* should *never* fail! */
224224
if (sTemp[len - 1] == DEFAULT_SLASH) {
225-
temporary_directory = zend_strndup(sTemp, len - 1);
225+
temporary_directory = estrndup(sTemp, len - 1);
226226
} else {
227-
temporary_directory = zend_strndup(sTemp, len);
227+
temporary_directory = estrndup(sTemp, len);
228228
}
229229
return temporary_directory;
230230
}
@@ -236,9 +236,9 @@ PHPAPI const char* php_get_temporary_directory(TSRMLS_D)
236236
int len = strlen(s);
237237

238238
if (s[len - 1] == DEFAULT_SLASH) {
239-
temporary_directory = zend_strndup(s, len - 1);
239+
temporary_directory = estrndup(s, len - 1);
240240
} else {
241-
temporary_directory = zend_strndup(s, len);
241+
temporary_directory = estrndup(s, len);
242242
}
243243

244244
return temporary_directory;
@@ -247,12 +247,12 @@ PHPAPI const char* php_get_temporary_directory(TSRMLS_D)
247247
#ifdef P_tmpdir
248248
/* Use the standard default temporary directory. */
249249
if (P_tmpdir) {
250-
temporary_directory = strdup(P_tmpdir);
250+
temporary_directory = estrdup(P_tmpdir);
251251
return temporary_directory;
252252
}
253253
#endif
254254
/* Shouldn't ever(!) end up here ... last ditch default. */
255-
temporary_directory = strdup("/tmp");
255+
temporary_directory = estrndup("/tmp", sizeof("/tmp"));
256256
return temporary_directory;
257257
#endif
258258
}

0 commit comments

Comments
 (0)