StreamReader ReadLine 内存泄漏

Question

我正在制作一个服务器，我注意到它有一点奇怪的内存泄漏。在我的服务器上，我将一个 IP 地址可以建立的总连接数限制为 8。这非常有效。

但是，当我使用 LOIC 对我的服务器进行 DoS 攻击时，我注意到虽然它只创建了网络类的 8 个实例，但 StreamReader 的 ReadLine 方法占用了很大一部分内存。

我已经使用 Console.WriteLine 验证了实际上没有收到任何数据，它只是在该方法处等待。内存增加并填满了我的整个 16GB。

请注意，这个类的实例只有 8 个来自一个 IP，而且由于我是 DoS 攻击而不是 DDoS 攻击，它只来自我的 IP（我已经验证了这一点）。

这个问题的原因是什么，我该如何解决？

编辑：
相关代码为：

        string data;
        Dictionary<string, string> variables = null;
        try {

            while (_listen) {

                data = null;

                while ((data = _reader.ReadLine()) != null) {

Answer 1

ReadLine 函数会将数据缓冲到一个字符串中，它遇到\r\n（单独\n 不会这样做）。它为此使用StringBuilder。因此，如果您收到的数据不包含序列\r\n，它最终会为单个字符串分配大量内存。

您有一个简单的解决方案：您必须限制服务器愿意读取的最大字符串长度。这是处理恶意数据所必需的 - 如果您的服务器可能是 DoS 攻击的受害者，请不要相信您收到的数据。

所以，解决方案，作为TextReader 扩展方法：

public static String ReadLineSafe(this TextReader reader, int maxLength)
{ 
    var sb = new StringBuilder();
    while (true) {
        int ch = reader.Read();
        if (ch == -1) break; 
        if (ch == '\r' || ch == '\n')
        { 
            if (ch == '\r' && reader.Peek() == '\n') reader.Read(); 
            return sb.ToString();
        } 
        sb.Append((char)ch);

        // Safety net here
        if (sb.Length > maxLength)
            throw new InvalidOperationException("Line is too long");
    }
    if (sb.Length > 0) return sb.ToString();
    return null; 
}

这是原始的ReadLine 代码，添加了安全检查。

哦，您应该将NetworkStream 包装在BufferedStream 中以供网络读取，以避免在每个字节上调用recv。