Check for path permissions before opening file in the CLI file reader

This commit is contained in:
Alex Beregszaszi 2017-04-20 20:01:17 +01:00
parent 965de29772
commit b30fad4a49

View File

@ -638,8 +638,6 @@ bool CommandLineInterface::processInput()
ReadFile::Callback fileReader = [this](string const& _path)
{
auto path = boost::filesystem::path(_path);
if (!boost::filesystem::exists(path))
return ReadFile::Result{false, "File not found."};
auto canonicalPath = boost::filesystem::canonical(path);
bool isAllowed = false;
for (auto const& allowedDir: m_allowedDirectories)
@ -656,6 +654,8 @@ bool CommandLineInterface::processInput()
}
if (!isAllowed)
return ReadFile::Result{false, "File outside of allowed directories."};
else if (!boost::filesystem::exists(path))
return ReadFile::Result{false, "File not found."};
else if (!boost::filesystem::is_regular_file(canonicalPath))
return ReadFile::Result{false, "Not a valid file."};
else