Merge branch 'PHP-7.4'

Fix out of bounds read in sccp

Avoid reliance on arena details on phpdbg oplog

Instead of guessing what the address of the first arena allocation
is going to be, embed the sentinel in the oplog_list structure
directly.

Update error message in libmysql test

Fix compilation errors when building against libmysql

Closes GH-4316.

Merge branch 'PHP-7.4'

Fix out-of-bounds read in array compilation

UNPACK elements only have one child. Don't access the second one
until we have excluded this case.

Add tracked arena allocator

Available under -DZEND_TRACK_ARENA_ALLOC. This will use the system
allocator combined with arena checkpointing & release semantics
and allows analyzing arena usage under asan/valgrind.

I've sacrificed the duplicate arena implementation in mysqlnd, as
the integration with mysqlnd alloc is not worth the code duplication
to me.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix bug #78192 PDO SQLite SegFault when reuse statement after schema has changed

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix bug #78192 PDO SQLite SegFault when reuse statement after schema has changed

Merge branch 'PHP-7.2' into PHP-7.3

* PHP-7.2:
  Fix bug #78192 PDO SQLite SegFault when reuse statement after schema has changed

Fix bug #78192 PDO SQLite SegFault when reuse statement after schema has changed

Reset stmt->columns when column count changed on new execution of prepared statement

Merge branch 'PHP-7.4'

Add support for proc_open() with a command array

In this case the progarm will be executed directly, without a shell.
On Linux the arguments are passed directly to execvp and no escaping
is necessary. On Windows we construct a command string using escaping
with the default Windows command-line argument parsing method described
at https://docs.microsoft.com/en-us/cpp/cpp/parsing-cpp-command-line-arguments.

Apart from avoiding the issue of argument escaping, passing an array
and bypassing shell has the advantage of allowing proper signal
delivery to the opened process (rather than the shell).

Merge branch 'PHP-7.4'

* PHP-7.4:
  Reduce overhead

Reduce overhead

Merge branch 'PHP-7.4'

* PHP-7.4:
  implement tsrm_is_shutdown API

implement tsrm_is_shutdown API

Merge branch 'PHP-7.4'

* PHP-7.4:
  Use the portable zend_strndup() instead of strndup()

Use the portable zend_strndup() instead of strndup()

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fixed use-after-free

Fixed use-after-free

Merge branch 'PHP-7.4'

Fix mysqlnd memory leak

The actual leak is observed in ext/pdo_mysql/tests/bug_74376.phpt.
The persistent connection leaks because a refcount decrement on a
result is missed. The refcount decrement is missed because
free_result_contents is used, rather than free_result.

Looking at other uses of free_result_contents, it looks like they
could also suffer from this problem. Apart from one case,
free_result_contents is always used to release the result entirely
(I've adjusted the one differing case to only free meta), so I'm
moving most of the logic from free_result into free_result_contents.
The only difference is now that free_result will skip_result first.

Free ZTS lock in opcache

For some reason this only shows up as a leak when using phpdbg.

Fix init_file leak in phpdbg

run-tests: Don't pass --INI-- settings to --SKIPIF--

If we're testing ini settings that cause startup failures, we'll
never get to that SKIPIF block...

Also change settings2param to return a value instead of modifying
(to a different type no less!) in place.

Fix custom heap free

This seems to be designed around the use-case where the custom
allocator is a wrapper around ZMM.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Bugfix #78208 Needs rehash with an unknown algo should always return true.

Bugfix #78208 Needs rehash with an unknown algo should always return true.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Catch up with some recent changes [ci skip]

Catch up with some recent changes [ci skip]

- Mention mysqli::stat()
- Mention removal of HAVE_* for always defined extensions
- Mention removed symbols defined during the build

Fix typo

Merge branch 'PHP-7.4'

* PHP-7.4:
  Remove HAVE_STRCOLL check

Remove HAVE_STRCOLL check

The strcoll function is defined in the C89 standard and should be
on today's systems always available via the <string.h> header.

https://port70.net/~nsz/c/c89/c89-draft.html#4.11.4.3

- Remove also SKIPIF strcoll check in test

Merge branch 'PHP-7.4'

* PHP-7.4:
  Sync HAVE_HASH, HAVE_HASH_EXT, PHAR_HASH_OK symbols

Sync HAVE_HASH, HAVE_HASH_EXT, PHAR_HASH_OK symbols

The hash extension is always available since PHP-7.4. The symbol
HAVE_HASH_EXT is kept for BC reasons and removed in PHP-8.0.

This patch also removes the PHAR_HASH_OK since it is no longer
relevant.

Merge branch 'PHP-7.4'

Remove posix_getlogin_basic.phpt test

The premise of this test (that the getlogin user has to be the same
as the geteuid user) is incorrect.

Fix opcache blacklist leak

Don't recompute (and leak) the regex for every glob element, only
do this once at the end.

Merge branch 'PHP-7.4'

Fix phpdbg shutdown order

In particular, make sure that everything using zmm is released
before zmm is shut down. phpdbg currently gets away with this,
because either a) its custom handlers are used and no auto-free
happens or b) the system allocator is used and no auto-free happens.
With the tracking allocator for asan this no longer works.

Don't use zmm for PHPDBG_G(exec)

Uses system allocator when backing up settings ... let's avoid
unnecessary confusion.

Fix persistent XML memory leaks in SOAP

SOAP uses a horrible bailout based error handling approach -- avoid
leaking persistent XML memory by catching bailouts in a number of
places.

Add tracked allocator mode

In this case we will use the system allocator, but still remember
all allocations and free them the same way that Zend MM does. This
allows us to accurately model leak behavior.

Enabled using USE_ZEND_ALLOC=0 USE_TRACKED_ALLOC=1.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Show actual enchant version if possible

Show actual enchant version if possible

As of libenchant 1.6.0 there is enchant_get_version()[1], so we use it,
if available, to show the actual enchant version in the PHP info.  We
also drop the fake ENCHANT_VERSION_STRING altogether, but stick with
showing version 1.5.x if at least HAVE_ENCHANT_BROKER_SET_PARAM is
defined.

Future scope: we may consider requiring enchant 1.6.0 (or later), since
this has been released in April 2010, and likely is available
everywhere.

[1] <https://www.abisource.com/projects/enchant/>

Merge branch 'PHP-7.4'

* PHP-7.4:
  Separate common code abd eliminate useless checks

Separate common code abd eliminate useless checks

Merge branch 'PHP-7.4'

* PHP-7.4:
  Inline hot path

Inline hot path

Merge branch 'PHP-7.4'

Make zend_hash_rehash() a void function

It always succeeds.

Closes GH-4304.

Fix persistent connection leak in mysqli

Fix use-after-free in stream freeing during shutdown

Streams will be freed in an unpredictable order during shutdown.
Ignore explicit calls to php_stream_close() entirely to avoid
use-after-free -- instead let the stream resource destructor
deal with it. We have to account for a few special cases:

 * Enclosed streams should be freed, as the resource destructor
   will forward to the enclosing stream.
 * Stream cookies also directly free streams, because we delegate
   to the cookie destruction if one exists.
 * Mysqlnd also directly frees streams, because it explicitly
   removes stream resources (because mysqlnd!)

Merge branch 'PHP-7.4'

* PHP-7.4:
  Clean headers checks

Clean headers checks

Some headers were checked multiple times in the main configure.ac file
and in the bundled extensions or SAPIs themselves. Also many of these
checks are then used accross other extensions or SAPIs so a central
configure.ac makes most sense for these checks.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Avoid double checks on early binding

Avoid double checks on early binding

Merge branch 'PHP-7.4'

* PHP-7.4:
  Remove warnings for bison and re2c checks

Remove warnings for bison and re2c checks

This patch removes warnings when lexer and parser files are already
generated. For example, when downloading a PHP release. Realistically,
it is not so trivial to rebuild lexer and parser files without removing
them first. And considering that tarballs don't have cleaning tools
such as Git, this would require manual removali.

Related also to https://bugs.gentoo.org/593278

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix concurrent testing of bug 61964

Fix concurrent testing of bug 61964

When tests are run concurrently using the -j option there might be a
change of failure and writing to the same directory from both tests.

Use test filenames for generated temp dirs

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix posix_errno test without PCNTL dep

Fix posix_errno test without PCNTL dep

When PCNTL extension is not enabled, the SIGKILL constant is also not
available.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Remove unused HAVE_DATE and HAVE_SPL symbols

Remove unused HAVE_DATE and HAVE_SPL symbols

These two are never defined anymore since PHP 7.4 and can be
considered to have always available date and hash extensions.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Remove HAVE_STRTOD and strtod check

Remove HAVE_STRTOD and strtod check

There is no need to check for the strtod function. It is part of C89
standard which PHP-7.4+ supports.

http://port70.net/~nsz/c/c89/c89-draft.html#4.10.1.4

Merge branch 'PHP-7.4'

* PHP-7.4:
  Set Computer English language for credits sorting

Set Computer English language for credits sorting

This makes the script a bit more portable when used on systems with
different LC_ALL and LANG settings.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Remove stream_socket_sendto.phpt

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Remove stream_socket_sendto.phpt

Merge branch 'PHP-7.2' into PHP-7.3

* PHP-7.2:
  Remove stream_socket_sendto.phpt

Remove stream_socket_sendto.phpt

Fails for me locally due to different number of warnings with
different messages. Rather than adding more wildcards I'm dropping
this test entirely, as it doesn't seem to test anything particularly
useful.

(cherry picked from commit 84333cad674890174c47f3c21b1b1cf85a4276ac)

Merge branch 'PHP-7.4'

* PHP-7.4:
  Private methods don't have to be duplicated

Private methods don't have to be duplicated

Merge branch 'PHP-7.4'

Fix xml doc leak in soap

Use ldap_destroy instead of ldap_unbind_ext

Fix fpm limit_extensions leak

The logic here is somewhat ugly because the worker pool structure
gets freed early, so we need to explicitly transfer ownership.

[ci skip] Mention HAVE_HASH_EXT

Merge branch 'PHP-7.4'

* PHP-7.4:
  Reorder conditions to minimize number of checks on fast path

Reorder conditions to minimize number of checks on fast path

Merge branch 'PHP-7.4'

* PHP-7.4:
  Remove always true/false conditions, remove dead conde and simplify code.

Remove always true/false conditions, remove dead conde and simplify code.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Replace previosly checked conditions by ZEND_ASSERT()

Replace previosly checked conditions by ZEND_ASSERT()

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fixed variance check for abstract constructor during erlay binding

Fixed variance check for abstract constructor during erlay binding

Merge branch 'PHP-7.4'

* PHP-7.4:
  Cleanup

Cleanup

Merge branch 'PHP-7.4'

FFI: Perform bitfield operations byte-wise

Otherwise we may perform reads/writes outside the allocation, as
already happens in 032.phpt.

Fix out of bounds write in phpdbg

It seems that this code has a peculiar interpretation of "len",
where it actually points to the last character, not one past it.
So we need +1 here for that extra char and another +1 for the
terminating null byte.

Fix CURLINFO_COOKIELIST leak

Fix leak on sqlite3 open error

sqlite3_open creates the database object even if the operation
fails.

Fix SSL_CTX leak in ftp extension

SSL_CTX is a refcounted structure, which will be held by the SSL
handle, so we can free it here.

Fix UConverter leak

Fix TimeZone leak in intl MessageFormat

I'm just giving each format a distinct owned object here ... sharing
it looks complicated.