[VarDumper] fix dumping uninitialized SplFileInfo

nicolas-grekas · nicolas-grekas · commit 7d8450e4943d · 2019-10-04T09:39:26.000+02:00
diff --git a/src/Symfony/Component/VarDumper/Caster/SplCaster.php b/src/Symfony/Component/VarDumper/Caster/SplCaster.php
@@ -90,6 +90,12 @@ public static function castFileInfo(\SplFileInfo $c, array $a, Stub $stub, $isNe
 
         $prefix = Caster::PREFIX_VIRTUAL;
 
+        if (false === $c->getPathname()) {
+            $a[$prefix.'⚠'] = 'The parent constructor was not called: the object is in an invalid state';
+
+            return $a;
+        }
+
         foreach ($map as $key => $accessor) {
             try {
                 $a[$prefix.$key] = $c->$accessor();
diff --git a/src/Symfony/Component/VarDumper/Tests/Caster/SplCasterTest.php b/src/Symfony/Component/VarDumper/Tests/Caster/SplCasterTest.php
@@ -205,9 +205,32 @@ public function testArrayIterator()
 EOTXT;
         $this->assertDumpEquals($expected, $var);
     }
+
+    public function testBadSplFileInfo()
+    {
+        if (\defined('HHVM_VERSION')) {
+            $this->markTestSkipped('HHVM as different internal details.');
+        }
+        $var = new BadSplFileInfo();
+
+        $expected = <<<EOTXT
+Symfony\Component\VarDumper\Tests\Caster\BadSplFileInfo {
+  ⚠: "The parent constructor was not called: the object is in an invalid state"
+}
+EOTXT;
+        $this->assertDumpEquals($expected, $var);
+
+    }
 }
 
 class MyArrayIterator extends \ArrayIterator
 {
     private $foo = 123;
 }
+
+class BadSplFileInfo extends \SplFileInfo
+{
+    public function __construct()
+    {
+    }
+}
