Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #78538: shmop memory leak

Fix #78538: shmop memory leak

If the descriptor's refcount drops to zero, we have to unmap the
respective file view, to avoid leaking memory.

Slightly increase tolerance in time_sleep_until() test

With the 1ms tolerance this still occasionally fails on macos.
I love macos. It always works and causes no problems at all.

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.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.

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.

Add test for bug #73816

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

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.

Fixed bug #79056

Add test case for bug #78883

The original test has been provided by divinity76.

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.

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.

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.

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

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

Add test case for bug #78883

The actual test has been provided by divinity76.

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.

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.

Avoid undefined behavior

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.

Fixed bug #79046

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 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.3' into PHP-7.4

* PHP-7.3:
  Add missing skipif clause

Add missing skipif clause

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.

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

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.

Skip test for clang builds

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

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)

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.

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.

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

Partial fix for bug #79029

Similar problem (#79022) also exists in Interfaces

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

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.

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.

Try to make proc_open_bug69900.phpt more robust

Merge branch 'PHP-7.3' into PHP-7.4

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

Fix #78982: pdo_pgsql returns dead persistent connection

Call PQconsumeInput() before PQstatus() to update the status.

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #78980: pgsqlGetNotify() overlooks dead connection

Fix #78980: pgsqlGetNotify() overlooks dead connection

pgsqlGetNotify() didn't check result of PQconsumeInput().

Fixed bug #79000

Don't report EAGAIN/EWOULDBLOCK as errors for fwrite on
non-blocking socket streams. This matches behavior for fread,
as well as behavior for plain file streams.

Closes GH-5026.

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #74170: locale information change after mime_content_type

Fix #74170: locale information change after mime_content_type

Some functions in libmagic (distributed with fileinfo extension) perform this sequence of calls:
func() {
setlocale(LC_TYPE, "C")
.. do some work ..
setlocale(LC_TYPE, "")
}

It effectively resets LC_TYPE if it that was set before the function call.

To avoid manipulations with current locale at all, the problematic functions
were modified to use locale-independent functions.

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #78402: pcntl_signal() misleading error message

Fix #78402: pcntl_signal() misleading error message

An error message can be misleading when a handler
passed to pcntl_signal() is not callable.

Fix early free of assign_obj op_data

We need to make sure that op_data is only freed after populating
result, as op_data may be the only thing holding the value in the
case of an overloaded assignment.

This reverts the code to how it looked like in 7.3.

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fixed bug #78999 (Cycle leak when using function result as temporary)

Fixed bug #78999 (Cycle leak when using function result as temporary)

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #78983: pdo_pgsql config.w32 cannot find libpq-fe.h

Fix #78983: pdo_pgsql config.w32 cannot find libpq-fe.h

When configured with a path specified.

Avoid signed integer overflow in string offset check

Cast to size_t before performing operations instead of afterwards.

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Add destructor annotations in ini parser

Add destructor annotations in ini parser

It would be better to switch this to use %union and %type annotations,
but not going to do that change for 7.3.

Fix leak in assign_ref with function

As far as I can see, the retval copying is already done in all
callers of this function, so it should not be duplicated here.

Fixed bug #78776

By using the normal inheritance check if the parent is abstract
as well.

fix release date

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  fix release date
  fix release date

fix release date

Merge branch 'PHP-7.2' into PHP-7.3

* PHP-7.2:
  fix release date

fix release date

Fixed bug #78986

Don't assume that handlers live in the arena, they may also be in
SHM.

Add CONFLICTS tag

These tests can be the reason file_get_contents_variation8.phpt fails times to times.

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Rethrow generator exception even without active stack frame

Rethrow generator exception even without active stack frame

Finally blocks in generators may be invoked during shutdown, in
which case we don't have a stack frame. Similar to what
zend_call_function does, we still need to rethrow these exceptions,
otherwise they will be hidden (and leak).

Fix use-after-free when trying to write to closure property

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix freeing of dynamic call name

Fix freeing of dynamic call name

We need to free op2 if the call construction fails.

Also remove a redundant check for !call.

Fix const/cv freeing on failed reference assignment

More updates to CONTRIBUTING.md

7.4 is stable.

Update CONTRIBUTING.md

7.1 is EOL; 7.2 is security fixes only.

Fix #78976: SplFileObject::fputcsv returns -1 on failure

`php_stream_write()` returns `-1` on failure now, which has to be
catered to by `SplFileObject::fputcsv()` which still is supposed to
return `false` on failure.

Free buffer in php_verror even if it is empty

vspprintf allocates a buffer even if the resulting string is empty.

Fix resetting of internal_encoding and friends

We need to update the value even if new_value is NULL. In particular,
it should be reset back to NULL after each request if the setting was
not specified on startup. Otherwise we leave dangling pointers.

Avoid float to int cast UB in exif

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Update/fix NEWS [ci skip]

Update/fix NEWS [ci skip]

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  [ci skip] Update NEWS
  [ci skip] Update  NEWS
  Fix test