laruence / taint

Taint is a PHP extension, used for detecting XSS codes

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

User registered opcode handler should call ones already set by other extensions

derickr opened this issue · comments

The taint extension marks a conflict with Xdebug.

This extension registers an opcode handler. This handler currently returns the ZEND_USER_OPCODE_DISPATCH value which means that the Zend engine users its internal implementation to further handle that opcode. If however another extension (such as Xdebug) has also overridden the opcode, its handler will not be called.

Make sure to remember already set handlers, and then call these if they're not NULL, and otherwise continue returning ZEND_USER_OPCODE_DISPATCH.

taint/taint.c

Line 1225 in 4a6c4cb

static void php_taint_register_handlers() /* {{{ */ {

See also https://bugs.xdebug.org/1759, which I've recently fixed.

@laruence could we get a new release so we can use taint in tandem with xdebug without issues?