Don't mark variadic functions as ZEND_FASTCALL

Variadic functions do not support the fastcall calling convention.

Improve type declarations for Zend APIs

Voidification of Zend API which always succeeded
Use bool argument types instead of int for boolean arguments
Use bool return type for functions which return true/false (1/0)
Use zend_result return type for functions which return SUCCESS/FAILURE as they don't follow normal boolean semantics

Closes GH-6002

Don't load IP when entering inlined function

Fix tracked_realloc

We should only drop the information about the old allocation after
checking the memory limit. This makes the code a bit more awkward...

Promote incomplete class to modification to Error

In line with engine diagnostic promotions, property writes throw
an Error exception, while reads throw a warning.

Promote ArrayObject modification during sorting to Error exception

Throw Error instead of E_ERROR when calling method on incomplete class

There's no reason for this to abort execution completely.

Remove unused BG(str_ebuf) global

Also change BG(strtok_len) to size_t. This stores a string length,
so it should be size_t rather than zend_ulong.

Export and reuse zend_is_valid_class_name API

Unserialization does the same check as zend_lookup_class, so let's
share the same optimized implementation.

Use zend_string when storing incomplete class name

No need to copy the string here...

Store incomplete_class entry as normal global

I don't see any reason why this needs to live in a thread-safe
global, unlikely all other class entries.

Avoid useless IP load

Remove useless register allocations

Replace "mov %eax, %ecx; shl $1, $ecx" by "lea (%eax,%eax), %ecx"

Fix typo

This was supposed to include the header, not the C file...

Generate execute corpus in generate_all.php

And add crypt() to the function blacklist, it can be very slow.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Don't assert when comparing uninit DateTimeZone objects

Don't assert when comparing uninit DateTimeZone objects

Nothing guarantees that the objects are initialized here... just
check as usual.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Require non-negative length in stream_get_contents()

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Require non-negative length in stream_get_contents()

Require non-negative length in stream_get_contents()

If the length is not -1, require it to be non-negative.

Using such lengths doesn't make sense (as only -1 is special-case
to read in chunks, anything else will end up doing a huge upfront
allocation) and can lead to string allocation overflow.

A similar check is already in place for file_get_contents(). That
one does not allow -1 (and uses null instead), but this function
is explicitly specified to accept -1, so stick to that behavior.

Use zend_dval_to_lval() in spl_offset_convert_to_long()

Avoid fptoi UB and use the standard PHP conversion behavior.

Fix leak in typed reference assignment

We're only creating tmp here to compare against the existing
coerced_value. We need to destroy it in all cases.

Update operands range information accoring to the taken branches and use this indormation to eliminate useless overflow checks.

Add experimental "execute" fuzzer

This is an end-to-end fuzzer that executes arbitrary PHP code.

We replace the executor with a finite-step executor to avoid
getting stuck in loops or recursion.

Fix leak on failed DatePeriod initialization

We need to free not only p here, but also b and e.

Allow removing non-existing key from WeakMap

In line with usual PHP semantics. This previously triggered an
assertion failure.

Clean up strtok implementation

Store the zend_string instead of performing a copy and storing
in a zval. Also make sure the string is released immediately if
it's no longer needed. Finally, avoid null pointer offset UB if
no string has been set -- though I'm wondering if this case
shouldn't be generating a warning?

Fix use-after-free is WeakMap key and value are the same

Drop the object from the WeakMap as the last step, as this might
end up destroying the object.

Eliminate ZSTR_IS_INTERNED() check

Merge branch 'PHP-7.4' into master

* PHP-7.4:
  Catch potential exceptions during to string conversion

Catch potential exceptions during to string conversion

As of PHP 7.4.0, exceptions are allowed to be thrown from inside
`__toString()` methods; we have to cater to that, and catch these
exceptions early.

Closes GH-6042

Enforce memory limit in tracked allocation mode

A very basic limit (for single allocations) was already enforced.
This extends it to count the total memory allocations.

This is useful to avoid out of memory conditions while fuzzing.

Check interrupt only if the link is a part of the loop.

Rename zend_jit_var_may_be_modified_indirectly() into zend_jit_var_may_alias()

Check reference guard once

Use "movaps" instead of "movsd" to copy the whole %xmm register.

Fixed reference-counting

Merge branch 'PHP-7.4'

* PHP-7.4:
  Property handle read_property exception in fetch_property_address

Property handle read_property exception in fetch_property_address

Otherwise we leak (and corrupt uninitialized_zval).

Merge branch 'PHP-7.4' into master

* PHP-7.4:
  Fix com_safearray_proxy related memory management issues

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix com_safearray_proxy related memory management issues

Fix com_safearray_proxy related memory management issues

Lowercase method name in zend_call_method()

Fix pass by ref error for named params

Merge branch 'PHP-7.4' into master

* PHP-7.4:
  Separate COM::__construct()s $server_name array

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Separate COM::__construct()s $server_name array

Separate COM::__construct()s $server_name array

This may otherwise be modified.

Merge branch 'PHP-7.4' into master

* PHP-7.4:
  Fix #64130: COM obj parameters passed by reference are not updated

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #64130: COM obj parameters passed by reference are not updated

Fix #64130: COM obj parameters passed by reference are not updated

`ITypeInfo_GetIDsOfNames()` is supposed to fail with `E_NOTIMPL` for
out-of-process servers, thus we should not remove the already available
typeinfo of the object in this case.

We also properly free the `byref_vals`.

Drop various unused macros/APIs

Also convert_libmagic_pattern() to return a zend_string*

Closes GH-6029

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix file name clash in test

Fix file name clash in test

Fix memory leak on unknown named param in iterator unpack

Merge branch 'PHP-7.4'

* PHP-7.4:
  Avoid socket path clash in test

Avoid socket path clash in test

Improved JIT for FETCH_DIM_R/IS and ISSET_DIM_OBJ

Fix passing of undef var to named arg

This needs to use the previously computed argument target.

Fix memory leak on unknown named param

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix memory leak when yielding from non-iterable

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix memory leak when yielding from non-iterable

Fix memory leak when yielding from non-iterable

Fix over-eager named params optimization

We can't relax a named param to a positional param if we encountered
any unknown parameters in the meantime.

Prevent ResourceBundle double-construction

Prevent double-construction of NumberFormatter

Prevent double-construction of IntlGregorianCalendar

Prevent double-construction of IntlRuleBasedBreakIterator

Use guard to check if array is packed or hash

Prevent negative array index access

Merge branch 'jit_match'

* jit_match:
  Support for CASE_STRICT
  JIT for MATCH and CASE_STRICT instructions

Avoid priniting "array [long, string] of"

Merge branch 'PHP-7.4'

* PHP-7.4:
  Don't enable --with-mm in CI

Don't enable --with-mm in CI

Turns out this has a large negative effect on startup time,
making tests much slower.

Record information about packed arrays

Promote warnings to exceptions in ext/pcre

Closes GH-6006

Infer information about packed/hash arrays and use it for JIT

Ensure RuleBasedBreakIterator constructor throws on failure

Constructors must throw on failure indepdendent of the configured
intl error mode.

Promote warnings to exceptions in ext/simplexml

Closes GH-6011

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

Fixed JIT failure on "$a = []; $a[1] += 2;"

Merge branch 'PHP-7.4' into master

* PHP-7.4:
  Avoid double-free

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Avoid double-free

Clean up BreakIterator create_object handler

Use standard zend_object_alloc() function and fix the
object_init_properties() call (which works out okay because there
are no properties).

Fix XMLWriter::writeDtdEntity() stub

$isparam is optional.

Avoid double-free

As of commit b2e3fd1[1] the `authid.User` is no longer newly allocated,
so we must not free it.

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

Promote warnings to exceptions in ext/shmop

Closes GH-5986

Promote warnings to exceptions in ext/pcntl

Closes GH-6004

Merge branch 'PHP-7.4'

* PHP-7.4:
  Remove bogus REGISTER_LONG_CONSTANT

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Remove bogus REGISTER_LONG_CONSTANT

Remove bogus REGISTER_LONG_CONSTANT

This shouldn't be in this function, probably a copy/paste mistake...

Remove useless same_zval function

Closes GH-6039.

Promote warnings to exceptions in ext/phar

Closes GH-6008

Promote warnings to exceptions in ext/xmlreader

Closes GH-6021

Don't return temporary from SXE write_property handler

Return the original value. If we don't return the original value,
we need to own the zval, which we don't.

For clarity also switch things to work on a zend_string* value
instead of a zval*.

Simplify change_node_zval implementation

At this point, the value has already been converted into a string.

Move AVOID_REFCOUNTING type info flag into a separate bit

Support for CASE_STRICT

JIT for MATCH and CASE_STRICT instructions

Merge branch 'PHP-7.4' into master

* PHP-7.4:
  Fix potential integer overflow detected by oss-fuzz

Promote warning to exception in ext/enchant

Closes GH-6022