Revert "Microoptimization (avoid code duplication)"

This reverts commit 25d97f5eeeaf2eca760139f04fe2711e71791d0f.

Microoptimization (avoid code duplication)

Remove HOT attribute from some VM handlers. Comparisons almost always followed by JMPZ/JMPNZ; JMPZNZ is rare used.

Remove --with-zlib-dir option from mysqlnd config

mysqlnd doesn't actually use this. PDO MySQL uses it, but also
declares it separately.

Improve exception handling for abstract/deprecated calls

Reuse existing arg freeing loop instead of duplicating it.

Additionally also handle deprecated in DO_FCALL_BY_NAME.

Fix handling of abstract/deprecated exception

The exception mechanism assumes that exceptions from DO_FCALL are
already happening after the function call. This means that we are
currently leaking the passed arguments, and I think we can also
corrupt the VM stack due to incorrect frame linking in some cases
(there are assertion failures if the VM stack page size is reduced).

Instead handle the stack frame freeing manually for this special
case.

Fixed bug #78488 (OOB in ZEND_FUNCTION(ffi_trampoline)).

Add missing opcache return types for functions in spl

(excluding spl_autoload)

spl_object_id() is of the most interest to me,
since I frequently call it in an application.

This includes false/null types caused by wrong argument types and wrong argument
counts.

I can't rule out iterator_to_array returning null in spl_iterator_apply,
so leave MAY_BE_NULL in.

With review comments by nikic:
Co-Authored-By: Nikita Popov <nikita.ppv@googlemail.com>

Add the last missing SKIPIF

Add missing SKIPIF (mbstring)

Add missing SKIPIF (openssl)

Co-Authored-By: Gabriel Caruso <carusogabriel34@gmail.com>

Add missing SKIPIF (sqlite)

Add missing SKIPIF

Document opcache.preload_user directive

Report false for inherited private methods in method_exists()

These shadow methods only exist as internal implementation markers.
This mirrors the behavior of property_exists().

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #41997: SP call yields additional empty result set

Merge branch 'PHP-7.2' into PHP-7.3

* PHP-7.2:
  Fix #41997: SP call yields additional empty result set

Fix #41997: SP call yields additional empty result set

When stored procedures are called, the "final result set is a status
result that includes no result set".  Calling `::nextRowset()` on the
actual last result set should return FALSE, since there is actually no
further result set to be processed.

Execute preload script under user defined by opcache.preload_user directive

Avoid pointer UB in strip_header

Don't calculate header_start if lc_header_start is NULL, as we're
going to overflow the address space in that case.

Use unsigned subtraction in php_random_int()

This subtraction may overflow the signed domain, which is UB. Use
an unsigned subtraction instead.

Merge branch 'PHP-7.4' of git://github.com/php/php-src into PHP-7.4

Add missing opcache return info for ext/standard.

array_key_first/last returns null for invalid args,
wrong argument counts, and empty arrays.

random_bytes returns a string or throws.

Add new missing functions from ext/hash

Add zpp check to spl_classes()

Update NEWS for 7.4.0RC2

Update NEWS for PHP 7.4.0RC1

Clarify failure behavior of spl_iterator_apply()

It only fails if it throws, in which case it is meaningless to
set a return value.

Add missing argument checks

These functions don't expect any arguments, so we check that none are
given.

Merge branch 'PHP-7.3' into PHP-7.4

Merge branch 'PHP-7.2' into PHP-7.3

Fix opcache return type for hash_update_stream

It can return false if the resource type is wrong.

```
php > var_export(hash_update_stream(hash_init('md5'),
        imagecreate(1,1)));

Warning: hash_update_stream(): supplied resource is not a valid stream
resource in php shell code on line 1
false
```

The return types were initially added in
c88ffa9a5673cb3141660626ba1921671f0b84d6

Update PHP_API_VERSION

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Update NEWS

Update NEWS

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #76577: outdated documentation concerning track_errors

Merge branch 'PHP-7.2' into PHP-7.3

* PHP-7.2:
  Fix #76577: outdated documentation concerning track_errors

Fix #76577: outdated documentation concerning track_errors

Free two bits in fn_flags by merging ZEND_ACC_HEAP_RT_CACHE/ZEND_ACC_USER_ARG_INFO and ZEND_ACC_DONE_PASS_TWO/ZEND_ACC_ARENA_ALLOCATED that may be used only for user/internal functions

Fixed information about unused flags

Merge branch 'PHP-7.3' into PHP-7.4

Merge branch 'PHP-7.2' into PHP-7.3

Fix pkg-config version constraint for ICU

On PHP 7.2 our minimum ICU version is 4.0, not 40.

Drop E_STRICT notice in mysqli extension

Reenable alloca support on ZTS

This got disabled due to an incorrect change in a preprocessor condition
in 2104bea5d756dfa40b605a4a2765a3bc4637a76c. It was not supposed to
be disabled.

Fix file clashes in copy tests

Create the "*" file in a sub-directory rather than the main test
directory, so that it's different for each test.

Remove bogus vcwd_open

This is a left-over from the Linux version...

Merge branch 'PHP-7.3' into PHP-7.4

Merge branch 'PHP-7.2' into PHP-7.3

Fixed bug #78469

fcgi_accept_request function is supposed to call a FastCGI implementation's
on_accept hook when entering an "accepting" stage (that is right before
calling "accept"). This hook implementation (fpm_request_accepting) updates
a worker state to an "accepting" state which is effectively an "Idle" state,
and updates counters on the scoreboard of the corresponding pool (idle++,
active--).

But this is not done when listening for client connections on a named pipe on
Windows platform. In that case a combination of
ConnectNamedPipe/WaitForSingleObject is used (to be able to catch in_shutdown
as far as I understand), but it is nonetheless functionally equivalent to
"accept" call. Also by not calling on_hook neither a worker's state is updated
to "accepting" state nor scoreboard counters are updated.

Avoid accessing undefined index in CLI server test

Don't call Reflection::export() in --rf etc implementation

This method is deprecated ... instead simply directly print the
object.

Modernize some connectors in tests to remove env key duplication

Add job for community projects

Run some open-source projects through an aggressive debug
configuration with asan and ubsan. We don't care about test results,
only check that we don't assert or crash.

Currently testing laravel, symfony and amp.

Update NEWS

Fix #78460: PEAR installation failure

When building PHP outside of the source tree:

  mkdir custom-build-dir
  cd custom-build-dir
  ../path/to/php-src/configure

The directories need to be manually created including the pear directory
so the pear installation PHAR file doesn't need to be downloaded from
the remote location.

Closes GH-4639

Fix potentially uninitialized use in ldap

If ctrl->ldctl_value.bv_len is zero context is uninitialized. Only
free it if the call took place and succeeded.

Addref static vars when not copying private method

While we don't need to give this method separate static vars, we
do still need to perform an addref, as there will be a corresponding
delref in the dtor.

Use explicit unaligned type in md5 implementation

Merge branch 'PHP-7.3' into PHP-7.4

Merge branch 'PHP-7.2' into PHP-7.3

Fix use-after-free of immediately invoked closure with extra args

Set opcache.protect_memory=1 on Azure

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #78473: odbc_close() closes arbitrary resources

Merge branch 'PHP-7.2' into PHP-7.3

* PHP-7.2:
  Fix #78473: odbc_close() closes arbitrary resources

Fix #78473: odbc_close() closes arbitrary resources

We have to bail out, if an invalid resource is given.  For consistency
with the other `zend_fetch_resource(2)` calls, we return `FALSE`.

Add missed mutability check

Remove delref in free_object_storage

This gives us better object leak diagnostics.

Destroy error handlers before final GC run

Fixed bug #78335

Destroy static properties and variables prior to the final GC run,
as they may hold GC roots.

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Upgrade oniguruma lib to 6.9.3

Upgrade oniguruma lib to 6.9.3

Don't specify precedence for T_INC/T_DEC

As these do not operate on expressions, precedence is meaningless
for them.

Add --with-pear to Travis build

add ffi to php.ini extension [ci skip]

Fix signature and calls

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Update SDK version for AppVeyor

Merge branch 'PHP-7.2' into PHP-7.3

* PHP-7.2:
  Update SDK version for AppVeyor

Update SDK version for AppVeyor

Merge branch 'PHP-7.3' into PHP-7.4

Fix bug #78334 (fpm log prefix message includes wrong stdout/stderr notation)

Merge branch 'PHP-7.3' into PHP-7.4

Merge branch 'PHP-7.2' into PHP-7.3

Remove properties HT from nested GC data

The properties HT may be a GC root itself, so we need to remove it.
I'm not sure this issue actually applies to PHP 7.2, but committing
it there to be safe. As seen from the test case, the handling here
is rather buggy on 7.2.

Don't use needs_live_range hook for "special" live ranges

In particular we were disgarding SILENCE live ranges in opcache,
because we decided that a MAY_BE_LONG type does not need a live
range.

Replace deprecated libzip functions

We replace all deprecated libzip functions with their recommended
substitutes, and add proper comment length checks including a test
case.

Deprecate AI_IDN_ALLOW_UNASSIGNED and AI_IDN_USE_STD3_ASCII_RULES

These flags have been deprecated in glibc 2.28, so we also
deprecate them in PHP.

As we can't deprecate constants, we can only check for their use
in socket_addrinfo_lookup().

Merge branch 'PHP-7.3' into PHP-7.4

Merge branch 'PHP-7.2' into PHP-7.3

Also fix signature for passthru

Backported from a1a8d144854acb1c891cf0c21abb0f612b1d8de7
https://www.php.net/manual/en/function.passthru.php#refsect1-function.passthru-returnvalues

`passthru()` is false with invalid args
`passthru('command')` is null.

Fix opcache optimizer info for time_nanosleep

This can also return an array. See
https://www.php.net/manual/en/function.time-nanosleep.php#refsect1-function.time-nanosleep-returnvalues

> If the delay was interrupted by a signal, an associative array will be
returned with the components:
>
> - seconds - number of seconds remaining in the delay
> - nanoseconds - number of nanoseconds remaining in the delay

Sending a SIGUSR1 to the below program would trigger this behavior.

```
pcntl_signal(\SIGUSR1, function ($signo, $signinfo) {
    echo "Handling a signal $signo\n";
});
echo "Sleeping for 100 seconds\n";
var_export(time_nanosleep(100, 0));
```

The incomplete signature existed since c88ffa9a5.
No phpt tests existed for time_nanosleep returning an array

Fixed bug #78456

Merge branch 'PHP-7.3' into PHP-7.4

Merge branch 'PHP-7.2' into PHP-7.3

Fix overflow in memory limit checks

Due to overflows in the memory limit checks, we were missing cases
where the allocation size was close to the address space size, and
caused an OOM condition rather than a memory limit error.

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix test

Fix test

Not sure why offset changed... probably different PCRE version calculates
them in different way.

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #75457: heap-use-after-free in php7.0.25

Merge branch 'PHP-7.2' into PHP-7.3

* PHP-7.2:
  Fix #75457: heap-use-after-free in php7.0.25

Merge branch 'PHP-7.1' into PHP-7.2

* PHP-7.1:
  Fix #75457: heap-use-after-free in php7.0.25