commit cd8c886536693cb6bb45244e8b70ea4ffef359f9
author John Stultz <john.stultz@linaro.org> Thu Nov 07 13:49:20 2019 -0800
committer android-build-merger <android-build-merger@google.com> Thu Nov 07 13:49:20 2019 -0800
tree 8ebcd1a331478d2af6988096927083a04039dd45
parent c04eb2004188ae428788d45fe58a71aca7e31811
parent 6ca32eda90edf1ba0a30a2966821d38bf0b59cc8

Merge "healthd: Fix incorrect return check" am: ec56c59dbb
am: b4bd2480e3

Change-Id: I0be5f079cdf4aea182c67ac98952bf7ec5b5658b

healthd/BatteryMonitor.cpp

diff --git a/healthd/BatteryMonitor.cpp b/healthd/BatteryMonitor.cpp
index e80f368..bdf4aac 100644
--- a/healthd/BatteryMonitor.cpp
+++ b/healthd/BatteryMonitor.cpp
@@ -181,7 +181,7 @@
         return ANDROID_POWER_SUPPLY_TYPE_UNKNOWN;
 
     auto ret = mapSysfsString(buf.c_str(), supplyTypeMap);
-    if (ret < 0) {
+    if (!ret) {
         KLOG_WARNING(LOG_TAG, "Unknown power supply type '%s'\n", buf.c_str());
         *ret = ANDROID_POWER_SUPPLY_TYPE_UNKNOWN;
     }