Issue with ob handler context and execution in PHP

[aswingit]

Issue with ob handler context and execution in PHP

[Girgias]

@haszi do you maybe have an idea about this?

[haszi]

No idea as this doesn't seem to be output buffering related. If I understand the above correctly what is being asked here is whether zend_get_executed_filename() can retrieve the current filename during shutdown but as is also mentioned, EG(current_execute_data) is set to NULL. This is the source code and the comment that goes with it:

php-src/main/main.c

Lines 1829 to 1832 in bb6e611

	/* EG(current_execute_data) points into nirvana and therefore cannot be safely accessed
	* inside zend_executor callback functions.
	*/
	EG(current_execute_data) = NULL;

[aswingit]

Thank you. When the shutdown phase occurs and the output handler (in this case, 'system') is executed, it triggers zend_call_function. The issue arises because the current execution context is set to NULL, resulting in the absence of the previous execution context when the system or output handler callback command is executed.

zend_get_executed_filename navigates through the context stack to locate the type of user-defined function in order to retrieve the filename. Without previous context, it's not able to derive filename.

[haszi]

The description of your issue and your question under Expected behavior seem like two different issue to me, but to comment on the first, passing system() to ob_start() will result in a warning as the second argument ob_start() passes to system() is not passed by reference so results in this warning:

PHP Warning: system(): Argument #2 ($result_code) must be passed by reference

I cannot comment on your question other than that the comment in the source code suggests that that is currently not possible.

[aswingit]

The problem isn't with the 'system' function, but with 'ob_flush'. Let's say I use a custom function from my extension that tries to read zend_get_executed_filename(). It doesn't get the right context when flush occurs during the shutdown phase.