ACPR/android_system_core
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Merge "Make the host_init_verifier init.rc parser follow symlinks."

Browse source
This commit is contained in:
Jingwen Chen 2023-03-17 06:42:04 +00:00 • committed by Gerrit Code Review
commit 095374cde7
4 changed files with 7 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
init/fuzzer/init_parser_fuzzer.cpp
View file

@ -125,7 +125,7 @@ void InitParserFuzzer::InvokeParser() {
std::string path = fdp_.ConsumeBool() ? fdp_.PickValueInArray(kValidPaths)
: fdp_.ConsumeRandomLengthString(kMaxBytes);
parser.ParseConfig(path);
parser.ParseConfigFileInsecure(path);
parser.ParseConfigFileInsecure(path, false /* follow_symlinks */);
}
void InitParserFuzzer::Process() {

4
init/host_init_verifier.cpp
View file

@ -326,7 +326,9 @@ int main(int argc, char** argv) {
}
}
} else {
if (!parser.ParseConfigFileInsecure(*argv)) {
if (!parser.ParseConfigFileInsecure(*argv, true /* follow_symlinks */)) {
// Follow symlinks as inputs during build execution in Bazel's
// execution root are symlinks, unlike Soong or Make.
LOG(ERROR) << "Failed to open init rc script '" << *argv << "'";
return EXIT_FAILURE;
}

4
init/parser.cpp
View file

@ -131,9 +131,9 @@ void Parser::ParseData(const std::string& filename, std::string* data) {
}
}
bool Parser::ParseConfigFileInsecure(const std::string& path) {
bool Parser::ParseConfigFileInsecure(const std::string& path, bool follow_symlinks = false) {
std::string config_contents;
if (!android::base::ReadFileToString(path, &config_contents)) {
if (!android::base::ReadFileToString(path, &config_contents, follow_symlinks)) {
return false;
}

2
init/parser.h
View file

@ -77,7 +77,7 @@ class Parser {
void AddSingleLineParser(const std::string& prefix, LineCallback callback);
// Host init verifier check file permissions.
bool ParseConfigFileInsecure(const std::string& path);
bool ParseConfigFileInsecure(const std::string& path, bool follow_symlinks);
size_t parse_error_count() const { return parse_error_count_; }