commit db15b6f93d828d0c4a9705e845b41d48be22b6f2
author Kelvin Zhang <zhangkelvin@google.com> Wed May 03 15:28:39 2023 -0700
committer Kelvin Zhang <zhangkelvin@google.com> Tue Jun 27 13:53:43 2023 -0700
tree b056e6d295b780141f180c2ea3ea94d0a2a97f6c
parent 786dac3d5068594735ef38f4a51bbb89e9a12003

Improve error message of libmodprobe

Log path of module if that module is not found.

Bug: 273752147
Test: th
Change-Id: I428a116ee26f97455229df642acbdc1297098a32

libmodprobe/libmodprobe.cpp

diff --git a/libmodprobe/libmodprobe.cpp b/libmodprobe/libmodprobe.cpp
index 858b955..d20b6b1 100644
--- a/libmodprobe/libmodprobe.cpp
+++ b/libmodprobe/libmodprobe.cpp
@@ -230,7 +230,7 @@
     }
 
     std::vector<std::string> lines = android::base::Split(cfg_contents, "\n");
-    for (const std::string line : lines) {
+    for (const auto& line : lines) {
         if (line.empty() || line[0] == '#') {
             continue;
         }
@@ -421,7 +421,8 @@
     }
 
     if (strict && !module_loaded) {
-        LOG(ERROR) << "LoadWithAliases was unable to load " << module_name;
+        LOG(ERROR) << "LoadWithAliases was unable to load " << module_name
+                   << ", tried: " << android::base::Join(modules_to_load, ", ");
         return false;
     }
     return true;