我们对代码进行了安全审核,其中提到我们的代码容易受到 XML EXternal Entity (XXE) 攻击。
Explanation
XML External Entities attacks benefit from an XML feature to build documents dynamically at the time of processing. An XML entity allows inclusion of data dynamically from a given resource. External entities allow an XML document to include data from an external URI. Unless configured to do otherwise, external entities force the XML parser to access the resource specified by the URI, e.g., a file on the local machine or on a remote system. This behavior exposes the application to XML External Entity (XXE) attacks, which can be used to perform denial of service of the local system, gain unauthorized access to files on the local machine, scan remote machines, and perform denial of service of remote systems.
The following XML document shows an example of an XXE attack.
<?xml version="1.0" encoding="ISO-8859-1"?> <!DOCTYPE foo [ <!ELEMENT foo ANY > <!ENTITY xxe SYSTEM "file:///dev/random" >]><foo>&xxe;</foo>
This example could crash the server (on a UNIX system), if the XML parser attempts to substitute the entity with the contents of the /dev/random file.
Recommendation
The XML unmarshaller should be configured securely so that it does not allow external entities as part of an incoming XML document.
To avoid XXE injection do not use unmarshal methods that process an XML source directly as
java.io.File
,java.io.Reader
orjava.io.InputStream
. Parse the document with a securely configured parser and use an unmarshal method that takes the secure parser as the XML source as shown in the following example:DocumentBuilderFactory dbf = DocumentBuilderFactory.newInstance(); dbf.setExpandEntityReferences(false); DocumentBuilder db = dbf.newDocumentBuilder(); Document document = db.parse(<XML Source>); Model model = (Model) u.unmarshal(document);
以下代码是审计发现 XXE 攻击的地方:
Transformer transformer = TransformerFactory.newInstance().newTransformer();
transformer.setOutputProperty(OutputKeys.INDENT, "yes");
System.out.println("outputing to : " + outputLocation);
File outputFile = new File(outputLocation);
StreamResult result = new StreamResult(outputFile);
DOMSource source = new DOMSource(doc);
transformer.transform(source, result);
如何在我的代码中实现上述建议?我在哪里遗漏了东西?
最佳答案
您可以对 DocumentBuilderFactory
使用相同的方法:
DocumentBuilderFactory factory = DocumentBuilderFactory.newInstance();
factory.setFeature(XMLConstants.FEATURE_SECURE_PROCESSING, true);
...
为了让每个人都自动使用它,您需要创建自己的实现(通过扩展您当前使用的实现;使用调试器来查找)。在构造函数中设置该功能。
然后,您可以将要在系统属性 javax.xml.parsers.DocumentBuilderFactory
中使用的新工厂传递给 Java VM,每个人都会使用它。
关于java - 如何防范XXE攻击,我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/57047719/