go - 如何处理 gosec linter 警告 : Potential file inclusion via variable

Question

如何解决来自 gosec 的以下警告 Lint :

::warning: Potential file inclusion via variable,MEDIUM,HIGH (gosec)

linter 在这个函数的第一行警告我:

func File2lines(filePath string) ([]string, error) {
    f, err := os.Open(filePath) //Warning here
    if err != nil {
        return nil, err
    }
    defer f.Close()
    return linesFromReader(f)
}

我曾尝试阅读有关本地文件包含的内容，但看不出这在此处如何适用。

Answer 1

路径从何而来？如果你不确定它永远不会有用户输入，最好在使用前清理它并使用已知的前缀，例如:

filePath = filepath.Join(basePath,filepath.Clean(filePath))
f, err := os.Open(filePath)

这应该可以解决投诉。这是一个合理的预防措施，即使您现在认为它是安全的，以防以后有人使用您的函数处理用户数据。