Merge branch 'PHP-7.4'

Fix pipe detection and stream position handling

There are two related changes here:
1. Also check for S_ISCHR/FILE_TYPE_CHAR when checking for pipes, so
   that we detect ttys as well, which are also not seekable.
2. Always set position=-1 (i.e. ftell will return false) when a pipe
   is detected. Previously position=0 was sometimes used, depending on
   whether we're on Windows/Linux and whether the FD or FILE codepath
   was used.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Revert "Microoptimization (avoid code duplication)"

Revert "Microoptimization (avoid code duplication)"

This reverts commit 25d97f5eeeaf2eca760139f04fe2711e71791d0f.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Microoptimization (avoid code duplication)

Microoptimization (avoid code duplication)

Merge branch 'master' of git.php.net:php-src

* 'master' of git.php.net:php-src:
  Load execute_data without global regs

Load execute_data without global regs

Merge branch 'PHP-7.4'

* PHP-7.4:
  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

Merge branch 'PHP-7.4'

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.

Fix deprecated args freeing with JIT

I'm including the logic for this rare case in the helper function
to avoid complicating the main JIT logic.

Merge branch 'PHP-7.4'

Warnings to Errors: imagesetstyle

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.

Warnings to errors in imageaffinematrix*()

Warnings to errors in imagecrop(auto)

Merge branch 'PHP-7.4'

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.

Warnings to errors imageconvolution

Warnings become errors for imagepolygon et al

Negative checks for gamma

Warnings for image colour range check now throw exceptions

Warnings to errors for imagecreatefromgd2part

We also delete tests which were duplicates of a completely unrelated
test.

Warnings to Errors imagetruecolortopalette

Warnings to Errors imagecreate(truecolor)

We also add a test helper which we will be using for other GD functions
as well.

Add ext/xmlreader stubs

Add missing zend_parse_paramters_none()

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fixed bug #78488 (OOB in ZEND_FUNCTION(ffi_trampoline)).

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

Merge branch 'master' of git.php.net:php-src

* 'master' of git.php.net:php-src:
  Add missing opcache return types for functions in spl

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fixed bug #78488 (OOB in ZEND_FUNCTION(ffi_trampoline)).

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

Merge branch 'PHP-7.4'

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>

Declare nullable arguments in stubs

Merge branch 'PHP-7.4'

* PHP-7.4:
  Add the last missing SKIPIF

Add the last missing SKIPIF

Merge branch 'PHP-7.4'

* PHP-7.4:
  Add missing SKIPIF (mbstring)

Add missing SKIPIF (mbstring)

Add missing SKIPIF (xml)

Merge branch 'PHP-7.4'

* PHP-7.4:
  Add missing SKIPIF (openssl)

Add missing SKIPIF (openssl)

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

Merge branch 'PHP-7.4'

* PHP-7.4:
  Add missing SKIPIF (sqlite)

Add missing SKIPIF (sqlite)

Merge branch 'PHP-7.4'

* PHP-7.4:
  Add missing SKIPIF

Add missing SKIPIF

Fix default value

Merge branch 'PHP-7.4'

* PHP-7.4:
  Document opcache.preload_user directive

Document opcache.preload_user directive

Merge branch 'PHP-7.4'

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.4'

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

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.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Execute preload script under user defined by opcache.preload_user directive

Execute preload script under user defined by opcache.preload_user directive

Don't explicitly set return value on ZPP failure in ext/xml

Failing ZPP throws as of PHP 8.0.0, so explicitly setting a return
value is useless, and also slightly confusing.

Add ext/xml stubs

Merge branch 'PHP-7.4'

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.

Don't perform coercions in internal func return type assertion

The returned value must match the specified type exactly, as we
can't perform any coercions in non-debug builds.

Fix incorrect stub for easter_date() that slipped through the
assertion for this reason...

Fix gen_stubs with out-of-tree build

We need to prefix with $(top_srcdir).

Support computing func info from ret arg info for internal funcs

Remove duplicate func info for array_key_first/last

Looks like this was already added in master separately...

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

Merge branch '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.

Merge branch 'PHP-7.4'

Add new missing functions from ext/hash

Merge branch 'PHP-7.4'

Add zpp check to spl_classes()

Update NEWS for 7.4.0RC2

Update NEWS for PHP 7.4.0RC1

Merge branch 'PHP-7.4'

Clarify failure behavior of spl_iterator_apply()

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

Fix opcache optimizer info for time_nanosleep (again)

Fixes my earlier PR #4617

If I remember correctly,
The F0 macro is used for return values that are guaranteed to not be
reference counted.
The F1 macro is used for return values that may have up to 1 reference
(i.e. MAY_BE_RC1).

I didn't notice that time_nanosleep needed to be F1 since it could
return an array, and that array is reference counted.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Add missing argument checks

Add missing argument checks

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

php_stream_from_zval can no longer return false

Merge branch 'PHP-7.4'

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

Don't explicitly set return value on ZFR failure in ext/xmlwriter

Failing `zend_fetch_resource(2)` throws as of PHP 8.0.0, so explicitly
setting a return value is useless, and also slightly confusing.

Add ext/xmlwriter stubs

Fixed pow() function info

Merge branch 'PHP-7.4'

* PHP-7.4:
  Update PHP_API_VERSION

Update PHP_API_VERSION

Make ReflectionType an abstract class

This is never instantiated directly, only child classes are used.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Update NEWS

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Update NEWS

Update NEWS

Merge branch 'PHP-7.4'

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

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