Use RETURN_THROWS() after zend_value_error()

Use RETURN_THROWS() during ZPP in the remaining extensions

In reflection, sodium, and SPL

Merge branch 'PHP-7.4'

* PHP-7.4:
  [ci skip] Revert "Add test case for bug #78883"

[ci skip] Revert "Add test case for bug #78883"

This reverts commit 09e76cbe6d73399f3476942e7502de0bdbf65fa2, because
the test fails reliably on AppVeyor.  This needs closer investigation.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Add test case for bug #78883

Add test case for bug #78883

The actual test has been provided by divinity76.

Fix erroneous change

Merge branch 'PHP-7.4'

* PHP-7.4:
  Remove superfluous compiler option

Remove superfluous compiler option

`-fsanitize-address-use-after-scope` is the default as of clang 5.0.0,
and we don't allow to enable ASan for older versions anyway.

Use RETURN_THROWS() during ZPP in most of the extensions

Except for some bigger ones: reflection, sodium, spl

Use ZEND_THROWS() during ZPP in the GD, gettext, GMP, and hash extensions

Use ZEND_THROWS() during ZPP in enchant, exif, fileinfo, filter, and FTP extensions

Use RETURN_THROWS() during ZPP in the date, dba and dom extensions

Use RETURN_THROWS() during ZPP in bz2, calendar, com_dotnet extensions

Merge branch 'PHP-7.4'

* PHP-7.4:
  Also propagate include exceptions in opcache
  Make url_stats in resolve_path quiet

Also propagate include exceptions in opcache

Make url_stats in resolve_path quiet

These stats are used to check whether the file exists -- they
should not generate errors. Having the flag set is particularly
important for custom stream wrappers.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Avoid undefined behavior

Avoid undefined behavior

Merge branch 'PHP-7.4'

* PHP-7.4:
  Properly propagate url_stat exceptions during include

Properly propagate url_stat exceptions during include

Make sure we abort operations early, and that we don't emit
additional warnings or errors if an exception has been thrown.

Fix typo

Fix #78790: mysqli_get_client_info() expects exactly 0 parameters, 1 given

Although the `mysqli` parameter is unused, it had been accepted so far,
and the documentation even claims that parameter would be required.  To
not break BC, we allow it again.

Use RETURN_THROWS() during ZPP in main, sapi, win32, and Zend

Use RETURN_THROWS() when an exception is thrown

Closes GH-5036

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fixed bug #79046

Fixed bug #79046

Fix typo

Convert zend_parse_parameters_none() to fast ZPP in standard lib

Closes GH-5037

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix #79040: Warning Opcode handlers are unusable due to ASLR

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #79040: Warning Opcode handlers are unusable due to ASLR

Fix #79040: Warning Opcode handlers are unusable due to ASLR

We must not use the same shared memory OPcache instance for different
SAPIs, since their memory layout is different.  To avoid this, we add
the SAPI name (truncated to at most 20 characters) to the names of the
memory base file, the mutex and the file mapping.

Add FILTER_VALIDATE_BOOL as alias for FILTER_VALIDATE_BOOLEAN

Both filters are equivalent, but FILTER_VALIDATE_BOOL uses our
canonical name for the type (the only one permitted in type
declarations for example), so the new name is preferred long
term.

The old name may be deprecated in the future, but no specific
timeline is planned.

Fix SSA construction for ADD_ARRAY_ELEMENT in RC_INFERENCE mode

This was broken in cc29cbe80c0944097f8980384f883907d46512ce.

Regenerate stubs

Looks like these GD changes got lost along the way...

Merge branch 'PHP-7.4'

* PHP-7.4:
  Add test for bug #79031
  Revert "Increase serialize_lock while decoding session"

Add test for bug #79031

Fixed by preceding revert.

Revert "Increase serialize_lock while decoding session"

This reverts commit b8ef7c35abd31666d9fb317db4b09a9eef0ede6c.

See bug #79031. The semantics of serialize locking aren't quite
correct right now, and the use of the lock in this particular
place makes us hit the issue in a common case. I'm reverting this
commit for PHP 7.4 and will try to fix this properly for PHP 8,
as I believe it will require ABI breakage.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Add missing skipif clause

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Add missing skipif clause

Add missing skipif clause

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix #79033: Curl timeout error with specific url and post

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #79033: Curl timeout error with specific url and post

Fix #79033: Curl timeout error with specific url and post

We must not set an empty mime structure as `CURLOPT_MIMEPOST`; instead
we set it to `NULL` if `CURLOPT_POSTFIELDS` has been set to an empty
array.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Handle empty password fast path in caching_sha2_password
  Handle error response during caching_sha2_password auth
  Add support for caching_sha2_password in change user authentication
  Fix unix socket check during caching_sha2_password
  Support auth switch request during caching sha2 auth

Handle empty password fast path in caching_sha2_password

If an empty password is used, no additional packets are exchanged
during caching_sha2_password auth. We're only looking for an
OK/ERR response.

Handle error response during caching_sha2_password auth

In particular, this fixes handling of expired passwords.

Add support for caching_sha2_password in change user authentication

Same as for connection handshakes.

Fix unix socket check during caching_sha2_password

The fact that conn->unix_socket is set does not mean that a Unix
socket is actually in use -- this member is set in a default
configuration.

Instead check whether a unix_socket stream ops is used.

Support auth switch request during caching sha2 auth

Merge branch 'PHP-7.4'

* PHP-7.4:
  Don't use PASSWORD() function in test
  Specify explicit row order in PDO MySQL test
  Skip bug77956.phpt if local_infile disabled on server
  Don't check TIMESTAMP NOT NULL in fetch_field_flags test
  Don't test the RENAME DATABASE query
  Make GeomFromText tests compatible with MySQL 8

Don't use PASSWORD() function in test

Doesn't exist anymore in MySQL 8.

Specify explicit row order in PDO MySQL test

Skip bug77956.phpt if local_infile disabled on server

Don't check TIMESTAMP NOT NULL in fetch_field_flags test

The behavior of this is very dependent on the MySQL vendor, MySQL
version and MySQL configuration, in particular the
explicit_defaults_for_timestamp variable.

I don't think it's worthwhile to try and model this exactly, so
drop the test.

Don't test the RENAME DATABASE query

Apparently this only existed for a short time, because it was
found to be insecure. Don't try to test it.

Make GeomFromText tests compatible with MySQL 8

The function has been renamed to ST_GeomFromText.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Skip test for clang builds

Skip test for clang builds

On Windows, clang builds don't use __vectorcall, so executing this test
does not make sense.

Clarify usage of max_fd in php_poll2() on Windows

Actually, `max_fd` is not used on Windows, since `PHP_SAFE_MAX_FD` and
`select` ignore it; so it makes no sense to calculate it in the loop.
Even worse, since `php_socket_t` is unsigned on Windows, it will never
be modified during the loop, because `SOCK_ERR` is already the largest
representable value of that type.

Therefore we skip the loop on Windows, and add a clarifying comment.

Fixed type_info for new array element (it may be only NULL)

cleanup

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fixed bug #79029 (Use After Free's in XMLReader / XMLWriter).

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fixed bug #79029 (Use After Free's in XMLReader / XMLWriter).

Fixed bug #79029 (Use After Free's in XMLReader / XMLWriter).

We backport the fix PHP 7.3, since this branch is affected as well.

(cherry picked from commit b5e004379647bd1ebb75eb2eac8826fb6abdd3d8)
(cherry picked from commit e36daa6927c05d2e687bb77495ef206cde118b33)
(cherry picked from commit 2704ee6844c03348de9d15e74646d09007ef0f7c)

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix test case

Fix test case

As of PHP 7.3.0, unlinking files with open handles is possible on
Windows, but these file entries are still blocked until all open
handles are closed.  Since this test doesn't require to write to the
same file, we use three separate files.

We also add the missing skip check for XMLReader.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fixed bug #79029 (Use After Free's in XMLReader / XMLWriter).

Fixed bug #79029 (Use After Free's in XMLReader / XMLWriter).

Merge branch 'PHP-7.4'

* PHP-7.4:
  Revert "Fixed bug #79029 (Use After Free's in XMLReader / XMLWriter)"

Revert "Fixed bug #79029 (Use After Free's in XMLReader / XMLWriter)"

This reverts commit d8ad2f4695ebd6dc2d4e668d9baeac1c071791a6.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fixed bug #79029 (Use After Free's in XMLReader / XMLWriter)
  Partial fix for bug #79029

Conflicts:
ext/xmlwriter/php_xmlwriter.c

Fixed bug #79029 (Use After Free's in XMLReader / XMLWriter)

Partial fix for bug #79029

Build call graph before type inference

Implemented FR #78638 (__PHP_Incomplete_Class should be final)

This should be minor and won't impact anyone

Merge branch 'PHP-7.4'

* PHP-7.4:
  Similar problem (#79022) also exists in Interfaces
  Fixed bug #79022 (class_exists returns True for classes that are not ready to be used)

Similar problem (#79022) also exists in Interfaces

Fixed bug #79022 (class_exists returns True for classes that are not ready to be used)

Set opcode handler

Fix #78880: Spelling error report

We fix the most often occuring typos according to a recent codespell
report[1] in tests, code comments and documentation.

[1] <https://fossies.org/linux/test/php-src-master-f8f48ce.191129.tar.gz/codespell.html>.

Promote warning to exception for proc_open() when null byte is encountered

GH-5004

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix leak when generator closed during yield in finally

Fix leak when generator closed during yield in finally

In this case we need to free any pending exceptions or return values
that will be discarded.

Remove unnecessary FN type information from OPCache

GH-5029

Promote warnings to exceptions in string search related functions

GH-5004

Promote warnings to exceptions in proc_open() function

GH-5004

Promote warrnings to exceptions in chgrp() function

GH-5004

Promote warning to exception in dns_check_record() function

GH-5004

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix #79008: General performance regression with PHP 7.4 on Windows

Fix #79008: General performance regression with PHP 7.4 on Windows

We no longer try to retrieve the filename of a given stream when
fstat'ing, because this is very slow.  Since we neither didn't do that
in PHP 7.3 and earlier, we regard this as sensible trade-off.

Simplify constant expression

Co-Authored-By: Christoph M. Becker <cmbecker69@gmx.de>

Fix merge mistake

Make get_defined_vars() always return an array

GH-5025

Fix F1 type information in OPCache

Closes GH-5025

Fix smaller issues with stubs

GH-5025

Merge branch 'PHP-7.4'

* PHP-7.4:
  Try to make proc_open_bug69900.phpt more robust

Try to make proc_open_bug69900.phpt more robust

Fix string offset signed int UB in jit as well

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix #78982: pdo_pgsql returns dead persistent connection