Fixes #801 libxml_disable_entity_loader() is changed

src/PhpSpreadsheet/IOFactory.php

@@ -147,6 +147,7 @@ public static function createReaderForFile($filename)
                 if ($reader->canRead($filename)) {
                     return $reader;
                 }
+                unset($reader);
             }
         }
 

src/PhpSpreadsheet/Reader/Security/XmlScanner.php

@@ -14,7 +14,7 @@ class XmlScanner
     private $libxmlDisableEntityLoader = false;
 
     /**
-     * Store the initial setting of libxmlDisableEntityLoader so that we can resore t later.
+     * Stores the initial setting of libxmlDisableEntityLoader so that we can restore it later.
      *
      * @var bool
      */

tests/PhpSpreadsheetTests/IOFactoryTest.php

@@ -4,6 +4,7 @@
 
 use PhpOffice\PhpSpreadsheet\IOFactory;
 use PhpOffice\PhpSpreadsheet\Reader;
+use PhpOffice\PhpSpreadsheet\Shared\File;
 use PhpOffice\PhpSpreadsheet\Spreadsheet;
 use PhpOffice\PhpSpreadsheet\Writer;
 use PHPUnit\Framework\TestCase;
@@ -144,4 +145,35 @@ public function testRegisterInvalidReader()
 
         IOFactory::registerReader('foo', 'bar');
     }
+
+    /**
+     * @dataProvider providerBoolean
+     *
+     * @param bool $startWith
+     */
+    public function testLibxmlEntityLoaderValue($startWith)
+    {
+        $filename = tempnam(File::sysGetTempDir(), 'phpspreadsheet-test');
+
+        $valueBeforeTest = libxml_disable_entity_loader($startWith);
+
+        try {
+            IOFactory::createReaderForFile($filename);
+            self::assertFalse(true);
+        } catch (\Exception $e) {
+        }
+        $finalValue = libxml_disable_entity_loader($valueBeforeTest);
+        self::assertSame(
+            $startWith,
+            $finalValue,
+            sprintf('Started with: %b value at the end: %b', $startWith, $finalValue)
+        );
+
+        unlink($filename);
+    }
+
+    public function providerBoolean()
+    {
+        return [[true], [false]];
+    }
 }