如何在 .NET 反序列化期间防止 XML 外部实体 （XXE） 攻击

Question

我们正在使用 veracode 对代码进行安全分析，它显示了以下代码的 XXE 缺陷，特别是在调用 Deserialize（） 的地方。我们如何防止序列化程序访问外部实体。我在下面尝试将XMLresolver设置为XMLReader的null不起作用。

    public static T DeserializeObject(string xml, string Namespace)
    {
        System.Xml.Serialization.XmlSerializer serializer = new System.Xml.Serialization.XmlSerializer(typeof(T), Namespace);

        MemoryStream stream =
                new MemoryStream(Encoding.Default.GetBytes(xml));
        XmlReaderSettings settings = new XmlReaderSettings();

        // allow entity parsing but do so more safely
        settings.DtdProcessing = DtdProcessing.Ignore;
        settings.XmlResolver = null;

        using (XmlReader reader = XmlReader.Create(stream, settings))
        {
            return serializer.Deserialize(reader) as T;
        }
    }

有人能建议我可能缺少什么吗？或者是否有其他东西可以尝试。

Answer 1

我也有类似的问题。当您从字符串中读取时，您需要使用xmlTextReader更改xmlReader。

像这样的东西-

  public static T DeserializeObject(string xml, string Namespace)
  {
        System.Xml.Serialization.XmlSerializer serializer = new System.Xml.Serialization.XmlSerializer(typeof(T), Namespace);

        //**** I don't think you need this block of code *********
        //MemoryStream stream = new MemoryStream(Encoding.Default.GetBytes(xml));
        //XmlReaderSettings settings = new XmlReaderSettings();

        // allow entity parsing but do so more safely
        //settings.DtdProcessing = DtdProcessing.Ignore;
        //settings.XmlResolver = null;
        //*********************************************

        XmlTextReader reader = new XmlTextReader(xml)
        {
            XmlResolver = null
        };

        return serializer.Deserialize(reader) as T;
  }

万事如意！