Hi,

i created 2 patches that are attached to this mail, one to fix the issue and the other to add tests for it (I added them into reflectionProperty_setAccessible.phpt).

Roman







On Apr 4, 2009, at 1:23 PM, Johannes Schlüter wrote:

On Sat, 2009-04-04 at 13:08 +0200, Roman Borschel wrote:
My "patch" is simply to replace the first argument of
zend_update_property/zend_read_property in ext/php_reflection.c like
this:
[...]
So simply replacing Z_OBJCE_P(object) with ref->ce.

yes, that was what i was thinking about, too after my first look, but I have no test environment on this box, will try it later, if nobody steps
in tests&commits :-)

johannes


--
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php


-- 
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php

Reply via email to