diff options
| author | Nikita Popov <nikita.ppv@gmail.com> | 2019-08-17 10:57:26 +0200 |
|---|---|---|
| committer | Nikita Popov <nikita.ppv@gmail.com> | 2019-08-17 10:58:54 +0200 |
| commit | be7e819068985859f92e4af21e49b4f647dd0467 (patch) | |
| tree | 9c5b0f10549116ce14c54e2e5f04600c3ec66ea7 /Zend | |
| parent | a192499d2a3806bf55aa060539b5ce8580aa5db7 (diff) | |
| download | php-git-be7e819068985859f92e4af21e49b4f647dd0467.tar.gz | |
Fixed bug #77922
In PHP 7.3 shadow properties are no longer duplicated. Make sure we
only release them if the property was defined on the parent class,
which means that it changed from private->shadow, which is where
duplication does happen.
Diffstat (limited to 'Zend')
| -rw-r--r-- | Zend/tests/bug77922.phpt | 20 | ||||
| -rw-r--r-- | Zend/zend_opcode.c | 3 |
2 files changed, 22 insertions, 1 deletions
diff --git a/Zend/tests/bug77922.phpt b/Zend/tests/bug77922.phpt new file mode 100644 index 0000000000..2984f4c873 --- /dev/null +++ b/Zend/tests/bug77922.phpt @@ -0,0 +1,20 @@ +--TEST-- +Bug #77922: Double release of doc comment on inherited shadow property +--FILE-- +<?php + +class A { + /** Foo */ + private $prop; +} + +class B extends A { +} + +class C extends B { +} + +?> +===DONE=== +--EXPECT-- +===DONE=== diff --git a/Zend/zend_opcode.c b/Zend/zend_opcode.c index cd1525bdab..a7f32de379 100644 --- a/Zend/zend_opcode.c +++ b/Zend/zend_opcode.c @@ -235,7 +235,8 @@ ZEND_API void destroy_zend_class(zval *zv) efree(ce->default_static_members_table); } ZEND_HASH_FOREACH_PTR(&ce->properties_info, prop_info) { - if (prop_info->ce == ce || (prop_info->flags & ZEND_ACC_SHADOW)) { + if (prop_info->ce == ce || + ((prop_info->flags & ZEND_ACC_SHADOW) && prop_info->ce == ce->parent)) { zend_string_release_ex(prop_info->name, 0); if (prop_info->doc_comment) { zend_string_release_ex(prop_info->doc_comment, 0); |