2
0
Просмотр исходного кода

Merge pull request #410 from hella-gutmann/issue-393

Fix for issue #393 - Always reset libxml_disable_entity_loader
Frank Brückner 11 лет назад
Родитель
Сommit
822a7a4aa0
1 измененных файлов с 6 добавлено и 11 удалено
  1. 6 11
      library/Zend/Xml/Security.php

+ 6 - 11
library/Zend/Xml/Security.php

@@ -93,12 +93,13 @@ class Zend_Xml_Security
         $result = $dom->loadXml($xml, LIBXML_NONET);
         $result = $dom->loadXml($xml, LIBXML_NONET);
         restore_error_handler();
         restore_error_handler();
 
 
+        // Entity load to previous setting
+        if (!self::isPhpFpm()) {
+            libxml_disable_entity_loader($loadEntities);
+            libxml_use_internal_errors($useInternalXmlErrors);
+        }
+
         if (!$result) {
         if (!$result) {
-            // Entity load to previous setting
-            if (!self::isPhpFpm()) {
-                libxml_disable_entity_loader($loadEntities);
-                libxml_use_internal_errors($useInternalXmlErrors);
-            }
             return false;
             return false;
         }
         }
 
 
@@ -114,12 +115,6 @@ class Zend_Xml_Security
             }
             }
         }
         }
 
 
-        // Entity load to previous setting
-        if (!self::isPhpFpm()) {
-            libxml_disable_entity_loader($loadEntities);
-            libxml_use_internal_errors($useInternalXmlErrors);
-        }
-
         if (isset($simpleXml)) {
         if (isset($simpleXml)) {
             $result = simplexml_import_dom($dom);
             $result = simplexml_import_dom($dom);
             if (!$result instanceof SimpleXMLElement) {
             if (!$result instanceof SimpleXMLElement) {