Merge branch 'PHP-7.4'

* PHP-7.4:
  Try to fix intermittently failing test case

Try to fix intermittently failing test case

This test fails intermittently due to taskill reporting failure to kill
the process (because it already has been terminated).  We increase the
sleep time, to hopefully prevent that from happening again.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix #79015: undefined-behavior in php_date.c

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #79015: undefined-behavior in php_date.c

Fix #79015: undefined-behavior in php_date.c

We check that the given microsecond fraction is in the valid range
[0, 1000000[, and otherwise mark it as invalid.  We also drop the
useless do loop; a plain block is sufficient here.

Don't enable sanitizer by default for Windows snapshot builds

Snapshot builds are release builds, and therefore enabling sanitizers
is undesireable.

Convert zend_parse_parameters_none() to fast ZPP

I've done the conversion in those extensions where fast ZPP is predominant.

Use zend_parse_parameters_none() instead of zend_parse_parameters_none_throw()

Merge branch 'PHP-7.4'

* PHP-7.4:
  Remove unnecessary strncpy in zip

Remove unnecessary strncpy in zip

This causes a warning on GCC 9 and is unnecessary to boot: We
only use "cwd" for the open_basedir check, so we can just as well
pass in the original string.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Add test for bug #73816

Add test for bug #73816

This has been fixed by 0f2cdbf214efd98b4bdaf5ca41728faf00e7c037,
add the test case.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fixed bug #71876

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fixed bug #71876

Fixed bug #71876

This is a backport of fcdc0a6db0ae63fbed9e3828137b899b844623ce
to the PHP-7.3 branch. We need to make sure that OnUpdateString
is also called for a NULL value, otherwise the reset of the encoding
at the end of the request will not work.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fixed bug #79056

Fixed bug #79056

Implement WeakMap

RFC: https://wiki.php.net/rfc/weak_maps

Use RETURN_THROWS() after zend_throw_exception() in reflection extension

Use RETURN_THROWS() after zend_parse_method_parameters()

Merge branch 'PHP-7.4'

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

Add test case for bug #78883

The original test has been provided by divinity76.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fixed bug #79055
  Extract functions for file cache type serialization

Fixed bug #79055

Fix file cache serialization of property types. I'm changing the
overall type serialization format to perform additional adjustments
in order to yield a plausible pointer for zend_type, rather than
using an entirely separate serialization format, as was previously
done. That would have been annoying to extend to the case of CE
pointers.

Extract functions for file cache type serialization

This is already done in master.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix file cache run_time_cache unserialization
  Update ZCSG(map_ptr_last) only if for_shm

Fix file cache run_time_cache unserialization

If the script was serialized as file_cache_only (thus non-immutable)
and then gets unserialized into SHM, we need to allocate a new
run_time_cache slot and can't use the serialized arena pointer.

Update ZCSG(map_ptr_last) only if for_shm

Otherwise we may get a memory protection fault here. Updating of
ZCSG(map_ptr_last) is handled when loading from file cache to SHM.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Populate hashtable of __sleep() properties
  Deref names returned by __sleep()
  Extract php_var_serialize_nested_data() function

Populate hashtable of __sleep() properties

Instead of populating a hashtable of property names and then
directly serializing.

This has the advantage of a) detecting duplicate properties more
precisely and b) gives us the ability to discard values without
rewriting the serialization string after the fact for GH-5027.

Deref names returned by __sleep()

Extract php_var_serialize_nested_data() function

And split code-paths for arrays and objects based on it.

Use RETURN_THROWS() after zend_throw_exception() in most of the extensions

Switch hash() function to use fastzpp

For short inputs ZPP can be a large fraction of the hash()
execution time.

Switch php_hash_fetch_ops() to use zend_string

This has two advantages: If the string is already lowercase, we
do not need to copy it, and it will hopefully match the interned
string name of the hash, making the comparison more efficient.

Always delete tempfile created by ftruncate test.

The ftruncate() test will fill up the disk when the disk has less than
2GB of space left, on some systems.
I think it expands the file to less than 2GB, then returns an error code,
and the file remains at the larger size.

Because the disk is full, the `--CLEAN--` script can't be saved to
`ftruncate_bug76422.clean.php` to be executed, and the cleanup can't be run.
Subsequent tests also fail to run.

Closes GH-5043

Use RETURN_THROWS() after zend_throw_error()

Use RETURN_THROWS() after zend_type_error()

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