]> granicus.if.org Git - php/commitdiff
Revert "Fixed bug #73067 (__debugInfo crashes when throwing an exception)"
authorAnatol Belski <ab@php.net>
Tue, 11 Oct 2016 09:24:08 +0000 (11:24 +0200)
committerAnatol Belski <ab@php.net>
Tue, 11 Oct 2016 09:24:08 +0000 (11:24 +0200)
This reverts commit 2d8ab51576695630a7471ff829cc5ea10becdc0f.

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

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 39c4d3d7e8220de09b9327a9c084277f9d47ad31..70dab660b3c4fa34451b33a6bf81b456c4817b11 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... */