Safe execution timeout handling.

handle dummy frame

Fixed bug #71930 (_zval_dtor_func: Assertion `(arr)->gc.refcount <= 1' failed)

Merge zend_execute_data->called_scope into zend_execute_data->This.
"called_scope" made sense only for static method calls, for dynamic calls it was always equal to the class of $this.
Now EG

Merge zend_execute_data->called_scope into zend_execute_data->This.
"called_scope" made sense only for static method calls, for dynamic calls it was always equal to the class of $this.
Now EG(This) may store IS_OBJECT + $this or IS_UNUSED + "called_scope" (of course, "called_scope" may be NULL).
Some code might need to be adopted to support this change.
Checks (Z_OBJ(EX(This))) might need to be converted into (Z_TYPE(EX(This)) == IS_OBJECT).

show more ...

Use CALL_INFO flag to check if we need to destroy symbol_table.
This saves one memory load on most RETURN opcodes.

Fix ZEND_DEBUG condition

fix initializer for empty_fcall_info

Removed zend_fcall_info.symbol_table

PHP-7 zend_call_function() doesn't support symbol_table substitution

Fixed bug #71470

Don't report hashtable iterator leaks on unclean shutdown, those
are expected.

Use special type IS_ERROR instread of EG(error_zval). (we still need EG(error_zval) for SPL support).

bump year which is missed in rev 49493a2

bump year which is missed in rev 49493a2

Fixed bug #71163 (Segmentation Fault: cleanup_unfinished_calls)

Fixed iter leak on by-ref foreach over const/tmp array

FE_FREE does not unregister the iter for plain arrays. So always
wrap into a REF wrapper, even if not strictly necessary, in
RE

Fixed iter leak on by-ref foreach over const/tmp array

FE_FREE does not unregister the iter for plain arrays. So always
wrap into a REF wrapper, even if not strictly necessary, in
RESET_RW. Alternatively we could use a flag to distinguish plain
positions and interators.

Also added a check for leaked iterators in shutdown_executor.

show more ...

Use format string

Fixed bug #70914 zend_throw_or_error() format string vulnerability

Fixed reference cuntmting for closures (previously we didn't increment reference counter for "internal" closures)

Revert "Allow random $this on non-internal Closures again"

This reverts commit 35d0405c4790f0ce668c9e1b8b05197e55d29a05.

Allow random $this on non-internal Closures again
As it turns out, there is actually no reason to prevent this, it even was a bigger BC break than expected...

Also fixes a memory leak (t

Allow random $this on non-internal Closures again
As it turns out, there is actually no reason to prevent this, it even was a bigger BC break than expected...

Also fixes a memory leak (the Closure leaks) when calling internal functions via Closure by moving it out of leave helper onto caller side for TOP_CODE:

$z = new SplStack; $z->push(20);
$x = (new ReflectionMethod("SplStack", "pop"))->getClosure($z);
var_dump($x());

show more ...

Add myself into list of authors of the most refactored files.

Get rid of implicit type casting in GC_*() macros in Zend/zend_types.h.
This prevented compilation warnings and disclosed few incorrect usages in Zend/zend_vm_def.h and ext/dom/xpath.c.
Now e

Get rid of implicit type casting in GC_*() macros in Zend/zend_types.h.
This prevented compilation warnings and disclosed few incorrect usages in Zend/zend_vm_def.h and ext/dom/xpath.c.
Now explicit type casting may be required on call site.
This may break some C extension code, but it shoulfn't be a problem to add explicit casting.

show more ...

improve ZEND_TLS declaration even more and move it to the better place

use better name

simplify declaration