cvs: ZendEngine2 / zend_object_handlers.c /tests bug48248.phpt

felipe		Tue May 12 22:50:57 2009 UTC

  Added files:                 
    /ZendEngine2/tests	bug48248.phpt 

  Modified files:              
    /ZendEngine2	zend_object_handlers.c 
  Log:
  - Fixed bug #48248 (SIGSEGV when access to private property via &__get)
  
http://cvs.php.net/viewvc.cgi/ZendEngine2/zend_object_handlers.c?r1=1.214&r2=1.215&diff_format=u
Index: ZendEngine2/zend_object_handlers.c
diff -u ZendEngine2/zend_object_handlers.c:1.214 ZendEngine2/zend_object_handlers.c:1.215
--- ZendEngine2/zend_object_handlers.c:1.214	Thu Mar 26 22:16:47 2009
+++ ZendEngine2/zend_object_handlers.c	Tue May 12 22:50:56 2009
@@ -17,7 +17,7 @@
    +----------------------------------------------------------------------+
 */
 
-/* $Id: zend_object_handlers.c,v 1.214 2009/03/26 22:16:47 felipe Exp $ */
+/* $Id: zend_object_handlers.c,v 1.215 2009/05/12 22:50:56 felipe Exp $ */
 
 #include "zend.h"
 #include "zend_globals.h"
@@ -583,8 +583,8 @@
 		zend_guard *guard;
 
 		if (!zobj->ce->__get ||
-		    zend_get_property_guard(zobj, property_info, member, &guard TSRMLS_CC) != SUCCESS ||
-		    guard->in_get) {
+			zend_get_property_guard(zobj, property_info, member, &guard TSRMLS_CC) != SUCCESS ||
+			(property_info && guard->in_get)) {
 			/* we don't have access controls - will just add it */
 			new_zval = &EG(uninitialized_zval);
 

http://cvs.php.net/viewvc.cgi/ZendEngine2/tests/bug48248.phpt?view=markup&rev=1.1
Index: ZendEngine2/tests/bug48248.phpt
+++ ZendEngine2/tests/bug48248.phpt
--TEST--
Bug #48248 (SIGSEGV when access to private property via &__get)
--FILE--
<?php

class A
{
    public function & __get($name)
    {
        return $this->test;
    }
}

class B extends A
{
    private $test;
}

$b = new B;
var_dump($b->test);

?>
--EXPECTF--
Notice: Undefined property: B::$test in %s on line %d

Notice: Only variable references should be returned by reference in %s on line %d
NULL