diff options
author | Xinchen Hui <laruence@gmail.com> | 2017-05-31 13:08:26 +0800 |
---|---|---|
committer | Xinchen Hui <laruence@gmail.com> | 2017-05-31 13:08:26 +0800 |
commit | 9064dca58ba87ccdbf6cee71122e50595765cfa1 (patch) | |
tree | 32606c4f50526ec7b531ac0119076225cc61dc7b /ext/reflection/php_reflection.c | |
parent | 316aaca15503f8e40a6c578c0f90d122e393d283 (diff) | |
parent | 9c5717d0decd56710129a5599fe5d38f82a7bab2 (diff) | |
download | php-git-9064dca58ba87ccdbf6cee71122e50595765cfa1.tar.gz |
Merge branch 'PHP-7.0' into PHP-7.1
* PHP-7.0:
Fixed bug #74673 (Segfault when cast Reflection object to string with undefined constant)
Conflicts:
ext/reflection/php_reflection.c
Diffstat (limited to 'ext/reflection/php_reflection.c')
-rw-r--r-- | ext/reflection/php_reflection.c | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/ext/reflection/php_reflection.c b/ext/reflection/php_reflection.c index a06522d025..db7056e03b 100644 --- a/ext/reflection/php_reflection.c +++ b/ext/reflection/php_reflection.c @@ -468,6 +468,9 @@ static void _class_string(string *str, zend_class_entry *ce, zval *obj, char *in ZEND_HASH_FOREACH_STR_KEY_PTR(&ce->constants_table, key, c) { _class_const_string(str, ZSTR_VAL(key), c, ZSTR_VAL(sub_indent.buf)); + if (UNEXPECTED(EG(exception))) { + return; + } } ZEND_HASH_FOREACH_END(); } string_printf(str, "%s }\n", indent); @@ -732,7 +735,10 @@ static void _parameter_string(string *str, zend_function *fptr, struct _zend_arg string_write(str, " = ", sizeof(" = ")-1); ZVAL_DUP(&zv, RT_CONSTANT(&fptr->op_array, precv->op2)); - zval_update_constant_ex(&zv, fptr->common.scope); + if (UNEXPECTED(zval_update_constant_ex(&zv, fptr->common.scope) == FAILURE)) { + zval_ptr_dtor(&zv); + return; + } if (Z_TYPE(zv) == IS_TRUE) { string_write(str, "true", sizeof("true")-1); } else if (Z_TYPE(zv) == IS_FALSE) { |