[Debug] Remove GLOBALS from exception context to avoid endless recursion

Seldaek · Seldaek · commit a325f1f2f741 · 2016-11-14T21:45:45.000+01:00
diff --git a/src/Symfony/Component/Debug/ErrorHandler.php b/src/Symfony/Component/Debug/ErrorHandler.php
@@ -420,6 +420,14 @@ public function handleError($type, $message, $file, $line, array $context, array
                 self::$toStringException = null;
             } elseif (($this->scopedErrors & $type) && class_exists('Symfony\Component\Debug\Exception\ContextErrorException')) {
                 // Checking for class existence is a work around for https://bugs.php.net/42098
+
+                // if GLOBALS is included because an error occured in global scope, and it
+                // contains a reference to itself, we unset it to avoid endless recursion
+                // later on when the context is normalized and iterated over by Monolog
+                if (isset($context['GLOBALS']['GLOBALS']['GLOBALS'])) {
+                    unset($context['GLOBALS']);
+                }
+
                 $throw = new ContextErrorException($this->levels[$type].': '.$message, 0, $type, $file, $line, $context);
             } else {
                 $throw = new \ErrorException($this->levels[$type].': '.$message, 0, $type, $file, $line);
