如何從Fortify掃描報告中修復以下代碼示例的“路徑操縱”問題

Question

我有路徑操作問題。 以下代碼放置在ASPx頁面的Page_load方法中。

String rName = Request.QueryString["reportName"];
string path = "C:\\hari" + rName;
if (File.Exists(path))
{
    File.Delete(path);
}

但是上述示例代碼的Fortify掃描報告顯示“路徑操縱”問題為高。需要幫助來修改上述代碼，以便它可以通過強化掃描

Answer 1

Jackson是對的，這是一個直接的文件路徑操縱漏洞，可以通過間接選擇來修復。 從您的已知目錄中，列出所有文件。 使用來自您自己的目錄列表的值，而不是用戶提供的值。

String rName = Request.QueryString["reportName"];
String knownPath = "C:\\hari";
DirectoryInfo di = new DirectoryInfo(knownPath);
FileInfo[] files = di.GetFiles(rName);

if (files.length > 0)
{
    files[0].Delete();
}

Answer 2

我認為問題在於，有人可能會以reportName = "..\\\\Windows\\\\Something important"欺騙請求，這顯然是安全缺陷。 您需要更改代碼，以使其不會從請求查詢字符串中讀取部分文件名。