Bug #60536 [PATCH]: Traits Segfault

Edit report at https://bugs.php.net/bug.php?id=60536&edit=1 ID: 60536 Patch added by: gron@php.net Reported by: scottmac@php.net Summary: Traits Segfault Status: Open Type: Bug Package: Scripting Engine problem Operating System: ubuntu 11.11 PHP Version: 5.4SVN-2011-12-15 (SVN) Block user comment: N Private report: N New Comment: The following patch has been added/updated: Patch Name: property008.phpt Revision: 1324054001 URL: https://bugs.php.net/patch-display.php?bug=60536&patch=property008.phpt&revision=1324054001 Previous Comments: ------------------------------------------------------------------------ [2011-12-16 16:19:39] gron@php.net The following patch has been added/updated: Patch Name: property007.phpt Revision: 1324052379 URL: https://bugs.php.net/patch-display.php?bug=60536&patch=property007.phpt&revision=1324052379 ------------------------------------------------------------------------ [2011-12-16 16:19:24] gron@php.net The following patch has been added/updated: Patch Name: property006.phpt Revision: 1324052364 URL: https://bugs.php.net/patch-display.php?bug=60536&patch=property006.phpt&revision=1324052364 ------------------------------------------------------------------------ [2011-12-16 16:19:08] gron@php.net The following patch has been added/updated: Patch Name: property005.phpt Revision: 1324052348 URL: https://bugs.php.net/patch-display.php?bug=60536&patch=property005.phpt&revision=1324052348 ------------------------------------------------------------------------ [2011-12-16 16:17:37] gron@php.net Hi Laruence: Your patch was also what I had thought of for the first moment, however, that is not actually fixing the bug. The problem is another one. For some reason the property value does not get set properly. At least that is my current understanding. This leads to either some inconsistent ref count, or an inappropriately shared zval. Haven't figured that out yet. What you patch does is just changing the semantics of when properties are composed into the class. That is also something that happens to be broken (inconsistent with normal inheritance). I have updated tests that should describe the correct semantics for property handling. If you see where I do something stupid with the zvals, please let me know. Thanks Stefan ------------------------------------------------------------------------ [2011-12-16 15:57:01] laruence@php.net The following patch has been added/updated: Patch Name: bug60536.phpt Revision: 1324051021 URL: https://bugs.php.net/patch-display.php?bug=60536&patch=bug60536.phpt&revision=1324051021 ------------------------------------------------------------------------ The remainder of the comments for this report are too long. To view the rest of the comments, please view the bug report online at https://bugs.php.net/bug.php?id=60536 -- Edit this bug report at https://bugs.php.net/bug.php?id=60536&edit=1