diff options
| author | Gustavo André dos Santos Lopes <cataphract@php.net> | 2012-03-18 18:23:27 +0000 |
|---|---|---|
| committer | Gustavo André dos Santos Lopes <cataphract@php.net> | 2012-03-18 18:23:27 +0000 |
| commit | 227112cb0fb732d86da376e1b8ccc9dab2a5e225 (patch) | |
| tree | 10160525a08ba657a1e6ff7bd09c24b013800117 /ext/reflection/php_reflection.c | |
| parent | 714f1ff4b37c5101b3c61ea108a3d415f41e50df (diff) | |
| download | php-git-227112cb0fb732d86da376e1b8ccc9dab2a5e225.tar.gz | |
- Fixed bug #61388 (ReflectionObject:getProperties() issues invalid reads
when get_properties returns a hash table with (inaccessible) dynamic
numeric properties).
Diffstat (limited to 'ext/reflection/php_reflection.c')
| -rw-r--r-- | ext/reflection/php_reflection.c | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/ext/reflection/php_reflection.c b/ext/reflection/php_reflection.c index 06f806f289..94cc05ee58 100644 --- a/ext/reflection/php_reflection.c +++ b/ext/reflection/php_reflection.c @@ -3667,6 +3667,13 @@ static int _adddynproperty(zval **pptr TSRMLS_DC, int num_args, va_list args, ze zend_class_entry *ce = *va_arg(args, zend_class_entry**); zval *retval = va_arg(args, zval*), member; + /* under some circumstances, the properties hash table may contain numeric + * properties (e.g. when casting from array). This is a WONT FIX bug, at + * least for the moment. Ignore these */ + if (hash_key->nKeyLength == 0) { + return 0; + } + if (hash_key->arKey[0] == '\0') { return 0; /* non public cannot be dynamic */ } |