cvs: ZendEngine2(PHP_5_3) / zend_execute.c /tests bug36303.phpt bug44660.phpt php-src NEWS

"Dmitry Stogov" <[email protected]>
Newsgroups gmane.comp.php.cvs.zend
Message-ID <cvsdmitry1217095195@cvsserver>
dmitry		Sat Jul 26 17:59:55 2008 UTC

  Added files:                 (Branch: PHP_5_3)
    /ZendEngine2/tests	bug44660.phpt 

  Modified files:              
    /php-src	NEWS 
    /ZendEngine2	zend_execute.c 
    /ZendEngine2/tests	bug36303.phpt 
  Log:
  Fixed bug #44660 (Indexed and reference assignment to propery of non-object don't trigger warning)
  
  
http://cvs.php.net/viewvc.cgi/php-src/NEWS?r1=1.2027.2.547.2.965.2.224&r2=1.2027.2.547.2.965.2.225&diff_format=u
Index: php-src/NEWS
diff -u php-src/NEWS:1.2027.2.547.2.965.2.224 php-src/NEWS:1.2027.2.547.2.965.2.225
--- php-src/NEWS:1.2027.2.547.2.965.2.224	Sat Jul 26 15:30:24 2008
+++ php-src/NEWS	Sat Jul 26 17:59:52 2008
@@ -249,6 +249,8 @@
 - Fixed bug #44805 (rename() function is not portable to Windows). (Pierre)
 - Fixed bug #44779 (filter returns NULL in CLI when it shouldn't). (Arnaud)
 - Fixed bug #44742 (timezone_offset_get() causes segmentation faults). (Derick)
+- Fixed bug #44660 (Indexed and reference assignment to propery of non-object
+  don't trigger warning). (Dmitry)
 - Fixed bug #44648 (Attribute names not checked for wellformedness). (Rob)
 - Fixed bug #44414 (Incomplete reporting about abstract methods). (Dmitry)
 - Fixed bug #44390 (mysqli_bind_param/bind_result and Object member variables).
http://cvs.php.net/viewvc.cgi/ZendEngine2/zend_execute.c?r1=1.716.2.12.2.24.2.33&r2=1.716.2.12.2.24.2.34&diff_format=u
Index: ZendEngine2/zend_execute.c
diff -u ZendEngine2/zend_execute.c:1.716.2.12.2.24.2.33 ZendEngine2/zend_execute.c:1.716.2.12.2.24.2.34
--- ZendEngine2/zend_execute.c:1.716.2.12.2.24.2.33	Sat Jul 26 14:08:10 2008
+++ ZendEngine2/zend_execute.c	Sat Jul 26 17:59:54 2008
@@ -17,7 +17,7 @@
    +----------------------------------------------------------------------+
 */
 
-/* $Id: zend_execute.c,v 1.716.2.12.2.24.2.33 2008/07/26 14:08:10 dmitry Exp $ */
+/* $Id: zend_execute.c,v 1.716.2.12.2.24.2.34 2008/07/26 17:59:54 dmitry Exp $ */
 
 #define ZEND_INTENSIVE_DEBUGGING 0
 
@@ -1175,6 +1175,7 @@
 			}
 			object_init(container);
 		} else {
+			zend_error(E_WARNING, "Attempt to modify property of non-object");
 			result->var.ptr_ptr = &EG(error_zval_ptr);
 			PZVAL_LOCK(EG(error_zval_ptr));
 			return;
http://cvs.php.net/viewvc.cgi/ZendEngine2/tests/bug36303.phpt?r1=1.1.2.1.4.1&r2=1.1.2.1.4.2&diff_format=u
Index: ZendEngine2/tests/bug36303.phpt
diff -u ZendEngine2/tests/bug36303.phpt:1.1.2.1.4.1 ZendEngine2/tests/bug36303.phpt:1.1.2.1.4.2
--- ZendEngine2/tests/bug36303.phpt:1.1.2.1.4.1	Sat May 24 15:55:18 2008
+++ ZendEngine2/tests/bug36303.phpt	Sat Jul 26 17:59:54 2008
@@ -8,5 +8,7 @@
 echo "ok\n";
 ?>
 --EXPECTF--
+Warning: Attempt to modify property of non-object in %sbug36303.php on line 3
+
 Warning: Invalid argument supplied for foreach() in %sbug36303.php on line 3
 ok

http://cvs.php.net/viewvc.cgi/ZendEngine2/tests/bug44660.phpt?view=markup&rev=1.1
Index: ZendEngine2/tests/bug44660.phpt
+++ ZendEngine2/tests/bug44660.phpt



-- 
Zend Engine CVS Mailing List (http://cvs.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.