Merge branch 'PHP-7.4'

* PHP-7.4:
  Fixed bug #79011
  Fix memory leaks in mysqlnd debug functionality

Fixed bug #79011

auth_plugin_data_len here is 21, including the trailing null byte.
Directly use SCRAMBLE_LENGTH instead. Also add a sanity check that
the provided scramble is long enough.

Fix memory leaks in mysqlnd debug functionality

Merge branch 'PHP-7.4'

* PHP-7.4:
  Make test independent of online XSD schema
  Yet another check for php_strip_tags_ex()

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Make test independent of online XSD schema
  Yet another check for php_strip_tags_ex()

Make test independent of online XSD schema

The test still needs to access <http://www.w3.org/2009/01/xml.xsd>, but
at least we no longer depend on <http://x-road.eu/xsd/x-road.xsd>,
which may be moved again.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Add ZipArchive::CM_LZMA2 constant (since libzip 1.6.0)

Add ZipArchive::CM_LZMA2 constant (since libzip 1.6.0)

Merge branch 'PHP-7.4'

* PHP-7.4:
  Yet another check for php_strip_tags_ex()

Yet another check for php_strip_tags_ex()

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Yet another check for php_strip_tags_ex()

Yet another check for php_strip_tags_ex()

Merge branch 'PHP-7.4'

* PHP-7.4:
  Don't use CRLF when generating diffs

Don't use CRLF when generating diffs

Also use zend_memnrstr in mbfl_strpos

Base mbfl_strpos on zend_memnstr

The same algorithm is also used by zend_memnstr, but it also has
a fast-path for short strings / needles, where a more naive
search performs better.

Extract calculation of offset from pointer

Handle mb_str(r)ipos offset consistently as well

Move offset error checking into mbfl_strpos

This avoids calculating the full length only in order to validate
the offset, as mbfl_strpos needs to find the offset internally
anyway.

Add #defines for mbfl_strpos error conditions

Allow variadic arguments to replace non-variadic ones

Any number of arguments can be replaced by a variadic one, so
long as the variadic argument is compatible (in the sense of
contravariance) with the subsumed arguments.

In particular this means that function(...$args) becomes a
near-universal signature: It is compatible with any function
signature that does not accept parameters by-reference.

This also fixes bug #70839, which describes a special case.

Closes GH-5059.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix bug76348.phpt

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix bug76348.phpt

Fix bug76348.phpt

Adjust for URL change in XSD file and mark as online test.

Is it possible for use to store http://x-road.eu/xsd/xroad.xsd
locally instead? Do relative file system paths work here?
I'm not familiar with this.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fixed bug #78902
  Add unit test for bug #78902

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fixed bug #78902
  Add unit test for bug #78902

Fixed bug #78902

Add unit test for bug #78902

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fixed bug #79151

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fixed bug #79151

Fixed bug #79151

Make sure we also NULL out next/prev of the removed element on
pop/shift. This only matter is that element is still being referenced
by an iterator.

Apply custom format/length modifier removal to spprintf

This mimicks the changes made to the custom snprintf implementation
by removing the custom 'v' format and custom 'I' length modifier
from the spprintf implementation.

Closes GH-5108

Revert "Merge branch 'PHP-7.4'"

This reverts commit 046dcfb531e242d36a7af2942b9b148290c3c7fe, due to
segfaults on Travis.  This needs to be investigated.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fixed bug #79155

Fixed bug #79155

Make sure we only unset the NULLABLE flag temporarily for class
resolution, as the same type may be compiled multiple types.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Enable UBSan in addition to ASan

Enable UBSan in addition to ASan

UBSan is a useful tool, so we enable it for `--enable-sanitizer` in
addition to ASan.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Make MSVCRT memory leak checking usable for the test suite

Make MSVCRT memory leak checking usable for the test suite

While basic support for MSVCRT debugging has been added long
ago[1], the leak checking is not usable for the test suite, because we
are no longer calling `xmlCleanupParser()` on RSHUTDOWN of
ext/libxml[2], and therefore a few bogus leaks are reported whenever
ext/libxml is unloaded.

We therefore ignore memory leaks for this case.  We introduce
`ZEND_IGNORE_LEAKS_BEGIN()` and `ZEND_IGNORE_LEAKS_END()` to keep
those ignores better readable, and also because these *might* be
useful for other leak checkers as well.

We also explicitly free the `zend_handlers_table` and the `p5s` to
avoid spurious leak reports.

[1] <http://git.php.net/?p=php-src.git;a=commit;h=d756e1db2324c1f4ab6f9b52e329959ce6a02bc3>
[2] <http://git.php.net/?p=php-src.git;a=commit;h=8742276eb3905eb97a585417000c7b8df85006d4>

Merge branch 'PHP-7.4'

* PHP-7.4:
  More checks for php_strip_tags_ex

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  More checks for php_strip_tags_ex

Merge branch 'PHP-7.2' into PHP-7.3

* PHP-7.2:
  More checks for php_strip_tags_ex

More checks for php_strip_tags_ex

Add ValueError for invalid mode in count()

Closes GH-5106

Add UPGRADING notes [skip ci]

Add ReflectionProperty::getDefaultValue and ReflectionProperty::hasDefaultValue

Fix #78649 Provide ICU RELATIVE_ constants

Closes GH-5105

Merge branch 'PHP-7.4'

* PHP-7.4:
  Don't leak encoding_str

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Don't leak encoding_str

Don't leak encoding_str

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix #79154: mb_convert_encoding() can modify $from_encoding

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #79154: mb_convert_encoding() can modify $from_encoding

Fix #79154: mb_convert_encoding() can modify $from_encoding

We must not modify arrays passed by value.

Fix test expectation

As of PHP 8.0.0, array to string conversion raises a warning.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix #79149: SEGV in mb_convert_encoding with non-string encodings

Fix #79149: SEGV in mb_convert_encoding with non-string encodings

We must not assume that `hash_entry` `IS_STRING`, but rather use
`encoding_str` which is guaranteed to be.

Merge branch 'PHP-7.4'

* PHP-7.4:
  next will be 7.2.28

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  next will be 7.2.28

Merge branch 'PHP-7.2' into PHP-7.3

* PHP-7.2:
  next will be 7.2.28

next will be 7.2.28

Use smaller batch size in run-tests.php when appropriate

- When valgrind is used, communication overhead is relatively small,
  so just use a batch size of 1.
- If this is running a small enough number of tests,
  reduce the batch size to give batches to more workers.
  (Previously, if there were 90 tests and -j8,
  only 3 of 8 workers would get a batch of size 32 or less.
  After this change, the batch size is 12 or less)

Closes GH-5098

Don't start unnecessary processes with run-tests.php -j

If there's only 2 files to test, then only start 2 workers instead of N.

If there's only 1 file, then avoid parallelism entirely.
A separate option such as `--force-parallel` could be added
if this turns out to be something developers would want to do
when debugging test failures.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Avoid some @count() suppressions in run-tests.php

Avoid some @count() suppressions in run-tests.php

Merge branch 'PHP-7.4'

* PHP-7.4:
  Add sleep in FPM reload test

Add sleep in FPM reload test

Explicitly mark the point where we have to wait.

Prefer using declaring class rather than direct parent in error

Point to the class that actually declares the property, which is
not necessarily the same as the direct parent class.

Simplify __call fetching for static methods

If we have a __call on a parent, we should always also have a
__call on the child, so there's no need to perform this walk.

Simplify constant updating for properties

Instead of walking up the parent chain, use the scope stored in
the property info. This way we only need to walk one list of
property infos.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix #79145: openssl memory leak

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #79145: openssl memory leak

Fix #79145: openssl memory leak

We must increase the refcount of `return_value` only if `cert` is a
resource; this is already done in `php_openssl_evp_from_zval()`,
though.

Simplify ReflectionProperty::getDeclaringClass()

Drop the custom 'v' format from snprintf custom implementation.

Extensions should instead use the standard 's' format.

Closes GH-5100

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix #79146: cscript can fail to run on some systems

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #79146: cscript can fail to run on some systems

Fix #79146: cscript can fail to run on some systems

In the buildconf and configure batch files, Windows' cscript utility was being
run without the /e:jscript flag. This works on systems that have not had the
default .js file association changed, but if .js has been re-associated to
(say) an IDE, the batch files fail with the error message:

Input Error: There is no script engine for file extension ".js".

Merge branch 'PHP-7.4'

* PHP-7.4:
  Update NEWS wrt. sec fixes

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Update NEWS wrt. sec fixes

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix file clash in bug54446.phpt tests

Fix file clash in bug54446.phpt tests

Update NEWS wrt. sec fixes

Fix double free in array_reduce

return_value needs to be explicitly nulled out in the failure case.

fix test: fgetss() no longer exists

Merge branch 'PHP-7.4'

* PHP-7.4:
  Update NEWS
  Fix bug #79037 (global buffer-overflow in `mbfl_filt_conv_big5_wchar`)
  Fix #79099: OOB read in php_strip_tags_ex
  Fix #79091: heap use-after-free in session_create_id()

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Update NEWS
  Fix bug #79037 (global buffer-overflow in `mbfl_filt_conv_big5_wchar`)
  Fix #79099: OOB read in php_strip_tags_ex
  Fix #79091: heap use-after-free in session_create_id()

Merge branch 'PHP-7.2' into PHP-7.3

* PHP-7.2:
  Update NEWS
  Fix bug #79037 (global buffer-overflow in `mbfl_filt_conv_big5_wchar`)
  Fix #79099: OOB read in php_strip_tags_ex
  Fix #79091: heap use-after-free in session_create_id()

Update NEWS

Fix bug #79037 (global buffer-overflow in `mbfl_filt_conv_big5_wchar`)

Fix #79099: OOB read in php_strip_tags_ex

Fix #79091: heap use-after-free in session_create_id()

If the `new_id` is released, we must not use it again.

Drop the custom I length modifier from snprintf custom implementation.

Extensions should rather use the ZEND_LONG_FMT, ZEND_ULONG_FMT and
ZEND_XLONG_FMT macros defined in php-src/Zend/zend_long.h

Closes GH-5089

Speed up unserializing object properties

Hash table lookups are slow.
Don't do one a second time to update the property.

The call to zend_hash_update_ind goes back to 8b0deb8cd2d

Background: Properties are IS_INDIRECT when they're a declared property,
and point to properties_table.
See https://nikic.github.io/2015/06/19/Internal-value-representation-in-PHP-7-part-2.html#objects-in-php-7

Remove unnecessary result variable

Instead directly use return_value. Hopefully this also works
around the "may be uninitialized" warning...

Eliminate uses of ZVAL_ZVAL and friends

Instead add RETURN_COPY(_VALUE) macros will the expected behavior.

RETURN_ZVAL doesn't make any sense since PHP 7, but has stuck
around, probably because the alternative was to write directly to
the return_value variable.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix bug #79112: IMAP can't find OpenSSL during configure

Fix bug #79112: IMAP can't find OpenSSL during configure

Remove the check of PHP_OPENSSL inside SETUP_OPENSSL. It's the
responsibility of the caller to determine whether they want to
enable openssl or not. This makes SSL detection in IMAP work,
which uses a different option.

Additionally also clarify that --with-openssl-dir cannot actually
be used to specify an OpenSSL directory -- these options just
serve as a way to enable OpenSSL in extensions without also
enabling the OpenSSL extension. They need to be renamed to
something clearer in master.

Closes GH-5091.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Increase timeout in mysqli_reap_async_query test

Increase timeout in mysqli_reap_async_query test

The 2000 here is 2ms, which is a bit optimistic for parallelized
testing.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Let's make this safer from maliciously crafted filename