Fix conflict

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fixed support for named parameters (Zend/tests/named_params/references.phpt)
  Update NEWS for 8.0.0RC4
  Update NEWS for PHP 8.0.0RC3

Fixed support for named parameters (Zend/tests/named_params/references.phpt)

Update NEWS for 8.0.0RC4

Update NEWS for PHP 8.0.0RC3

Merge branch 'PHP-8.0'

* PHP-8.0:
  Don't throw for out of bounds offsets in strspn()

Don't throw for out of bounds offsets in strspn()

Make strspn($str1, $str2, $offset, $length) behaviorally
equivalent to strspn(substr($str1, $offset, $length), $str2)
by not throwing for out of bounds offset.

There have been two reports that this change cause issues,
including bug #80285.

Merge branch 'PHP-8.0'

Parameter type and name fixes in ext/standard

Closes GH-6382

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fixed support for typed references

Fixed support for typed references

Merge branch 'PHP-8.0'

* PHP-8.0:
  Skip bug46024.phpt under asan

Skip bug46024.phpt under asan

There is a legitimate leak here, but I don't think it's worthwhile
to fix it. It involves a process terminated while printing phpinfo.

UTF-32 conversion treats truncated characters as illegal

Add identify filter for UTF-32{,BE,LE}

Improve error handling for UTF-16{,BE,LE}

Catch various errors such as the first part of a surrogate pair not being
followed by a proper second part, the first part of a surrogate pair appearing
at the end of a string, the second part of a surrogate pair appearing out
of place, and so on.

UTF-16 text conversion handles truncated characters as illegal

This broke one old test (Zend/tests/multibyte_encoding_003.phpt), which used
a PHP script encoded as UTF-16. The problem was that to terminate the test
script, we need the text: "\n--EXPECT--". Out of that text, the terminating
newline (0x0A byte) becomes part of the resulting test script; but a bare
0x0A byte with no 0x00 is not valid UTF-16.

Since we now treat truncated UTF-16 characters as erroneous, an extra '?' is
appended to the output as an 'illegal character' marker.

Really, if we are running PHP scripts which are treated as encoded in UTF-16
or some other arbitrary text encoding (not ASCII), and the script is not
actually a valid string in that encoding, inserting '?' characters into the
code which the PHP interpreter runs is a bad thing to do. In such cases, the
script shouldn't be treated as UTF-16 (or whatever) at all.

I wonder if mbstring's encoding detection is being used in 'non-strict' mode?

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fixed tracing JIT for tests/classes/unset_properties.phpt

Fixed tracing JIT for tests/classes/unset_properties.phpt

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fixed memory leak

Fixed memory leak

Merge branch 'PHP-8.0'

* PHP-8.0:
  Add missing observer tests

Add missing observer tests

Closes GH-6378

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fixed register allocation
  Fixed MAY_BE_INDIRECT inference.

Fixed register allocation

Fixed MAY_BE_INDIRECT inference.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Better JIT support for aliases

Better JIT support for aliases

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fix inconsistency in PDO transaction state

Fix inconsistency in PDO transaction state

This addresses an issue introduced by #4996 and reported in
https://bugs.php.net/bug.php?id=80260.

Now that PDO::inTransaction() reports the real transaction state
of the connection, there may be a mismatch with PDOs internal
transaction state (in_tcx). This is compounded by the fact that
MySQL performs implicit commits for DDL queries.

This patch fixes the issue by making beginTransaction/commit/rollBack
work on the real transaction state provided by the driver as well
(or falling back to in_tcx if the driver does not support it).

This does mean that writing something like

    $pdo->beginTransaction();
    $pdo->exec('CREATE DATABASE ...');
    $pdo->rollBack(); // <- illegal

will now result in an error, because the CREATE DATABASE already
committed the transaction. I believe this behavior is both correct
and desired -- otherwise, there is no indication that the code did
not behave correctly and the rollBack() was effectively ignored.
However, this is also a BC break.

Closes GH-6355.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Deny serialization of finfo objects

Deny serialization of finfo objects

The resulting objects were already unusable, make it error out
earlier.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Deny cloning of fileinfo objects

Merge branch 'PHP-7.4' into PHP-8.0

* PHP-7.4:
  Deny cloning of fileinfo objects

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Deny cloning of fileinfo objects

Deny cloning of fileinfo objects

Using a cloned finfo object will crash.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Consistent error handling in mysqli_poll

Consistent error handling in mysqli_poll

This error condition should not actually be reachable, but change
it to be consistent with the other ones.

Also fix a memory leak.

Closes GH-6340.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Revert "Fix out-of-bounds write"

Merge branch 'PHP-7.4' into PHP-8.0

* PHP-7.4:
  Revert "Fix out-of-bounds write"

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Revert "Fix out-of-bounds write"

Revert "Fix out-of-bounds write"

This reverts commit bf6873a18e3b6b00f82a645c0893a281ae8eadb8.

CVE-2020-26159 is bogus; the "bug" was apparently a false positive
reported by Coverity, and the "fix" apparently wrong, see
<https://github.com/kkos/oniguruma/issues/221>.

Closes GH-6357.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Re-enable laravel tests

Re-enable laravel tests

Laravel is now sufficiently PHP 8 compatible.

Closes GH-6353.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fixed incorrect register allocation in ext/gd/tests/imagecopyresampled_variation1.phpt

Fixed incorrect register allocation in ext/gd/tests/imagecopyresampled_variation1.phpt

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fix #80268: loadHTML() truncates at NUL bytes

Merge branch 'PHP-7.4' into PHP-8.0

* PHP-7.4:
  Fix #80268: loadHTML() truncates at NUL bytes

Merge branch 'PHP-8.0'

* PHP-8.0:
  Don't set xdebug.default_enable in run-tests

Don't set xdebug.default_enable in run-tests

This throws a deprecation warning in XDebug 3.

Closes GH-6324.

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #80268: loadHTML() truncates at NUL bytes

Fix #80268: loadHTML() truncates at NUL bytes

libxml2 has no particular issues parsing HTML strings with NUL bytes;
these just cause truncation of the current text content, but parsing
continues generally.  Since `::loadHTMLFile()` already supports NUL
bytes, `::loadHTML()` should as well.

Note that this is different from XML, which does not allow any NUL
bytes.

Closes GH-6368.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Declare may_retry_reparse_point on windows only

Merge branch 'PHP-7.4' into PHP-8.0

* PHP-7.4:
  Declare may_retry_reparse_point on windows only

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Declare may_retry_reparse_point on windows only

Declare may_retry_reparse_point on windows only

Remove dated results from ext/hash/bench.php

These are not exactly representative anymore.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fix #62474: com_event_sink crashes on certain arguments

Merge branch 'PHP-7.4' into PHP-8.0

* PHP-7.4:
  Fix #62474: com_event_sink crashes on certain arguments

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #62474: com_event_sink crashes on certain arguments

Fix #62474: com_event_sink crashes on certain arguments

We have to make sure that the variant is of type `VT_DISPATCH` before
we access it as such.

Closes GH-6372.

Raise E_WARNING on PHP related errors

If Zip operations fails due to PHP error conditions before libzip even
has been called, there is no meaningful indication what failed; the
functions just return false, and the Zip status indicated that no error
did occur.  Therefore we raise `E_WARNING` in these cases.

Closes GH-6356.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fix #80258: Windows Deduplication Enabled, randon permission errors

Merge branch 'PHP-7.4' into PHP-8.0

* PHP-7.4:
  Fix #80258: Windows Deduplication Enabled, randon permission errors

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #80258: Windows Deduplication Enabled, randon permission errors

Fix #80258: Windows Deduplication Enabled, randon permission errors

A recent bug fix regarding symlinks claimed:

> After resolving reparse points, the path still may be a reparse
> point; in that case we have to resolve that reparse point as well.

While that is basically correct, some reparse points may point to
inaccessible system folders (e.g. `IO_REPARSE_TAG_DEDUP` points to
"\System Volume Information").  Since we don't know details about
arbitrary reparse points, and are mainly interested in nested symlinks,
we take a step back, and only resolve `IO_REPARSE_TAG_SYMLINK` for now.

Close GH-6354.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fix #80280: ADD_EXTENSION_DEP() fails for ext/standard and ext/date

Merge branch 'PHP-7.4' into PHP-8.0

* PHP-7.4:
  Fix #80280: ADD_EXTENSION_DEP() fails for ext/standard and ext/date

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #80280: ADD_EXTENSION_DEP() fails for ext/standard and ext/date

Fix #80280: ADD_EXTENSION_DEP() fails for ext/standard and ext/date

`ADD_EXTENSION_DEP()` relies on the `PHP_<extname>` config variables to
be set to `"yes"`, and since the standard and date extension are always
enabled, we define the respective variables uncoditionally.

Closes GH-6383.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fix bug 76618
  Fix bug 76618

Merge branch 'PHP-7.4' into PHP-8.0

* PHP-7.4:
  Fix bug 76618
  Fix bug 76618

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix bug 76618

Merge branch 'PHP-8.0'

* PHP-8.0:
  Use separate directory in dit_004.phpt

Use separate directory in dit_004.phpt

Make sure the directory is not modified while we're iterating it,
which may give unstable results.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fix parameter stats generation

[ci skip]

Fix parameter stats generation

[ci skip]

Merge branch 'PHP-8.0'

* PHP-8.0:
  Document breaking change in strspn/strcspn

[ci skip]

Document breaking change in strspn/strcspn

See https://phabricator.wikimedia.org/T264502.

[ci skip]

Merge branch 'PHP-8.0'

* PHP-8.0:
  Revert "Make ReflectionUnionType final"

Revert "Make ReflectionUnionType final"

This reverts commit ef6adb4e27853eb19bf50bad6486311920d6af7b.

Per Ondrej's comment, this is already being used by BetterReflection
adaptors, ugh.

Merge branch 'PHP-8.0'

Make ReflectionUnionType final

Closes GH-6384

Merge branch 'PHP-8.0'

* PHP-8.0:
  [ci skip] Fix typos in UPGRADING

[ci skip] Fix typos in UPGRADING

Merge branch 'PHP-8.0' into master

* PHP-8.0:
  Define config var PHP_JSON

Define config var PHP_JSON

Otherwise, `ADD_EXTENSION_DEP('foo', 'json')` fails, even though the
JSON extension is available.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Throw Value/TypeError for invalid $bodies in imap_mail_compose()

Throw Value/TypeError for invalid $bodies in imap_mail_compose()

Small drive by refactoring to use HashTables

Closes GH-6371

waitpid in ftp server tests

This is not relevant right now, but ensures consistent order if
the tests are repeated.

Allow running session tests in parallel

Use unique session IDs and/or save paths. Also removes the annoying
order-dependence of session tests.

Merge branch 'PHP-8.0'

Require stubs to declare return types for magic methods when possible

Closes GH-6376

Use ephemeral port in more server tests

Port the main php_cli_server.inc to use ephemeral ports, thus
allowing CLI server tests to be parallelized.

A complication here is that we also need to give each test a
separate doc root, to avoid index.php files writing over each
other.

Closes GH-6375.

Fix pasv_port determination

Apparently the ftp server is also used by ext/standard/tests/streams,
and only that part actually uses pasv.

Use ephemeral port in ftp tests

And enable them to run in parallel.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Make GC default threshold handling consistent

Make GC default threshold handling consistent

While the initial threshold is set to 10001 roots, the threshold
adjustment logic may then set it to 10000. The exact value really
doesn't matter, but we should make it consistent.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Don't allow passing unknown named params to class without ctor

Don't allow passing unknown named params to class without ctor

See also https://externals.io/message/112083.

Closes GH-6364.