diff --git a/init/init_parser.cpp b/init/init_parser.cpp index b44ca59d5..f2e5d6df9 100644 --- a/init/init_parser.cpp +++ b/init/init_parser.cpp @@ -122,14 +122,20 @@ bool Parser::ParseConfigDir(const std::string& path) { return false; } dirent* current_file; + std::vector files; while ((current_file = readdir(config_dir.get()))) { - std::string current_path = - android::base::StringPrintf("%s/%s", path.c_str(), current_file->d_name); // Ignore directories and only process regular files. if (current_file->d_type == DT_REG) { - if (!ParseConfigFile(current_path)) { - ERROR("could not import file '%s'\n", current_path.c_str()); - } + std::string current_path = + android::base::StringPrintf("%s/%s", path.c_str(), current_file->d_name); + files.emplace_back(current_path); + } + } + // Sort first so we load files in a consistent order (bug 31996208) + std::sort(files.begin(), files.end()); + for (const auto& file : files) { + if (!ParseConfigFile(file)) { + ERROR("Could not import file '%s'\n", file.c_str()); } } return true;