Merge branch 'PHP-7.4'

* PHP-7.4:
  Increase timeout in mysqli_reap_async_query test

Increase timeout in mysqli_reap_async_query test

The 2000 here is 2ms, which is a bit optimistic for parallelized
testing.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Let's make this safer from maliciously crafted filename

Let's make this safer from maliciously crafted filename

Merge branch 'PHP-7.4'

* PHP-7.4:
  minor cleanup

minor cleanup

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fixed bug #79114 (Eval class during preload causes class to be only half available)

Fixed bug #79114 (Eval class during preload causes class to be only half available)

Fix #78880 Another round

Bump required libcurl version to 7.29.0

libcurl 7.29.0 has been released almost eight years ago, so this
version is supposed to be available practically everywhere.  This bump
also allows us to get rid of quite some conditional code and tests
catering to very old libcurl versions.

Remove leftover ZEND_CAST code for (unset) cast.

Followup for d74d3922ce6f9ea07c1226b0cb2a94fc333f7a02

Attempting to require a file with (unset) casts results in an E_COMPILE_ERROR
that can't be caught or handled by set_exception_handler/set_error_handler.

Also remove the (bool) cast, because the ZEND_BOOL opcode handles that.

Remove inference that array -> object cast can throw.
It was added in 2a286ad5991 - I don't know how creating an stdClass would throw.
(numeric keys, references, etc. don't cause it to throw)

Closes GH-5042

Make error messages more consistent by fixing capitalization

Closes GH-5066 As a first step, let's capitalize their initial letter when it is applicable.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fixed bug #79115

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fixed bug #79115

Fixed bug #79115

Add support for deprecating constants

Internal constants can be marked as CONST_DEPRECATED, in which
case accessing them will throw a deprecation warning.

For now this is only supported on global constants, not class
constants. Complain to me if you need to deprecate a class
constant...

Closes GH-5072.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Update php.ini files to add missing FTP extension

Update php.ini files to add missing FTP extension

FTP functionality isn't enabled by default and not listed as a disabled
extension in the `php.ini` file either.

This should spare a few googles to anyone needing FTP functionality in PHP.

Closes GH-5084.

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.

I believe I already tried to land this before once, but it didn't
actually end up on the PHP-7.3 branch due to a push conflict that
I only noticed just now.

Improve SSA representation of FE_FETCH

The op2 of FE_FETCH is only written if the loop edge is taken.
Fix up the SSA form to use the pre-assignment value if the exit
edge is taken.

This allows us to properly infer the type of the loop variable,
without letting the pre-loop type leak in.

Closes GH-5040.

Use zend_type inside type lists

Instead of having a completely independent encoding for type list
entries. This is going to use more memory, but I'm not particularly
concerned about that, as type unions that contain multiple classes
should be uncommon. On the other hand, this allows us to treat
top-level types and types inside lists mostly the same.

A new ZEND_TYPE_FOREACH macros allows to transparently treat list
and non-list types the same way. I'm not using it everywhere it could be
used for now, just the places that seemed most obvious.

Of course, this will make any future type system changes much simpler,
as it will not be necessary to duplicate all logic two times.

Fix #78880: Final spelling fixes

Fix reflection leak if type inside type list is resolved

Merge branch 'PHP-7.4'

* PHP-7.4:
  Disable optimizations for ASan instrumented builds

Disable optimizations for ASan instrumented builds

ASan instrumentation does not support the MSVC debug runtime, but still
it does not make sense to enable optimizations for such builds, since
they are not meant for production usage anyway, and although memory
corruption issues are still found in optimized builds, the generated
diagnostics are close to being useless, and apparently sometimes even
outright wrong.  Therefore, we disable all optimizations for ASan
instrumented builds.

We also introduce and use `ZEND_WIN32_NEVER_INLINE` for ASan enabled
builds to avoid inlining of functions, so we get even better
diagnostics.

Fix #78880: Yet another batch of spelling errors

Fix #78880: Another bunch of spelling errors

Remove useless skip check

As of PHP 5.4.0, `detect_unicode` is always available, so this check
cannot fail.

Use proper printf() specifiers

Remove va_copy from zend_portability as it's always available since C99

Rename run-test.php to run-tests.php in the test packs

In the php-src repository, the test runner is named run-tests.php, but
when it is copied to the tests packs, it is renamed to run-test.php.
This renaming does not make sense, and is actually somewhat confusing.
Although changing the name back to run-tests.php constitutes a BC
break, we think the benefit of having a single name outweights the
disadvantages in the long run.

Add imagegetinterpolation()

While `imagesetinterpolation()` is available as of PHP 5.5.0,
there is no according getter function, so users would have to track the
current interpolation method manually.

To remedy this, we introduce `imagegetinterpolation()` as thin wrapper
for `gdImageGetInterpolationMethod()` (which has been introduced with
libgd 2.1.1), and use `im->interpolation_id` as fallback for older
libgd.  Since our bundled libgd does not yet have this function, we add
it.

We also simplify the recently introduced bug79068.phpt, where it is
sufficient to check that the interpolation method has not been changed.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix #79096: FFI Struct Segfault

Fix #79096: FFI Struct Segfault

We must not assume that the size of a function's return value is at
most `sizeof(ffi_arg)`, but rather have to use the size which already
has been determined for the return type if it is larger than
`sizeof(ffi_arg)`.

To be able to have a regression test, we export the required test
function from the zend-test extension, and make sure that the test
can be run on different platforms regardless of whether zend-tests was
built statically or dynamically.

Inline hot parts of bitwise instructions into hybrid VM

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix #79106: PDO may fetch wrong column indexes with PDO::FETCH_BOTH

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #79106: PDO may fetch wrong column indexes with PDO::FETCH_BOTH

Fix #79106: PDO may fetch wrong column indexes with PDO::FETCH_BOTH

Column names can be numeric strings, so we have to make sure to insert
the column values with the appropriate numeric keys, instead of adding
them.

typo

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix #79084: mysqlnd may fetch wrong column indexes with MYSQLI_BOTH

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #79084: mysqlnd may fetch wrong column indexes with MYSQLI_BOTH

Fix #79084: mysqlnd may fetch wrong column indexes with MYSQLI_BOTH

Column names can be numeric strings, so we have to make sure to insert
the column values with the appropriate numeric keys, instead of adding
them.

Try to fix ZTS build

Use helper in one more place

Fix build

Improve access to zend_map_ptr data

Handle one more case I missed

Make class name references use the class_name production

Throw a compile error for "static" references instead, where it
isn't already the case.

Also extract the code that does that -- we have quite a few places
where we get a const class ref and require it to be default.

Don't use yystrlen()

This is unnecessary and not available when GLR is used.

Fixed #78385: Distinguish absent/empty query/fragment

http://example.com/foo   => query = null, fragment = null
http://example.com/foo?  => query = "",   fragment = null
http://example.com/foo#  => query = null, fragment = ""
http://example.com/foo?# => query = "",   fragment = ""

Closes GH-5078.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix test

Fix test

Hopefully (32-bit)

Merge branch 'PHP-7.4'

* PHP-7.4:
  Bring back test case support for older MySQL versions

Bring back test case support for older MySQL versions

These test cases have recently been adjusted to work with MySQL 8[1],
but some older, but still supported database versions, such as MySQL
5.6, still need the password to be sent hashed, so we fall back to
using `PASSWORD()`, if the `SET PASSWORD` query fails without it.

[1] <http://git.php.net/?p=php-src.git;a=commit;h=b0efd18f7844da29931737b8a1cf461c7493e168>.

Remove dead code and unused DEFINEs

Closes GH-5073

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix test cases which fail on Windows debug builds

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix test cases which fail on Windows debug builds

Fix test cases which fail on Windows debug builds

We use the portable {TMP} instead of the hard-coded /tmp, and skip
mysqli_debug_append.phpt on Windows, because unlinking the trace file
while the connection is still open won't work there.

(cherry picked from commit 60081ca20d5701111de5f94ae64909b1cc265f1e)

Merge branch 'PHP-7.4'

* PHP-7.4:
  Remove unnecessary whitespace sensitivtiy from some tests

Remove unnecessary whitespace sensitivtiy from some tests

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix test for Windows ZTS builds

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix test for Windows ZTS builds

Fix test for Windows ZTS builds

Cf. <https://bugs.php.net/bug.php?id=61685#1578672905>.

Make constant() error handling consistent with plain const lookup

This means we get an Error exception and a much better error
message indicating the root cause (e.g. accessing a private class
constant).

Move undefined constant error into get_constant_ex

All the other error conditions are already handled in there, so
this one should be as well.

Remove unused delimiter parameter in php_fgetcsv_lookup_trailing_spaces subroutine.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Silence potential taskkill error messages

Silence potential taskkill error messages

That test is still intermittently failing, because failure to kill the
child process is reported.  Therefore we silence these error messages.

Convert infallible mysqli checks to asserts

Remove unused MYSQLI_STATUS_CLEARED

Fix test

The typo in the echoed string had been fixed in master; fix the
expectation as well.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix useless tests

Fix useless tests

These tests are certainly not supposed to fail due to a typo in the
class names, but expect this failure.  We fix the typos and the
expectations.

Allows further tests to run on Windows

Use IS_WINDOWS

Replace fakemail with minimal PHP script

Unify mail related tests for *nix and Windows

Currently mail related tests are split for *nix and Windows (if there
are even Windows versions).  The basic difference is that the *nix
variants set the INI directive sendmail_path to just write the email to
disk, while the Windows tests use ext/imap.  The latter tests are way
more verbose, and such duplicated tests are generally a pain point.
Furthermore, the Windows tests are much slower, and could not be run
without ext/imap being available.

We therefore introduce a small fakemail application, which basically
works like `tee <path> >/dev/null`, and which will be shipped with the
Windows tests packs.  fakemail.exe would also need to be added to the
PHP binary SDK, so these tests could be run during developments.

To cater to the remaining differences, we also introduce support for
`{MAIL:<path>}` placeholders in the INI sections to run-tests.php.  How
to use this can be seen in mail_basic.phpt, which is currently the only
modified test case, because these tests are yet supposed to fail on
Windows, due to the missing fakemail.exe in the PHP SDK.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Increase select timeout in FPM tester

Increase select timeout in FPM tester

Let's see if that helps with the recent failure spree on Azure.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix #79086: Bump version not applied

Fix #79086: Bump version not applied

We must not forget to update the version in configure.ac as well.

Convert some warnings to Errors in BZip2

Closes GH-4984

Convert some warnings to Error in zlib extension

Closes GH-4985

[ci skip] Update UPGRADING

Avoid superfluous assignments

These values are already set via the initializers.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix #79078: Hypothetical use-after-free in curl_multi_add_handle()

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #79078: Hypothetical use-after-free in curl_multi_add_handle()

Fix #79078: Hypothetical use-after-free in curl_multi_add_handle()

To avoid this, we have to verify the handlers already in
`curl_multi_add_handle()`, not only in `curl_multi_exec()`.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Bump version

Bump version

Merge branch 'PHP-7.4'

* PHP-7.4:
  Extract function

Extract function

As suggested by Nikita[1].

[1] <https://github.com/php/php-src/pull/5045#discussion_r364265013>

Merge branch 'PHP-7.4'

* PHP-7.4:
  Revert "Display a message if select in FPM test timeouts"

Revert "Display a message if select in FPM test timeouts"

This reverts commit e2361498d519561e7eb5b73d138c1eaa80da2a20.

Ooops, this occurs normally during some tests, but I didn't notice
because I have slow tests disabled...

Merge branch 'PHP-7.4'

* PHP-7.4:
  Display a message if select in FPM test timeouts

Display a message if select in FPM test timeouts

Fix #49555: Improve "function must be a string" error message

Be more specific for the individual cases and provide relevant
type information.

Refer to method rather than function name in error message