Promote Warnings to Error in PostgreSQL PDO driver

Closes GH-6119

Convert Division by 0 warnings to Error per the Engine Warning RFC

Closes GH-6125

JIT for ASSIGN_OBJ_OP

Convert E_ERROR to Error exception in mysqli_result iterator

The same condition uses Error exceptions in lots of other classes.

Drop some dead code

Slightly improve error handling in DatePeriod::__construct()

Declare array|int and object-of-class|int types in stubs

Closes GH-6081

Co-Authored-By: Nikita Popov <nikic@php.net>

Make mb_send_mail() consistent with mail()

The $additional_headers parameter shouldn't accept null.

Fix a few stub parameter types in ext/standard

Add Z_PARAM_OBJ_OF_CLASS ZPP macro

Unmark tests as XFAIL on Windows

This issue has been resolved[1], so the tests are expected to pass on
Windows again.

[1] <https://github.com/php/php-src/commit/0a03291b51e7cf199c06ba4ae914277ce0ac4202>

Allocate Windows 64 shadow stack space

Remove @jit Docblock support for now due to dev-time constraints

Convert warnings to assetions in OCI PDO driver

Closes GH-6118

Mark tests as XFAIL on Windows

These tests segfault with tracing JIT on Windows as of a recent
commit[1].  A link to a backtrace is available in a comment[2].  For
now, we mark these tests as XFAIL, to keep AppVeyor CI helpful.

[1] <https://github.com/php/php-src/commit/7b0a05316957982f9e4cc021aec5c20cc80f7542>
[2] <https://github.com/php/php-src/commit/7b0a05316957982f9e4cc021aec5c20cc80f7542#commitcomment-42267637>

Mark tests as XFAIL on Windows

These tests segfault with tracing JIT on Windows as of a recent
commit[1].  A link to a backtrace is available in a comment[2].  For
now, we mark these tests as XFAIL, to keep AppVeyor CI helpful.

[1] <https://github.com/php/php-src/commit/817ae414962480d133ed76618b88b3e7ba482718>
[2] <https://github.com/php/php-src/commit/817ae414962480d133ed76618b88b3e7ba482718#commitcomment-42227420>

Fix broken test

Since e8e4ddce

Use Error for uninitialized SQLite object

Closes GH-6113

Fix SKIPIF section for SQLite3 test

Test is borked if the posix_geteui() function is unavailable.

Improve parameter handling in ext/openssl

Closes GH-6025

Fix prototype of get_active_function_or_method_name()

Normalize behaviour of DNS function on Windows

This is a follow-up on commit 4a438b44694d4caf86d8e57266ed2fc2f4ae6213

Add some tests to hopefully not forget it next time

Handle missing result_var in binary_op_result_type

For dim/obj compound ops we don't have a result_var. Not sure why
this never caused issues before, but this can crash with JIT.

Fix mbstring fuzzer

mb_ereg can throw now, so we need a dummy frame and need to
free the exception afterwards.

Promote some warnings in BCMath to Errors

Also do a bit of refactoring at the same time.

Closes GH-6105

Use string|int union types in pgsql

For "field name or field offset" parameters.

Also make $ctor_params an ?array parameter.

Make argument type error message consistent for variadics

If an argument error refers to a variadic argument, we normally
do not print the name of the variadic (as it is not referring to
an individual argument, but to the collection of all of them).
However, this was not the case for the userland argument type
error message, which did it's own formatting.

Closes GH-6101.

Also forbid null bytes in mail()

I've adjusted mb_send_mail() already, but of course the handling
in mail() should be the same.

Merge branch 'PHP-7.4' into master

* PHP-7.4:
  Fix #80067: Omitting the port in bindto setting errors

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #80067: Omitting the port in bindto setting errors

Fix #80067: Omitting the port in bindto setting errors

A recent commit[1] which fixed a memory leak introduced a regression
regarding the formerly liberal handling of IP addresses to bind to.  We
fix this by reverting that commit, and fix the memory leak where it
actually occurs.  In other words, this fix is less intrusive than the
former fix.

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

Closes GH-6104.

Clean up ext/session errors

Closes GH-6111

Consolidate new union type ZPP macro names

They will now follow the canonical order of types. Older macros are
left intact due to maintaining BC.

Closes GH-6112

Refactor ReflectionMethod::__construct()

Closes GH-6098

Throw ValueError on null bytes in mb_send_mail()

Instead of silently replacing with spaces.

Remove the duplicate line in zend_ast.c

Closes GH-6107.

Add missing condition on azure community job

Eliminate repeatable class guards and unnecessary IS_UNDEF property checks

Fix leak in snmp

JIT for ASSIGN_OBJ

Fixed 32-bit JIT

Avoid gap in AST_CLASS child nodes for attributes

See https://github.com/nikic/php-ast/pull/181

> Hm, I'm thinking it would make more sense to change the structure in php-src.
> All the function types have consistent AST structure, but there's no reason at
> all why classes should be consistent with functions.

It's unusual to have an unused child node between other child nodes that are
used (for name, extends, implements, and attributes of AST_CLASS)

> That gap is a leftover from a previous refactoring. An earlier version of
> attributes extended `zend_ast_decl` with a new member called `attributes` and
> therefore did not need to handle functions and classes in different ways.

Closes GH-6088

fix zend_hash_get_current_key_type_ex proto

fix zend_hash_get_current_key_ex proto

Simplify error type filter

Closes GH-6049.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix azure i386 build, again

Fix azure i386 build, again

Squash compile warnings the easy way

Make PDO_OCI test more resilient

Skip PDO test for Oracle

Fix premature test termination

Attempt to fix bug #80014 (PHP 8.0 beta2 crashes with default JIT flags due to hardware incompatibility)

Added FFI\CType::getName() method

Eliminate unnecessary IS_INDIRECT guards

Avoid more exception checks

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix #79825: opcache.file_cache causes SIGSEGV with custom opcode handlers

More pgsql func info fixes

Fix opcache return type info for pgsql functions

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #79825: opcache.file_cache causes SIGSEGV with custom opcode handlers

Fix #79825: opcache.file_cache causes SIGSEGV with custom opcode handlers

Modules may have changed after restart which can cause dangling pointers from custom opcode handlers in the second-level cache files. This fix includes the installed module names and versions in the accel_system_id hash as entropy. Closes GH-5836

JIT for ASSIGN_DIM[_OP] with first IS_VAR + IS_INDIRECT operand

Use proper int type for parameter

Fix default value

Use PGSQL_RETURN_OID() in more places

Add stub for pgsql extension

Remove deprecated pgsql signatures

As the comment indicates, these are deprecated in PHP 4.2...

Eliminate unnecessary exception checks

Promote some warnings in MBString Regexes

Closes GH-5341

Promote pgsql no link to Error exception

Remove deprecated multi-parameter form of pg_connect()

Make array_multisort() signature more variadic

The second and third arguments are not always the sort_order and
sort_flags -- they can also be in reverse order, or be arrays
altogether. Move them into the variadic parameter to avoid awkward
error messages.

Test snmp on azure

Closes GH-6100.

Mark snmp tests as conflicting

Suppress uninitialized variable warning in snmp

These are false positive warnings.

Allow array_diff() and array_intersect() with single array argument

Both of these functions are well-defined when used with a single
array argument -- rejecting this case was an artificial limitation.
This is not useful when called with explicit arguments, but removes
edge-cases when used with argument unpacking:

    // OK even if $excludes is empty.
    array_diff($array, ...$excludes);

    // OK even if $arrays contains a single array only.
    array_intersect(...$arrays);

This matches the behavior of functions like array_merge() and
array_push(), which also allow calls with no array or a single
array respectively.

Closes GH-6097.

Improve error messages mentioning parameters instead of arguments

Closes GH-5999

Fixed memory leak

Wildcard output differences in snmp tests

And point out that snmp-mibs-downloader is needed.

Tracin JIT support for FETCH_DIM_W/RW with IS_VAR + IS_INDIRECT first operand.

Avoid use of remote_port in snmp

This field is not used (and has not been used for a long time --
I've seen some mailing list thread from 2003 about it!) and throws
a deprecation warning.  The port is part of peername instead (for
transports that support a port at all).

Pass correct op_info to zend_jit_fetch_dim()

Flesh out ZEND_HASH_REVERSE_FOREACH_* macros

Closes GH-6080.

Fix 3b0fecd and resulting test fail lob_012.phpt

JIT for FETCH_DIM_W/RW insructions

Fix Windows build for shared ext/sockets

We can't use `PHPAPI` for extensions which may be built shared.  Thus,
we introduce `PHP_SOCKETS_API`.

Don't handle missing value in zend_verify_arg_error()

This error condition is handled separately, never reaches this
function.

mb_str_split is already documented on php.net

So remove TODO comment which implies that it's not.

JIT for IN_ARRAY instruction.

Merge branch 'PHP-7.4' into master

* PHP-7.4:
  Fix #72941: Modifying bucket->data by-ref has no effect any longer

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #72941: Modifying bucket->data by-ref has no effect any longer

Fix #72941: Modifying bucket->data by-ref has no effect any longer

To match the PHP 5 behavior, we have to explicitly cater to `buffer` or
`data` being references.

Closes GH-6096.

Try to fix 32-bit setcookie tests, again

add socket_ce and socket_import_file_descriptor in public API

Fix some snmp stubs

As well as some basic mistakes in tests.

Suppress unused variable in snmp

force_ipv6 may be unused if compiling without ipv6 support.

Run the opcache-only configuration only for scheduled builds

The opcache-only configuration has very little signal (i.e. it is
very rare that it fails while non-opcache and opcache+jit both
pass). Switch it to run only for nightly builds, so we get faster
results on normal builds.

Try to fix setcookie() tests on 32-bit

micro-optimization

JIT for FETCH_LIST_R

Fix UNKNOWN default values in ext/oci8

Closes GH-6089