Add CLEAN sections to mysqli and PDO mysql tests

Closes GH-6756.

Merge branch 'PHP-8.0'

* PHP-8.0:
  ext/opcache: fix configure output while checking mmap MAP_ANON support

ext/opcache: fix configure output while checking mmap MAP_ANON support

It seems that f3efb9e3fb introduced a "typo" which may result
in the following confusing message:

checking for mmap() using MAP_ANON shared memory support... no=yes

Let's fix this.

Signed-off-by: Michael Heimpold <mhei@heimpold.de>
Closes GH-6758.

Deprecate mysqli driver_version property

The driver version hasn't been updated in 13 years, so this number
is essentially meaningless. Check PHP_VERSION_ID instead.

Remove dead mysqli_driver_construct() function

This function was never registered with PHP, and the body is
commented out.

Clean up mysqli_driver test cases

And remove an unnecessary test case that is already covered by the
other two.

Remove redundant macros in mysqli_driver implementation

This class doesn't define many properties, and the macros make
customization hard.

Remove unused mysqli global

And fix formatting issues.

client_info is a constant and doesn't need a connection

Closes GH-6767.

Use zend_string* & more legible API for php_get_display_errors_mode()

Return early on php display error for better legibility

Closes GH-6760

Signed-off-by: George Peter Banyard <girgias@php.net>

hash: xxhash: Use canonicalization routine from existing API

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

Remove deprecated code

Fixed compilation warning

Inline "array" part of FE_FETCH_R handler into HYBRID VM

zend_verify_recv_arg_type_helper is not "cold".

Avoid repeatable work when error_reporting() is called with the same argument few times.

Inheritance cache optimization

Fixed assertion (ext/opcache/zend_persist.c:327: zend_accel_get_type_map_ptr: Assertion `ret > 2' failed)

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fixed bug #80814 (threaded mod_php won't load on FreeBSD: No space available for static Thread Local Storage)

Fixed bug #80814 (threaded mod_php won't load on FreeBSD: No space available for static Thread Local Storage)

Use spl_filesystem_object.file_name for SPL_FS_DIR as a cache and prevent multiple file name reconstruction.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fix #51903: simplexml_load_file() doesn't use HTTP headers

Merge branch 'PHP-7.4' into PHP-8.0

* PHP-7.4:
  Fix #51903: simplexml_load_file() doesn't use HTTP headers

Fix #51903: simplexml_load_file() doesn't use HTTP headers

The `encoding` attribute of the XML declaration is optional; it is good
practice to use external encoding information where available if it is
missing.  Thus, we check for `charset` info of `Content-Type` headers,
and see whether the encoding is supported.

We cater to trailing parameters and quoted-strings, but not to escaped
backslashes and quotes in quoted-strings, since no known character
encoding contains these anyway.

Co-authored-by: Michael Wallner <mike@php.net>
Closes GH-6747.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fix #80838: HTTP wrapper waits for HTTP 1 response after HTTP 101

Merge branch 'PHP-7.4' into PHP-8.0

* PHP-7.4:
  Fix #80838: HTTP wrapper waits for HTTP 1 response after HTTP 101

Fix #80838: HTTP wrapper waits for HTTP 1 response after HTTP 101

Don't wait for further responses after a HTTP 101 (Switching Protocols) response

Closes GH-6730.

Merge branch 'PHP-8.0'

Fix the error message of attribute flag validation

Merge branch 'PHP-8.0'

Merge branch 'PHP-7.4' into PHP-8.0

Merge branch 'PHP-7.4' of git.php.net:php-src into PHP-7.4

LiteSpeed SAPI v7.9: Process manager keeps forked child process alive for longer time based on load. Reduces PM overhead for busy setup.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Add supports for FreeBSD's PROT_MAX to let mprotect knows X flag can be applied in addition.

Add supports for FreeBSD's PROT_MAX to let mprotect knows X flag can be applied in addition.

Closes GH-6738.

Reduce date test threshold further

Still seeing the occasional spurious failure.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Print error code if CreateMutex() fails

Merge branch 'PHP-7.4' into PHP-8.0

* PHP-7.4:
  Print error code if CreateMutex() fails

Print error code if CreateMutex() fails

This issue came up recently in a bug report[1]; without the error code,
users can barely guess why the function failed.

[1] <https://bugs.php.net/80812>

Closes GH-6745.

typo

Reduce threshold in ext/date/tests/bug73837.phpt

Now that this code is faster, we're seeing many spurious failures.

Update mysqli tests to work with newer MySQL server

* @@session.old_passwords is present only in 5.6 and 5.7. Newer versions
  already behave as if they have old_passwords=2.
* SET PASSWORD FOR ... = PASSWORD(...) syntax was removed in 8.0.

For all affected tests, versions that don't support newer SQL are
already explicitly excluded with version checks.

Closes GH-6729.

Make the nocache tag actually work

If nocache is used, then we should return an empty string result,
so the test doesn't get marked as borked.

Also limit to an occurrence at the start, just like other skip
tags.

Add mechanism to skip the skipif cache

If the SKIPIF section return nocache, don't cache the result. This
is used by tests that perform complex test set in conjunction with
skip checks, and need to be evaluated every time.

Fix dynamic func def persist with preloading

If we find an existing serialization of the op_array, of course
we also need to actually make use of it...

run-tests.php: Preserve .php on leak in one more place

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fix ASAN warning (Zend/zend_string.h:165:2: runtime error: null pointer passed as argument 2 of memcpy(), which is declared to never be null)

Fix ASAN warning (Zend/zend_string.h:165:2: runtime error: null pointer passed as argument 2 of memcpy(), which is declared to never be null)

Cache haschildren/getchildren methods of recursive iterators.

Remove duplicated SSL_CTX_set_verify()

Duplicated with line 920.
Our minimal OpenSSL version is v1.0.1 (See https://github.com/openssl/openssl/blob/OpenSSL_1_0_1-stable/ssl/ssl_lib.c#L2039).
Removing it does not affect program behavior.
Closes GH-6751.

Avoid useless date conversion

Explicitly print reference wrappers in debug_zval_dump()

Closes GH-6750.

Switch to new ZPP

Switch to new ZPP

Loop invariant code motion

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fix #80825: ZipArchive::isCompressionMethodSupported does not exist

Fix #80825: ZipArchive::isCompressionMethodSupported does not exist

`HAVE_METHOD_SUPPORTED` should have been defined after we updated to
libzip 1.7.1 months ago.

Add test cases for bcmath ValueErrors

Closes GH-6714

Signed-off-by: George Peter Banyard <girgias@php.net>

Replaced deprecated `hw.ncpu` with `hm. logicalcpu`

Found that at ziglang/zig#1252.

Closes GH-5541

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fix test wrt. server.inc changes

Fix test wrt. server.inc changes

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fix #78719: http wrapper silently ignores long Location headers

Merge branch 'PHP-7.4' into PHP-8.0

* PHP-7.4:
  Fix #78719: http wrapper silently ignores long Location headers

Fix #78719: http wrapper silently ignores long Location headers

When opening HTTP streams, and reading the headers, we currently
discard header lines longer than `HTTP_HEADER_BLOCK_SIZE` (1024 bytes).
While this is not generally forbidden by RFC 7230, section 3.2.5, it
is not generally allowed either, since that may change the "message
framing or response semantics".

We thus fix this by allowing arbitrarily long header lines.

Closes GH-6720.

Fix bug #80816 Document the removal of alias class entries from ext/spl

Closes GH-6748 [skip-ci]

Add const to zend_extension_version_info.build_id

This is a follow-up to:
https://github.com/php/php-src/commit/3c6ab4b3ea5854fd44a2945bb65bf027a6c94890

Closes GH-6744.

Fixed bug #80808

If the ZEROFILL flag is set for a field, do not convert it into
an integer (text protocol) or convert it explicitly into a padded
string (binary protocol).

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fixed bug #80802: (zend_jit_fetch_indirect_var assert failure with tracing JIT)

Fixed bug #80802: (zend_jit_fetch_indirect_var assert failure with tracing JIT)

Optimized object conversion to array without rebulding properties HashTable

Optimized object encoding without rebulding properties HashTable

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fix #80751: Comma in recipient name breaks email delivery

Merge branch 'PHP-7.4' into PHP-8.0

* PHP-7.4:
  Fix #80751: Comma in recipient name breaks email delivery

Fix #80751: Comma in recipient name breaks email delivery

So far, `SendText()` simply separates potential email address lists at
any comma, disregarding that commas inside a quoted-string do not
delimit addresses.  We fix that by introducing an own variant of
`strtok_r()` which caters to quoted-strings.

We also make `FormatEmailAddress()` aware of quoted strings.

We do not cater to email address comments, and potentially other quirks
of RFC 5322 email addresses, but catering to quoted-strings is supposed
to solve almost all practical use cases.

Co-authored-by: Nikita Popov <nikita.ppv@gmail.com>
Closes GH-6735.

Declare XMLReader properties

Closes GH-6741

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fixed bug #80811

Fixed bug #80811

When filling in defaults for skipped params, make sure that
reference parameters get the expected reference wrapper.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Always remove HT iterators, even for uninit HT

Merge branch 'PHP-7.4' into PHP-8.0

* PHP-7.4:
  Always remove HT iterators, even for uninit HT

Always remove HT iterators, even for uninit HT

Fixes oss-fuzz #31423.

Optimize serializing class names

Because of the memcpy, compilers can't infer that ZSTR_LEN (i.e. class_name->len)
did not change, so they copy it out of memory into a register for the last two
accesses.
php_var_serialize_string already does something similar.

Closes GH-6734

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fixed bug #80805

Merge branch 'PHP-7.4' into PHP-8.0

* PHP-7.4:
  Fixed bug #80805

Fixed bug #80805

Handle missing result_var in binary_op_result_type.

(cherry picked from commit 8446e2827585c37d0739f8d44fa8d359cbbb6551)

Optimized object serialization without rebulding properties HashTable

Incomplete class may have only single "MAGIC_MEMBER"

Change the order of properties used in foreach(), var_dump(), serialize(), comparison, etc. Now properties are ordered according to their layout in zend_object structure.

Added UPGRADING note.

Reference dynamic functions through dynamic_defs

Currently, dynamically declared functions and closures are inserted
into the function table under a runtime definition key, and then later
possibly renamed. When opcache is not used and a file containing a
closure is repeatedly included, this leads to a very large memory leak,
as the no longer needed closure declarations will never be freed
(https://bugs.php.net/bug.php?id=76982).

With this patch, dynamic functions are instead stored in a
dynamic_func_defs member on the op_array, which opcodes reference
by index. When the parent op_array is destroyed, the dynamic_func_defs
it contains are also destroyed (unless they are stilled used elsewhere,
e.g. because they have been bound, or are used by a live closure). This
resolves the fundamental part of the leak, though doesn't completely
fix it yet due to some arena allocations.

The main non-obvious change here is to static variable handling:
We can't destroy static_variables_ptr in destroy_op_array, as e.g.
that would clear the static variables in a dynamic function when
the op_array containing it is destroyed. Static variable destruction
is separated out for this reason (we already do static variable
destruction separately for normal functions, so we only need to
handle main scripts).

Closes GH-5595.

Change the order of properties used for var_dump(), serialize(), comparison, etc.
Now properties are ordered according to their layout in zend_object structure.

Fix E_DEPRECATED in zend_vm_gen.php

explode(): Passing null to parameter #2 ($string) of type string is
deprecated

Closes GH-6698.

run-tests: drop support for ancient Valgrind versions

Valgrind 3.3.0 was released in 2007, not even RHEL has crap older than
this. It could be argued that 3.8.0, released in 2012, could be a safe
cutoff too.

Closes GH-6728.

Clarify types in XmlReader property handling

Make it clearer that the specified type is really the only type
that can be returned.

Remove useless conditional #ifdef

Closes GH-6737

Signed-off-by: George Peter Banyard <girgias@php.net>

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fix Bug #80800 imap_open() fails when the flags parameter includes CL_EXPUNGE

Fix Bug #80800 imap_open() fails when the flags parameter includes CL_EXPUNGE

This also affected imap_reopen().
Add a supplementary test that the CL_EXPUNGE flag does have
the intended effect.

Closes GH-6732

serialize() optimization

Optimize out zend_strpprintf() usage for simple concationaton

Improve SPL directory and stat() cache using zend_srting* instead of char*