diff options
| author | Christoph M. Becker <cmbecker69@gmx.de> | 2020-06-29 16:10:33 +0200 |
|---|---|---|
| committer | Christoph M. Becker <cmbecker69@gmx.de> | 2020-06-29 17:45:28 +0200 |
| commit | 816b4c1235d70b1b83d26c415f044fc04a48875f (patch) | |
| tree | 6a4eb0c0ab36485fd281512a534a05f13f124eae /main | |
| parent | 43cd3f68143eb6b35e19605855dac45db4007fc1 (diff) | |
| download | php-git-816b4c1235d70b1b83d26c415f044fc04a48875f.tar.gz | |
Fix #79756: finfo_file crash (FILEINFO_MIME)
If `ctime` or `asctime` return `NULL`, we must not attempt to copy the
buffer, but rather return `NULL` as well.
Diffstat (limited to 'main')
| -rw-r--r-- | main/reentrancy.c | 14 |
1 files changed, 10 insertions, 4 deletions
diff --git a/main/reentrancy.c b/main/reentrancy.c index 213e82bd8c..6699817510 100644 --- a/main/reentrancy.c +++ b/main/reentrancy.c @@ -187,11 +187,14 @@ PHPAPI char *php_ctime_r(const time_t *clock, char *buf) local_lock(CTIME_R); tmp = ctime(clock); - strcpy(buf, tmp); + if (tmp) { + strcpy(buf, tmp); + tmp = buf; + } local_unlock(CTIME_R); - return buf; + return tmp; } #endif @@ -205,11 +208,14 @@ PHPAPI char *php_asctime_r(const struct tm *tm, char *buf) local_lock(ASCTIME_R); tmp = asctime(tm); - strcpy(buf, tmp); + if (tmp) { + strcpy(buf, tmp); + tmp = buf; + } local_unlock(ASCTIME_R); - return buf; + return tmp; } #endif |