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

Fix #65274: Add class name to undef class constant error

Add warning and convert to exception in string offset assignment:

Convert the empty string assignment to an Error as per RFC [1]
Add a warning that only the first byte will be assigned to the offset if provided
a needle that is longer than one byte.

[1] https://wiki.php.net/rfc/engine_warnings

Closes GH-5063

Allow empty needles in mb_strpos and mb_strstr function family.

MBstring analogous implementation to 6d578482a933be7597b686b59a935b316161d251

Closes GH-4977

Add warning and convert to exception in string offset assignment:

Convert the empty string assignment to an Error as per RFC [1]
Add a warning that only the first byte will be assigned to the offset if provided
a needle that is longer than one byte.

[1] https://wiki.php.net/rfc/engine_warnings

Merge branch 'PHP-7.4'

* PHP-7.4:
  Make test more resilient

Make test more resilient

Depending on the libcurl version and perhaps configuration, it may show
additional info (due to `CURLOPT_VERBOSE` being activated), which we
have to ignore, to avoid spurious test failures.

Renumber zval types, clarify allowed overlap

Make it clear that types used for type declarations can overlap
with the rest, and can also overlap in MAY_BE space.

This makes things more robust against the addition of new primitive
types.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Increase sleep() time in test case

Increase sleep() time in test case

Apparently, the former increase to `sleep(2)` helped somewhat, but
still the test fails occassionally.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Bump version

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Bump version

Bump version

Merge branch 'PHP-7.4'

* PHP-7.4:
  Remove support for preloading on Windows

Remove support for preloading on Windows

Due to ASLR restrictions, preloading on Windows does not work with
any code that has preloading dependencies on internal classes.
This effectively makes it unusable for any non-trivial codebase.

Instead of pretending like preloading is going to work, only to
make people realize that it really doesn't once they get beyond
a dummy example, we disable support for preloading on Windows
entirely.

Closes GH-4999.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Throw Error when referencing uninit typed prop in __sleep

Throw Error when referencing uninit typed prop in __sleep

Previously this generated a notice, but would likely generate an
Error when unserializing.

Now we treat it with the same distinction as direct property
accesses, i.e. referencing an unset/undefined normal property
stays a notice, while a typed property becomes an Error exception.

This fixed bug #79002.

Closes GH-5050.

[ci skip] Merge branch 'PHP-7.4'

Empty merge.

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

Empty merge.

Revert "Extend CURLFile to support streams"

This reverts commit 17a9f1401aeb35fe1e3657b38102a410d151d42f, because
this commit would break ABI, and also due to bug #79013.
We keep the commit for PHP 7.4+, though.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix #54298: Using empty additional_headers adding extraneous CRLF

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #54298: Using empty additional_headers adding extraneous CRLF

Fix #54298: Using empty additional_headers adding extraneous CRLF

If the header string is empty, we pass `NULL` to `php_mail()` to avoid
further checks on the string length.

Rename skeleton stub file

Add the .php suffix

Fix hypothetical segfault in gdTransformAffineCopy()

This has been reported to upstream[1], but since ext/gd never calls
`gdTransformAffineCopy()` for a palette image, it is not a bug for PHP.
Therefore we apply the fix to master only.

[1] <https://github.com/libgd/libgd/issues/586>

Merge branch 'PHP-7.4'

* PHP-7.4:
  Don't link against openssl 1.1 in curl