php 8 compatibility in Zend_Xml_Security class

magento · Sep 10, 2021 · c5206c6 · c5206c6
1 parent 3f2af6c
commit c5206c6
Showing 1 changed file with 10 additions and 3 deletions.
diff --git a/library/Zend/Xml/Security.php b/library/Zend/Xml/Security.php
@@ -83,7 +83,10 @@ public static function scan($xml, DOMDocument $dom = null)
         }
 
         if (!self::isPhpFpm()) {
-            $loadEntities = libxml_disable_entity_loader(true);
+            if (LIBXML_VERSION < 20900) {
+                // this function no longer has an effect in PHP 8.0, but it's required in earlier versions
+                $loadEntities = libxml_disable_entity_loader(true);
+            }
             $useInternalXmlErrors = libxml_use_internal_errors(true);
         }
 
@@ -97,7 +100,9 @@ public static function scan($xml, DOMDocument $dom = null)
         if (!$result) {
             // Entity load to previous setting
             if (!self::isPhpFpm()) {
-                libxml_disable_entity_loader($loadEntities);
+                if (isset($loadEntities)) {
+                    libxml_disable_entity_loader($loadEntities);
+                }
                 libxml_use_internal_errors($useInternalXmlErrors);
             }
             return false;
@@ -117,7 +122,9 @@ public static function scan($xml, DOMDocument $dom = null)
 
         // Entity load to previous setting
         if (!self::isPhpFpm()) {
-            libxml_disable_entity_loader($loadEntities);
+            if (isset($loadEntities)) {
+                libxml_disable_entity_loader($loadEntities);
+            }
             libxml_use_internal_errors($useInternalXmlErrors);
         }