| Commit message (Collapse) | Author | Age | Files | Lines |
|\
| |
| |
| |
| | |
* PHP-7.2:
Fix #68180: iconv_mime_decode can return extra characters in a header
|
| |\
| | |
| | |
| | |
| | | |
* PHP-7.1:
Fix #68180: iconv_mime_decode can return extra characters in a header
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Basically, the algorithm to append a converted string to an existing
`smart_str` works by increasing the `smart_str` buffer, to let `iconv`
convert characters until there is no more space, to set the new length
of the `smart_str` and to repeat until there is no more input.
Formerly, the new length calculation has been wrong, though, since we
would have to take the old `out_len` into account (`buf_growth -
old_out_len - out_len`). However, since there is no need to take the
old `out_len` into account when increasing the `smart_str` buffer, we
can simplify the fix, avoiding an additional variable.
|
|\ \ \
| |/ /
| | |
| | |
| | | |
* PHP-7.2:
Fix #60494: iconv_mime_decode does ignore special characters
|
| |\ \
| | |/
| | |
| | |
| | | |
* PHP-7.1:
Fix #60494: iconv_mime_decode does ignore special characters
|
| | |
| | |
| | |
| | |
| | |
| | | |
We must not ignore erroneous characters in mime headers, but rather let
iconv_mime_decode() fail in this case, issuing the usual notice
regarding illegal characters.
|
|\ \ \
| |/ /
| | |
| | |
| | | |
* PHP-7.2:
Fix #63839: iconv_mime_decode_headers function is skipping headers
|
| |\ \
| | |/
| | |
| | |
| | | |
* PHP-7.1:
Fix #63839: iconv_mime_decode_headers function is skipping headers
|
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
We have to cater to the possibility that `=?` is not the start of an
encoded-word, but rather a literal `=?`. If a line break is found
while we're still looking for the charset, we can safely assume that
it's a literal `=?`, and act accordingly.
|
|\ \ \
| |/ /
| | |
| | |
| | | |
* PHP-7.2:
Fix #55146: iconv_mime_decode_headers() skips some headers
|
| |\ \
| | |/
| | |
| | |
| | | |
* PHP-7.1:
Fix #55146: iconv_mime_decode_headers() skips some headers
|
| | |
| | |
| | |
| | |
| | |
| | | |
If we're expecting the start of an encoded word (`=?`), but instead of
the question mark get a line break (CR or LF), we must not append it to
the `pretval`.
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
The minimum length of an encoded-word is actually the pure encoding
overhead plus the length of the `output-charset` plus the minimum unit
of encoded text, which is 4 for B-encoding and (for simplicity) 3 for
Q-encoding. We also cater to the possibility that we need further
encoded words, which would be split by the `line-break-chars` followed
by a space character. Obviously, the former `out_charset_len + 12` is
too simplistic and wrong in the given case (where the magic number
would be 13).
These simplifications are somewhat wasteful, but iconv_mime_encode()
with Q-encoding is wasteful anyway (see bug 66828[1]), and the proper
solution to convert the whole input to the desired output charset
upfront, and applying the encoding afterwards appears too much a change
for the stable releases.
[1] <https://bugs.php.net/66828>
|
| | |
| | |
| | |
| | |
| | |
| | | |
We work around this peculiarity of libxml by using xmlNodeSetContent(),
which does not exhibit this behavior. This also saves us from manually
calculating the string length.
|
|\ \ \
| |/ /
| | |
| | |
| | | |
* PHP-7.2:
Simplify regression test
|
| |\ \
| | |/
| | |
| | |
| | | |
* PHP-7.1:
Simplify regression test
|
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
There's no need to actually try to trigger an out-of-memory condition
to proof the leak; instead we can simply rely on the Zend MM to report
the memory leaks in debug mode (at least on Linux). Therefore we
simplify the regression test, which also makes it run much faster.
|
|\ \ \
| |/ /
| | |
| | |
| | | |
* PHP-7.2:
Fix #68825: Exception in DirectoryIterator::getLinkTarget()
|
| |\ \
| | |/
| | |
| | |
| | | |
* PHP-7.1:
Fix #68825: Exception in DirectoryIterator::getLinkTarget()
|
| | |
| | |
| | |
| | |
| | |
| | | |
intern->file_name may not have been properly set when
DirectoryIterator::getLinkTarget() is called, so we make sure it is
before using it.
|
|\ \ \
| |/ /
| | |
| | |
| | | |
* PHP-7.2:
Increase memory_limit to prevent test failures
|
| |\ \
| | |/
| | |
| | |
| | | |
* PHP-7.1:
Increase memory_limit to prevent test failures
|
| | | |
|
|\ \ \
| |/ /
| | |
| | |
| | | |
* PHP-7.2:
Fix #76778: array_reduce leaks memory if callback throws exception
|
| |\ \
| | |/
| | |
| | |
| | | |
* PHP-7.1:
Fix #76778: array_reduce leaks memory if callback throws exception
|
| | |
| | |
| | |
| | | |
We have to release the result variable in the error case, too.
|
|\ \ \
| |/ /
| | |
| | |
| | | |
* PHP-7.2:
Fixed bug #76777 and added test
|
| |\ \
| | |/
| | |
| | |
| | | |
* PHP-7.1:
Fixed bug #76777 and added test
|
| | |
| | |
| | |
| | | |
Set undefined values to null rather than undefined.
|
| | |
| | |
| | |
| | | |
This fixes a Clang warning.
|
| | |
| | |
| | |
| | | |
Encode static variable offset into opline->extended_value.
|
| | | |
|
|\ \ \
| |/ / |
|
| |\ \
| | |/ |
|
| | | |
|
|\ \ \
| |/ /
| | |
| | |
| | |
| | | |
* PHP-7.2:
Add regression test for bug #68175
Fix #68175: RegexIterator pregFlags are NULL instead of 0
|
| |\ \
| | |/
| | |
| | |
| | |
| | | |
* PHP-7.1:
Add regression test for bug #68175
Fix #68175: RegexIterator pregFlags are NULL instead of 0
|
| | | |
|
| | | |
|
| | |
| | |
| | |
| | |
| | | |
Original repro. It's unwieldy, but it's otherwise hard to trigger
the wrong buffer handling behavior.
|
| | | |
|
| | |
| | |
| | |
| | | |
(cherry picked from commit 8b3174f256147a1708821621a8cbe2b257fca737)
|
| | |
| | |
| | |
| | | |
(cherry picked from commit 083285f22a74989689f97d1d53476e7eaec35acc)
|
| | |
| | |
| | |
| | | |
(cherry picked from commit 36f05a80d7cf11fffb827c7f0b6c8e73d3846e8e)
|
| | |
| | |
| | |
| | | |
(cherry picked from commit ef9ed19ec7f141311feea1d42467f5773cfc09bc)
|
| | | |
|
| | | |
|
| | | |
|
| | | |
|
|\ \ \
| |/ /
| | |
| | |
| | |
| | | |
* PHP-7.2:
Update NEWS
Fixed bug #76747 (Opcache treats path containing "test.pharma.tld" as a phar file)
|