Merge branch 'PHP-8.0'

* PHP-8.0:
  Fix variance checks on resolved union types

Fix variance checks on resolved union types

This is a bit annoying: When preloading is used, types might be
resolved during inheritance checks, so we need to deal with CE
types rather than just NAME types everywhere.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Don't ignore internal classes during preloading

Don't ignore internal classes during preloading

When preloading, it's fine to make use of internal class information,
as we do not support Windows. It is also necessary to allow proper
variance checks against internal classes.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fix persisting property info table with internal parent

Fix persisting property info table with internal parent

If the property info comes from an internal parent, we won't have
an xlat entry for it. Leave it alone in that case.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Preserve trait method alias name during preloading

Preserve trait method alias name during preloading

Modernize IMAP tests

Use constants when appropriate, drop some ouputs which depends on env vars

Enable parallel testing.

Closes GH-6380

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fix another implicit function declaration in configure

Merge branch 'PHP-7.4' into PHP-8.0

* PHP-7.4:
  Fix another implicit function declaration in configure

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix another implicit function declaration in configure

Fix another implicit function declaration in configure

As mentioned on bug #80171. This one is in libtool.m4, might get
lost on libtool updates.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fix typo in UPGRADING
  Improved error message for typed class properties with null as default value

Fix typo in UPGRADING

Closes GH-6395.

Improved error message for typed class properties with null as default value

Closes GH-6396.

Add test suite for ARMSCII-8 encoding

Fix mbstring support for ARMSCII-8

- Identify filter was completely wrong.
- Respect `mb_substitute_character` rather than converting invalid bytes to
  Unicode 0xFFFD (generic replacement character).
- Don't convert Unicode 0xFFFD to a valid ARMSCII-8 character.
- When converting ARMSCII-8 to ARMSCII-8, don't pass invalid bytes through
  silently.

Optimize (AND FIX) mb_check_encoding (cut execution time by 50%+)

Previously, `mb_check_encoding` did an awful lot of unneeded work. In order to
determine whether a string was valid or not, it would convert the whole string
into wchar (code points), which required dynamically allocating a (potentially
large) buffer. Then it would turn right around and convert that big 'ol buffer
of code points back to the original encoding again. Finally, it would check
whether any invalid bytes were detected during that long and onerous process.

The thing is, mbstring _already_ has machinery for detecting whether a string
is valid in a certain encoding or not, and it doesn't require copying any data
around or allocating buffers. Better yet, it can fail fast when an invalid byte
is found. Why not use it? It's sure a lot faster!

Further, the legacy code was also badly broken. Why? Because aside from
checking whether illegal characters were detected, it would also check whether
the conversion to and from wchars was lossless. But, some encodings have
more than one valid encoding for the same character. In such cases, it is
not possible to make the conversion to and from wchars lossless for every
valid character. So `mb_check_encoding` would actually reject good strings
in a lot of encodings!

Add test suite for KOI8-U encoding

Remove dead code from mbfilter_koi8u.c (and do general code cleanup)

All bytes are valid in KOI8-U encoding

Add test suite for KOI8-R encoding

Remove dead code from mbfilter_iso8859_{2,4,5,9,10,13,14,15,16}.c

...Plus some dead code related to ISO-8859-1.

Remove dead code from mbfilter_koi8r.c

All bytes are valid in KOI8-R encoding

Add test suite for CP850 encoding

Remove dead code from mbfilter_cp850.c (and do general code cleanup)

Since there are no invalid bytes in CP850, these `if` conditions will never
be true.

All bytes are valid in CP850 encoding

Add test suite for CP866 encoding

Remove dead code from mbfilter_cp866.c (and do general code cleanup)

Since there are no invalid bytes in CP866, these `if` conditions will never
be true.

All bytes are valid in CP866 encoding

Add test suite for CP1254 encoding

Fix mbstring support for CP1254 encoding

One funny thing: while the original author used Unicode 0xFFFD (generic
replacement character) for invalid bytes in CP1251 and CP1252, for CP1254
they used 0xFFFE, which is not a valid Unicode codepoint at all, but is a
reversed byte-order mark. Probably this was by mistake.

Anyways,

- Fixed identify filter, which was completely wrong.
- Don't convert Unicode 0xFFFE to a random (but valid) CP1254 byte.
- When converting CP1254 to CP1254, don't pass invalid bytes through silently.

Add test suite for CP1251 encoding

Fix mbstring support for CP1251 encoding

- Identify filter was as wrong as wrong can be.
- Invalid CP1251 byte 0x98 was converted to Unicode 0xFFFD (generic
  replacement character), rather than respecting `mb_substitute_character`.
- Unicode 0xFFFD was converted to some random CP1251 byte.
- When converting CP1251 to CP1251, don't pass invalid bytes through silently.

Test cases for mbstring encodings are less repetitive

Merge branch 'PHP-8.0'

* PHP-8.0:
  [ci skip] Fix small typo

[ci skip] Fix small typo

`as` -> `was`

Closes GH-6394.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Rename opcache.jit_max_loops_unroll to opcache.jit_max_loop_unrolls

Rename opcache.jit_max_loops_unroll to opcache.jit_max_loop_unrolls

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fix #70461: disable md5 code when it is not supported in net-snmp

Merge branch 'PHP-7.4' into PHP-8.0

* PHP-7.4:
  Fix #70461: disable md5 code when it is not supported in net-snmp

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #70461: disable md5 code when it is not supported in net-snmp

Fix #70461: disable md5 code when it is not supported in net-snmp

Patch contributed by Alexander Bergmann.

Closes GH-6389.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fix tests for MariaDB

Fix tests for MariaDB

Closes GH-6390.

hash: murmur: Fix GCC support version for no_sanitize

While AddressSanitizer is supported as of 4.8, the no_sanitize attribute
appears in 8.0.

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

hash: murmur: Suppress sanitize warnings under GCC

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

Bump minimal OpenSSL version to 1.0.2

UPGRADING: Add murmurhash note [ci skip]

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

NEWS: Add murmurhash info [ci skip]

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

hash: Add MurmurHash3 with streaming support

The implementation is based on the upstream PMurHash. The following
variants are implemented

- murmur3a, 32-bit hash
- murmur3c, 128-bit hash for x86
- murmur3f, 128-bit hash for x64

The custom seed support is not targeted by this implementation. It will
need a major change to the API, so then custom arguments can be passed
through `hash_init`. For now, the starting hash is always zero.

Fixes bug #68109, closes #6059

Signed-off-by: Anatol Belski <ab@php.net>
Co-Developed-by: Michael Wallner <mike@php.net>
Signed-off-by: Michael Wallner <mike@php.net>

Add test suite for CP1252 encoding

Also remove a bogus test (bug62545.phpt) which wrongly assumed that all invalid
characters in CP1251 and CP1252 should map to Unicode 0xFFFD (REPLACEMENT
CHARACTER).

mbstring has an interface to specify what invalid characters should be
replaced with; it's called `mb_substitute_character`. If a user wants to see
the Unicode 'replacement character', they can specify that using
`mb_substitute_character`. But if they specify something else, we should
follow that.

Fix mbstring support for CP1252 encoding

It's a bit surprising how much was broken here.

- Identify filter was utterly and completely wrong.
- Instead of handling invalid CP1252 bytes as specified by
  `mb_substitute_character`, it would convert them to Unicode 0xFFFD
  (generic replacement character).
- When converting ISO-8859-1 to CP1252, invalid ISO-8859-1 bytes would
  be passed through silently.
- Unicode codepoints from 0x80-0x9F were converted to CP1252 bytes 0x80-0x9F,
  which is wrong.
- Unicode codepoint 0xFFFD was converted to CP1252 0x9F, which is very wrong.

Also clean up some unneeded code, and make the conversion table consistent with
others by using zero as a 'invalid' marker, rather than 0xFFFD.

Merge branch 'PHP-8.0'

Fix test added in d44235acaeb4

Merge branch 'PHP-8.0'

Convert numeric string array keys to integers correctly in JITted code

While fixing bugs in mbstring, one of my new test cases failed with a strange
error message stating: 'Warning: Undefined array key 1...', when clearly the
array key had been set properly.

GDB'd that sucker and found that JIT'd PHP code was calling directly into
`zend_hash_add_new` (which was not converting the numeric string key to an
integer properly). But where was that code coming from? I examined the disasm,
looked up symbols to figure out where call instructions were going, then grepped
the codebase for those function names. It soon became clear that the disasm I
was looking at was compiled from `zend_jit_fetch_dim_w_helper`.

Add --repeat testing mode

This testing mode executes the test multiple times in the same
process (but in different requests). It is primarily intended to
catch tracing JIT bugs, but also catches state leaks across
requests.

Closes GH-6365.

Merge branch 'PHP-8.0'

* PHP-8.0:
  FIxed bug #80299

FIxed bug #80299

The must_wrap was leaking across iterations.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fix SSA integrity violation for type inference in dead code

Merge branch 'PHP-7.4' into PHP-8.0

* PHP-7.4:
  Fix SSA integrity violation for type inference in dead code

Fix SSA integrity violation for type inference in dead code

The foreach body can never be executed and thus may contain empty
types. We should still uphold our SSA integrity invariants in that
case.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fix bug79177.phpt wrt. JIT

Merge branch 'PHP-7.4' into PHP-8.0

* PHP-7.4:
  Fix bug79177.phpt wrt. JIT

Fix bug79177.phpt wrt. JIT

JIT ignores that the `zend_write` callback is overwritten, so we define
our own callback and caller.

We also fix the "inconsistent DLL binding" warnings on Windows, by
introducing `PHP_ZEND_TEST_API`.

Closes GH-6391.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fix signed integer overflow

Fix signed integer overflow

Fixes oss-fuzz #26763.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fixed bug #80290

Fixed bug #80290

Dropping the dtor arg args[3] rather than using STR_COPY: Since
PHP 8, we no longer support separation in call_user_function(),
so we also don't need to worry about things like arguments being
replaced with references.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Create TSSA loops for recursive call/return traces and move invariant type guards out of loops.

Create TSSA loops for recursive call/return traces and move invariant type guards out of loops.

Use constant size string in hash bench.php

I don't like the previous behaviour where the bytes to hash change
every time the code changes, that may make it difficult to compare
hash() performance changes over time.

Use a fixed number instead, and allow passing an override for a
different length.

Closes GH-6386.

[ci skip]

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fix bug #72413: Segfault with get_result and PS cursors

Merge branch 'PHP-7.4' into PHP-8.0

* PHP-7.4:
  Fix bug #72413: Segfault with get_result and PS cursors

Fix bug #72413: Segfault with get_result and PS cursors

We cannot simply switch to use_result here, because the fetch_row
methods in get_result mode and in use_result/store_result mode
are different: In one case it accepts a statement, in the other
a return value zval. Thus, doing a switch to use_result results
in a segfault when trying to fetch a row.

Actually supporting get_result with cursors would require adding
cursor support in mysqlnd_result, not just mysqlnd_ps. That would
be a significant amount of effort and, given the age of the issue,
does not appear to be particularly likely to happen soon.

As such, we simply generate an error when using get_result()
with cursors, which is much better than causing a segfault.
Instead, parameter binding needs to be used.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Handle errors during PDO row fetch

Merge branch 'PHP-7.4' into PHP-8.0

* PHP-7.4:
  Handle errors during PDO row fetch

Handle errors during PDO row fetch

The EOF flag also gets set on error, so we always end up ignoring
errors here.

However, we should only check errors for unbuffered results. For
buffered results, this function is guaranteed not to error, and
querying the errno may return an unrelated error.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fix format specifier

Merge branch 'PHP-7.4' into PHP-8.0

* PHP-7.4:
  Fix format specifier

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix format specifier

Fix format specifier

In this function, `i` is of type `size_t`.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Report gone away error in one more place

Merge branch 'PHP-7.4' into PHP-8.0

* PHP-7.4:
  Report gone away error in one more place

Report gone away error in one more place

Merge branch 'PHP-8.0'

* PHP-8.0:
  [ci skip] Fix NEWS order

Merge branch 'PHP-7.4' into PHP-8.0

* PHP-7.4:
  [ci skip] Fix NEWS order

[ci skip] Fix NEWS order

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fix #44618: Fetching may rely on uninitialized data

Merge branch 'PHP-7.4' into PHP-8.0

* PHP-7.4:
  Fix #44618: Fetching may rely on uninitialized data

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #44618: Fetching may rely on uninitialized data

Fix #44618: Fetching may rely on uninitialized data

Unless `SQLGetData()` returns `SQL_SUCCESS` or `SQL_SUCCESS_WITH_INFO`,
the `StrLen_or_IndPtr` output argument is not guaranteed to be properly
set.  Thus we handle retrieval failure other than `SQL_ERROR` by
yielding `false` for those column values and raising a warning.

Closes GH-6281.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Retain QUIT_SEND connection state

Merge branch 'PHP-7.4' into PHP-8.0

* PHP-7.4:
  Retain QUIT_SEND connection state

Retain QUIT_SEND connection state

If the server goes away while reading a packet, don't go back into
the READY state. This will just cause broken pipe errors down the
line.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fixed bug #65825

Merge branch 'PHP-7.4' into PHP-8.0

* PHP-7.4:
  Fixed bug #65825