Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  7.3 is now 7.3.22-dev

7.3 is now 7.3.22-dev

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #63527: DCOM does not work with Username, Password parameter

Fix #63527: DCOM does not work with Username, Password parameter

We must not mix multibyte and wide character strings in the
`COAUTHIDENTITY` structure.  Using wide character strings throughout
would have the advantage that the remote connection can be established
regardless of the code page of the server, but that would more likely
break BC, so we just drop the wide character string conversion of the
username.

Fixed bug #79839

Add reference type sources in array_walk.

Revert "Drop freetype from i386"

This reverts commit 8641d76f62273f3a73449fd720c93c63992f3b5a.

This appears to work again now...

Fix azure i386 build

Purge libsqlite3-0 to make sure we get the i386 library lateron.

Enable further tests on Windows

#79657 was cherry-picked in 7.4.7

Fix readlink related memory leak

Fix tests regarding negative fileinode()

The results of `fileinode()` may be negative due to wrap-around
behavior (at least on Windows as of PHP 7.4.0).

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix bug #78008: dns_check_record() always return true on Alpine

Fix bug #78008: dns_check_record() always return true on Alpine

- free handle before return result
- cleaned up remaining usage of MAXPACKET
- update dns_get_mx() to use the same approach

Closes GH-5854.

Fixed bug #79862

While normally a private property in the active scope would take
priority, we should not use this if it has the wrong "staticness".

Fixed bug #79820

Similar to what is done for ReflectionType itself, copy the
type name stored inside ReflectionProperty. Also make sure the
type field is always initialized for dynamic properties.

This is a non-issue in PHP 8, because we store a pointer to the
property_info there, rather than a copy.

Merge remote-tracking branch 'upstream/PHP-7.4' into PHP-7.4

* upstream/PHP-7.4:
  Fix test for x86 Windows

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fixed bug #79849

Fixed bug #79849

Closes GH-5853.

Fix test for x86 Windows

`st_dev` deliberately overflows on such systems, cf.
<http://svn.php.net/viewvc?view=revision&revision=350100>.

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fixed BC break of php_debug_zval_dump

Fixed BC break of php_debug_zval_dump

It introduced by fixing bug #79830

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fixed bug #79830 introduced by fixing bug #79821

Fixed bug #79830 introduced by fixing bug #79821

This also fixes memory error in debug_zval_dump and var_export.

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fixed bug #79821

Fixed bug #79821

HashTable was reallocated (zend_hash_packed_grow) during php_var_dump, so we should call GC_ADDREF to make SEPARATE_ARRAY work.

Closes GH-5837.

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fixed bug #79817

Fixed bug #79817

Use *_IND macros in a few places in string.c.

Enable further ext/standard/tests/file tests on Windows

Fix skip reasons

These tests fail on Windows for different reasons, but not because
symlinks, links, or lstat() would not be supported on Windows
generally.

Enable symlink_link_linkinfo_is_link_* tests on Windows

We mark symlink_link_linkinfo_is_link_error2.phpt as XFAIL on Windows
ZTS.  Several Windows API file system functions ignore trailing spaces
in absolute filenames after the final directory separator, which causes
`link(' ', $link)` to actually call `CreateHardLink()` which then
fails, because linking folders is not supported.  However, with NTS
builds (as well as on other systems), the $target is found to not
exist, so the function fails without actually attempting to create the
link.  This needs further investigation.

enable ext/ldap/tests on azure

fix some ext/ldap/tests

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix some memory bugs in ldap.c

Fix some memory bugs in ldap.c

Prevent parallel test conflicts

Both tests used the same folder, which could clash in case the tests
are run simultaneously.

Switch back to FREE_UNFETCHED_OP_DATA

Some of the code paths leading to this do not fetch op data.
Hopefully this fixes the release build failure.

Fix bug #79599 in a different way

Move the emission of the undefined variable notice before the
array separation.

Revert "Fixed bug #97599 (coredump in set_error_handler)"

This reverts commit ccd41e083359cf6dd264f88806dce4cc49d9358e.

This causes a large performance regression when notices are thrown
and large arrays are in scope, see bug #79794.

Revert "Partial fixed bug #79649 (Altering disable_functions from module init corrupts memory)"

This reverts commit a297c09da5ad355d53a8e8ea72655a06d15b7bc7.

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix bug #79787

Fix bug #79787

Closes GH-5807.

[ci skip] Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Report len as -1 instead of INT_MAX

Report len as -1 instead of INT_MAX

Per docs it should be -1. And would be on 32-bit systems, but
not on 64-bit systems.

(cherry picked from commit 39111585a2f8e40e72bdc662eb8b2e3c19e93615)

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Prep for 7.2.33
  Prep NEWS for 7.2.32 release

Merge branch 'PHP-7.2' into PHP-7.3

* PHP-7.2:
  Prep for 7.2.33
  Prep NEWS for 7.2.32 release

Prep for 7.2.33

Prep NEWS for 7.2.32 release

Fixed bug #79793

Make sure the string key is not released while throwing the
undefined index warning.

Fixed bug #79784

The fix here is essentially the same as for bug #78598, just for
the undefined variable notice, rather than the undefined index one.

[ci skip] Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Don't use deprecated curly brace offset syntax

Don't use deprecated curly brace offset syntax

(cherry picked from commit 7ec3aa1871074f5de25865af42c984a7668eb85f)

Better safe than sorry in case someone ever builds PHP 7.3 with a
future version of PHP SDK with bundled PHP 8.

Don't use deprecated curly brace offset syntax

Fixed bug #78598

When performing an RW modification of an array offset, the undefined
offset warning may call an error handler / OB callback, which may
destroy the array we're supposed to change. Detect this by temporarily
incrementing the reference count. If we find that the array has been
modified/destroyed in the meantime, we do nothing -- the execution
model here would be that the modification has happened on the destroyed
version of the array.

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fixed bug #79792

Fixed bug #79792

We need to remove the iterators even if the array is empty (we
will not create one if the first place, but the array may become
empty after the fact).

Fixed bug #79779

ASSIGN_OBJ_REF was not handling in zend_wrong_string_offset.

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fixed bug #79778

Fixed bug #79778

In the interest of avoiding side-effects during dumping, I'm
replacing the value with a <constant ast> string instead of
performing an update constant operation.

Fixed bug #79783

Make sure we don't drop the by-reference check when passing the
result of a VM builtin function.

Fix tests for x86 Windows

`st_dev` deliberately overflows on such systems, cf.
<http://svn.php.net/viewvc?view=revision&revision=350100>.

Skip test if SeCreateSymbolicLinkPrivilege is not given

Enable readlink_realpath_* tests on Windows

We modify _basic1.phpt so it runs on Windows as well.  The other test
cases hit the issue that `readlink()` fails normally for regular files,
but succeeds on Windows[1].  Therefore, we split these tests, but still
fix the skip reasons.

[1] <http://svn.php.net/viewvc?view=revision&revision=350097>

Enable most lstat_stat_* tests on Windows

Most of these have been skipped on Windows for no good reason (`lstat`
is available there as of PHP 4).  Several others would only fail,
because the `blksize` and `blocks` elements are always `-1` on Windows,
which can easily be fixed by using `%i` format specifiers instead of
`%d`.

Update php_version.h

That has apparently been overlooked.

Extract test helper function for SeCreateSymbolicLinkPrivilege check

Handle SO_ options only at SOL_SOCKET level

These options may have the same value as options at other levels.

This issue showed up on ppc64el.

Allow different error message for errno 58

Apparently this is EDEADLOCK on some systems.

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Remove bogus generator iterator dtor

Remove bogus generator iterator dtor

Fixes a use-after-free encountered in Symfony's SecurityBundle.
I don't have a reproducer for this, and believe the issue can only
occur if we leak an iterator (the leak is a separate issue).

We should not free the generator iterator here, because we do not
own it. The code that fetched the iterator is responsible for
releasing it. In the rare case where we do hit this code-path,
we cause a use-after-free.

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix leak when setting cyclic previous exception in finally

Fix leak when setting cyclic previous exception in finally

A curious exception handling pattern found in Symfony's HttpClient.

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #70362: Can't copy() large 'data://' with open_basedir

Fix #70362: Can't copy() large 'data://' with open_basedir

open_basedir is only relevant for plain files, so there is no need to
check it for other URL wrappers.

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #63208: BSTR to PHP string conversion not binary safe

Fix #63208: BSTR to PHP string conversion not binary safe

A `BSTR` is similar to a `zend_string`; it stores the length of the
string just before the actual string, and thus the string may contain
NUL bytes.  However, `php_com_olestring_to_string()` is supposed to
deal with arbitrary `OLECHAR*`s which may not be `BSTR`s, so we
introduce `php_com_bstr_to_string()` and use it for the only case where
we actually have to deal with `BSTR`s which may contain NUL bytes.

Contrary to `php_com_olestring_to_string()` we return a `zend_string`,
so we can save the re-allocation when converting to a `zval`.

We also cater to `php_com_string_to_olestring()` not being binary safe,
with basically the same fix we did for `php_com_olestring_to_string()`.

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #79756: finfo_file crash (FILEINFO_MIME)

Fix #79756: finfo_file crash (FILEINFO_MIME)

If `ctime` or `asctime` return `NULL`, we must not attempt to copy the
buffer, but rather return `NULL` as well.

Don't inline static call to instance method

Fixes the failure in bug79740.phpt with opcache.

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fixed bug #79741

Fixed bug #79741

Fixed bug #79740

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fixed bug #79030 Use usec from apache request time

Fixed bug #79030 Use usec from apache request time

Don't unnecessarily truncate to milliseconds.

Closes GH-5760.

Handle *0 / *1 more consistently

Avoid throwing a DES salt deprecation warning if the libc crypt
implementation is used.

Fix potential environment variable deadlock

We have to unlock the environment before bailing out.

Fix #69804: ::getStaticPropertyValue() throws on protected props

`ReflectionClass` allows reading of the values of private and protected
constants, and also to get private and protected static methods.
Therefore getting the values of private and protected static properties
is also permissible, especially since `::getStaticProperties()` already
allows to do so.

We also allow ::setStaticPropertyValue() to modify private and
protected properties, because otherwise this method is useless, as
modifying public properties can be done directly.

Fix #79487: ::getStaticProperties() ignores property modifications

When retrieving the static class properties via reflection, we have to
cater to possible modifications.

Revert "Fix #79487: ::getStaticProperties() ignores property modifications"

This reverts commit a895bb6885fbceea3e8375816969d5510d8d082e.

Fix #79487: ::getStaticProperties() ignores property modifications

When retrieving the static class properties via reflection, we have to
cater to possible modifications.

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #55857: ftp_size on large files

Fix #55857: ftp_size on large files

`atol()` returns a `long` which is not the same as `zend_long` on
LLP64; we use `ZEND_ATOL()` instead.

There is no need for a new test case, since filesize_large.phpt already
tests for that behavior; unfortunately, the FTP test suite relies on
`pcntl_fork()` and therefore cannot be run on Windows.

Restore XFAIL on fpm test

Still fails intermittently.

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  7.3 is now 7.3.21-dev

7.3 is now 7.3.21-dev

PHP-7.4 is now 7.4.9-dev

SplHeap: Avoid memcpy on overlapping pointer

Check if data would overlap and also add an assert. Previous
implementations didn't have this issue, as the direct assignment was
used.

Signed-off-by: Anatol Belski <ab@php.net>

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fixed bug #79570

Fixed bug #79570

Use the same logic for getgrgid_r, getpwnam_r and getpwuid_r
as for getgrnam_r in #75696

Closes GH-5740.

Merge branch 'PHP-7.3' into PHP-7.4

Fixed bug #79710

Make sure we don't use zresource after the stream has been destroyed.