asp.net 核心中的路径遍历漏洞

Question

我已经尝试过这些解决方案

Does my code prevent directory traversal in C#?

How to avoid Directory Traversal in my code

但是，Checkmarx 扫描仍然存在安全问题，下面是代码

public FileContentResult GetImageFromFilePath(string filePath)
        {
            byte[] image = null;
            if (!string.IsNullOrEmpty(filePath))
            {
                image = System.IO.File.ReadAllBytes(filePath);
            }
            if (image == null)
                return null;
            return File(image, "image/jpeg");
        }

问题出在这行代码上image = System.IO.File.ReadAllBytes(filePath);

请告诉我如何解决此安全问题。

提前致谢。

Answer 1

Path Traversal是关于你从用户输入构建路径，主要是你对用户输入有一个假设，例如用户给你year 和 index，你 return 正确的图像：$"App/Photos/${year}/${index}.png".

遍历就是用户给你一个相对的部分，比如对于year-../private这样路径就会是App/private/1.png.

为了净化这个例子（你需要考虑更多的情况，不要做任何假设），你需要使用 filePath.Replace("..", "")。在使用 filePath 访问文件系统之前。

asp.net 核心中的路径遍历漏洞

A Path Traversal vulnerability in asp.net core

c#

asp.net-core

checkmarx

secure-coding

path-traversal