Merge pull request #41 from eric-ch/oxt-1012

OXT-1012: xcpmd: Check for xenstore_ls return value.
OpenXT · Apr 6, 2017 · b695a5a · b695a5a
2 parents 224684e + 3d04a96
commit b695a5a
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/xcpmd/src/battery.c b/xcpmd/src/battery.c
@@ -346,6 +346,12 @@ static void make_xenstore_battery_dir(unsigned int battery_index) {
     bool flag;
 
     dir_entries = xenstore_ls(&num_entries, "/pm");
+    if (!dir_entries) {
+        xcpmd_log(LOG_WARNING,
+            "Listing directory /pm failed with error `%s'\n", strerror(errno));
+        return;
+    }
+
     snprintf(xenstore_path, 255, "%s%i", XS_BATTERY_PATH, battery_index);
 
     flag = false;