]> granicus.if.org Git - php/commitdiff
Revert "Fixed bug #73067 (__debugInfo crashes when throwing an exception)"
authorJoe Watkins <krakjoe@php.net>
Mon, 17 Oct 2016 13:22:55 +0000 (14:22 +0100)
committerJoe Watkins <krakjoe@php.net>
Mon, 17 Oct 2016 13:22:55 +0000 (14:22 +0100)
This reverts commit 2d8ab51576695630a7471ff829cc5ea10becdc0f.

NEWS
Zend/tests/bug73067.phpt [deleted file]
Zend/zend_object_handlers.c

diff --git a/NEWS b/NEWS
index 863c07b860cdd0fdb8b70b74a6dcca046db1d8f9..0b87e6a7cb5a6fe6e8a747cf7304eb751b635492 100644 (file)
--- a/NEWS
+++ b/NEWS
@@ -133,8 +133,6 @@ PHP                                                                        NEWS
 15 Sep 2016, PHP 7.1.0RC2
 
 - Core:
-  . Fixed bug #73067 (__debugInfo crashes when throwing an exception).
-    (Laruence)
   . Fixed bug #73025 (Heap Buffer Overflow in virtual_popen of
     zend_virtual_cwd.c). (cmb)
   . Fixed bug #73058 (crypt broken when salt is 'too' long). (Anatol)
diff --git a/Zend/tests/bug73067.phpt b/Zend/tests/bug73067.phpt
deleted file mode 100644 (file)
index f5d34a8..0000000
+++ /dev/null
@@ -1,21 +0,0 @@
---TEST--
-Bug #73067 (__debugInfo crashes when throwing an exception)
---FILE--
-<?php
-
-class Debug
-{
-       public function __debugInfo()
-       {
-               throw new Exception("Test Crash");
-       }
-}
-
-try {
-       var_dump(new Debug());
-} catch (Exception $e) {
-       echo $e->getMessage();
-}
-?>
---EXPECTF--
-Fatal error: __debuginfo() must return an array in %sbug73067.php on line %d
index d57bee862ad427cdbbfa35448f847c59a80f2783..434102035c37269eac06abef2d7998336effa67e 100644 (file)
@@ -182,7 +182,6 @@ ZEND_API HashTable *zend_std_get_debug_info(zval *object, int *is_temp) /* {{{ *
                return ht;
        }
 
-       zend_clear_exception();
        zend_error_noreturn(E_ERROR, ZEND_DEBUGINFO_FUNC_NAME "() must return an array");
 
        return NULL; /* Compilers are dumb and don't understand that noreturn means that the function does NOT need a return value... */