Merge branch 'PHP-8.0'

* PHP-8.0:
  Handle errors during PDO row fetch

Merge branch 'PHP-7.4' into PHP-8.0

* PHP-7.4:
  Handle errors during PDO row fetch

Handle errors during PDO row fetch

The EOF flag also gets set on error, so we always end up ignoring
errors here.

However, we should only check errors for unbuffered results. For
buffered results, this function is guaranteed not to error, and
querying the errno may return an unrelated error.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fix format specifier

Merge branch 'PHP-7.4' into PHP-8.0

* PHP-7.4:
  Fix format specifier

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix format specifier

Fix format specifier

In this function, `i` is of type `size_t`.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Report gone away error in one more place

Merge branch 'PHP-7.4' into PHP-8.0

* PHP-7.4:
  Report gone away error in one more place

Report gone away error in one more place

Merge branch 'PHP-8.0'

* PHP-8.0:
  [ci skip] Fix NEWS order

Merge branch 'PHP-7.4' into PHP-8.0

* PHP-7.4:
  [ci skip] Fix NEWS order

[ci skip] Fix NEWS order

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fix #44618: Fetching may rely on uninitialized data

Merge branch 'PHP-7.4' into PHP-8.0

* PHP-7.4:
  Fix #44618: Fetching may rely on uninitialized data

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #44618: Fetching may rely on uninitialized data

Fix #44618: Fetching may rely on uninitialized data

Unless `SQLGetData()` returns `SQL_SUCCESS` or `SQL_SUCCESS_WITH_INFO`,
the `StrLen_or_IndPtr` output argument is not guaranteed to be properly
set.  Thus we handle retrieval failure other than `SQL_ERROR` by
yielding `false` for those column values and raising a warning.

Closes GH-6281.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Retain QUIT_SEND connection state

Merge branch 'PHP-7.4' into PHP-8.0

* PHP-7.4:
  Retain QUIT_SEND connection state

Retain QUIT_SEND connection state

If the server goes away while reading a packet, don't go back into
the READY state. This will just cause broken pipe errors down the
line.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fixed bug #65825

Merge branch 'PHP-7.4' into PHP-8.0

* PHP-7.4:
  Fixed bug #65825

Fixed bug #65825

Set error_info when we fail to read a packet, instead of throwing
a warning. Additionally we also need to populate the right
error_info in rowp_read -- we'll later take the error from the
packet, not the connection.

No test case, as this is hard to reliably test. I'm using the
test case from:
https://github.com/php/php-src/pull/2131#issuecomment-538374838

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fix #79177: FFI doesn't handle well PHP exceptions within callback

Merge branch 'PHP-7.4' into PHP-8.0

* PHP-7.4:
  Fix #79177: FFI doesn't handle well PHP exceptions within callback

Fix #79177: FFI doesn't handle well PHP exceptions within callback

We have to error on unhandled exceptions in FFI callbacks, to avoid
passing back undefined values.

This has been discussed and agreed upon in a previous PR[1].

[1] <https://github.com/php/php-src/pull/5120>

Closes GH-6366.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fixed bug #66528

Merge branch 'PHP-7.4' into PHP-8.0

* PHP-7.4:
  Fixed bug #66528

Fixed bug #66528

Report errors in commit, rollback and autocommit handlers.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fixed run-time binding of preloaded dynamically declared function

Merge branch 'PHP-7.4' into PHP-8.0

* PHP-7.4:
  Fixed run-time binding of preloaded dynamically declared function

Fixed run-time binding of preloaded dynamically declared function

Merge branch 'PHP-8.0'

* PHP-8.0:
  Handle mysqli errors in more cases
  Sync test with master

Merge branch 'PHP-7.4' into PHP-8.0

* PHP-7.4:
  Handle mysqli errors in more cases
  Sync test with master

Handle mysqli errors in more cases

Report errors autocommit, commit, rollback and mysqli_stmt_attr_set.

Additionally, copy the error from conn to stmt when preparing fails,
so these errors are also handled by mysqli_stmt_prepare.

Closes GH-6157.

Sync test with master

Sync ext/mysqli/tests/mysqli_report.phpt with PHP-8.0/master, as
the current difference in indentation makes it hard to merge.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fix bug #79375

Merge branch 'PHP-7.4' into PHP-8.0

* PHP-7.4:
  Fix bug #79375

Fix bug #79375

Make sure deadlock errors are properly propagated and reports in
a number of places in mysqli and PDO MySQL.

This also fixes a memory and a segfault that can occur under these
conditions.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Don't force rebuild of symbol table, when populating $http_response_header variable by the HTTP stream wrapper

Don't force rebuild of symbol table, when populating $http_response_header variable by the HTTP stream wrapper

Check for null dbh methods in get_gc

Methods may not be set at the point where get_gc is called. Fixes
a segfault in Laravel.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fixed memory leak in Zend/tests/bug78999.phpt

Fixed memory leak in Zend/tests/bug78999.phpt

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fixed tracing JIT support for aliasses (Zend/tests/bug75420.13.phpt and ext/standard/tests/streams/bug60106.phpt)

Fixed tracing JIT support for aliasses (Zend/tests/bug75420.13.phpt and ext/standard/tests/streams/bug60106.phpt)

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