Fixed bug #36006 (Problem with $this in __destruct())

diff --git a/Zend/tests/bug36006.phpt b/Zend/tests/bug36006.phpt
new file mode 100755 (executable)
index 0000000..79f9897
--- /dev/null
+++ b/Zend/tests/bug36006.phpt
@@ -0,0 +1,30 @@
+--TEST--
+Bug #36006 (Problem with $this in __destruct())
+--FILE--
+<?php
+
+class Person {
+       public $dad;
+       public function __destruct() {
+               $this->dad = null; /* no segfault if this is commented out */
+       }
+}
+
+class Dad extends Person {
+       public $son;
+       public function __construct() {
+               $this->son = new Person;
+               $this->son->dad = $this; /* no segfault if this is commented out */
+       }
+       public function __destruct() {
+               $this->son = null;
+               parent::__destruct(); /* segfault here */
+       }
+}
+
+$o = new Dad;
+unset($o);
+echo "ok\n";
+?>
+--EXPECT--
+ok

diff --git a/Zend/zend_objects_API.c b/Zend/zend_objects_API.c
index 59e7b3317b58c577dc4bdf73ac64af622548c751..e7084f013b8d028f8433319bf1d3d6d2942b9028 100644 (file)
--- a/Zend/zend_objects_API.c
+++ b/Zend/zend_objects_API.c
@@ -52,7 +52,9 @@ ZEND_API void zend_objects_store_call_destructors(zend_objects_store *objects TS
                        if (!objects->object_buckets[i].destructor_called) {
                                objects->object_buckets[i].destructor_called = 1;
                                if (obj->dtor && obj->object) {
+                                       obj->refcount++;
                                        obj->dtor(obj->object, i TSRMLS_CC);
+                                       obj->refcount--;
                                }
                        }
                }