澄清整个PHP版本的XXE漏洞

$xml = <<<XML
<?xml version="1.0"?>
<!DOCTYPE root [
<!ENTITY c PUBLIC "bar" "/etc/passwd">
]>
<root>
    <test>Test</test>
    <sub>&c;</sub>
</root>
XML;

libxml_disable_entity_loader(true);
$dom = new DOMDocument();
$dom->loadXML($xml);

// Prints Test.
print $dom->textContent;

$xml = <<<XML
<?xml version="1.0"?>
<!DOCTYPE root [
<!ENTITY c PUBLIC "bar" "/etc/passwd">
]>
<root>
    <test>Test</test>
    <sub>&c;</sub>
</root>
XML;

$dom = new DOMDocument();
$dom->loadXML($xml,LIBXML_NOENT | LIBXML_DTDLOAD);

// Prints Test.
print $dom->textContent;

$xml = <<<XML
<?xml version="1.0"?>
<!DOCTYPE root [
<!ENTITY c "Blah blah">
]>
<root>
    <test>Test</test>
    <sub>&c;</sub>
</root>
XML;

$dom = new DOMDocument();
$dom->loadXML($xml);

// Prints Test.
print $dom->textContent;

保持简单..因为它应该很简单:-)

你的第一个代码片段

libxml_disable_entity_loader根据您的系统默认情况下是否解析实体(我的)不执行或不执行任何操作.这由libxml的LIBXML_NOENT选项控制.

没有它,文档处理器可能甚至不会尝试翻译外部实体,因此libxml_disable_entity_loader没有任何真正的影响(如果libxml未在默认情况下加载实体,这在您的测试用例中似乎是这种情况).

将LIBXML_NOENT添加到loadXML(),如下所示：

$dom->loadXML($xml,LIBXML_NOENT);

你会很快得到：

PHP Warning:  DOMDocument::loadXML(): I/O warning : failed to load external entity "/etc/passwd" in ...
PHP Warning:  DOMDocument::loadXML(): Failure to process entity c in Entity,line: 7 in ...
PHP Warning:  DOMDocument::loadXML(): Entity 'c' not defined in Entity,line: 7 in ...

你的第二个代码片段

在这种情况下,您通过使用LIBXML_NOENT选项启用实体解析,这就是为什么它在/ etc / passwd之后.

该示例在我的机器上正常工作,即使是外部URL – 我将ENTITY更改为外部URL：

<!ENTITY c PUBLIC "bar" "https://stackoverflow.com/opensearch.xml">

然而,它甚至可以受到例如的影响. allow_url_fopen PHP INI设置 – 将其设置为false,PHP将不会加载远程文件.

你的第三个代码片段

您提供的XML实体不是外部的,而是内部的(参见例如here).

您的实体：

<!ENTITY c "Blah blah">

内部实体的定义如何：

<!ENTITY % name "entity_value">

因此,PHP或libxml没有任何理由阻止此类实体的解析.

结论

我已经很快地建立了一个PHP XXE tester script,它尝试了不同的设置,并显示XXE是否成功,在哪种情况下.

实际显示警告的唯一一行是“LIBXML_NOENT”.

如果任何其他线路加载警告,外部实体加载！您的设置默认允许加载外部实体.

使用SHOULD USE libxml_disable_entity_loader(),无论您的/您的提供商的机器默认设置如何,您都不能出错.如果你的应用程序被迁移,它可能会立即变得脆弱.

正确使用

正如MediaWiki在link you’ve posted所说.

Unfortunately,the way that libxml2 implements the disabling,the library is crippled when external entities are disabled,and functions that would otherwise be safe cause an exception in the entire parsing.

$oldValue = libxml_disable_entity_loader(true);
// do whatever XML-processing related
libxml_disable_entity_loader($oldValue);

注意：libxml_disable_entity_loader()还禁止直接加载外部xml文件(而不是通过实体)：

<?php
$remote_xml = "https://stackoverflow.com/opensearch.xml";

$dom = new DOMDocument();

if ($dom->load($remote_xml) !== FALSE)
    echo "loaded remote xml!n";
else
    echo "failed to load remote xml!n";

libxml_disable_entity_loader(true);
if ($dom->load($remote_xml) !== FALSE)
    echo "loaded remote xml after libxml_disable_entity_loader(true)!n";
else 
    echo "failed to remote xml after libxml_disable_entity_loader(true)!n";

在我的机器上

loaded remote xml!
PHP Warning:  DOMDocument::load(): I/O warning : failed to load external entity "https://stackoverflow.com/opensearch.xml" in ...
failed to remote xml after libxml_disable_entity_loader(true)!

这可能与this PHP bug有关,但PHP真的很愚蠢：

libxml_disable_entity_loader(true);
$dom->loadXML(file_get_contents($remote_xml));

工作很好