diff options
| author | Christoph M. Becker <cmbecker69@gmx.de> | 2019-12-10 18:43:01 +0100 |
|---|---|---|
| committer | Stanislav Malyshev <stas@php.net> | 2019-12-15 13:13:30 -0800 |
| commit | 518a160b65fe1c535dc7e78972ba7428c2a4e197 (patch) | |
| tree | cf21f57b4ebd55237b47b3d8b2ee42687abf77be /win32 | |
| parent | 621598eaa8063a9e76447e07f6f3c30a8baca1e0 (diff) | |
| download | php-git-518a160b65fe1c535dc7e78972ba7428c2a4e197.tar.gz | |
Fix #78943: mail() may release string with refcount==1 twice
Since we need `headers_lc` as well as `headers_trim` in the following,
we do not release the former even if they are the same string, to avoid
complicating the release logic even more.
A new test case is not necessary, since we already have
mail_basic_alt2-win32.phpt and others.
Diffstat (limited to 'win32')
| -rw-r--r-- | win32/sendmail.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/win32/sendmail.c b/win32/sendmail.c index ea693ae1f9..273a3737c6 100644 --- a/win32/sendmail.c +++ b/win32/sendmail.c @@ -208,9 +208,6 @@ PHPAPI int TSendMail(char *host, int *error, char **error_message, /* Create a lowercased header for all the searches so we're finally case * insensitive when searching for a pattern. */ headers_lc = zend_string_tolower(headers_trim); - if (headers_lc == headers_trim) { - zend_string_release_ex(headers_lc, 0); - } } /* Fall back to sendmail_from php.ini setting */ |