commit 90e325670171c43f5a95c25881ea3565563d87b1
author xplodwild <xplod@ouverta.fr> Thu Nov 07 13:30:45 2013 +0100
committer Nick Reuter <nreuter85@gmail.com> Sat Jun 14 18:28:06 2014 -0700
tree e1c9d57d29a83e60d6ba44838771e53525908a68
parent 2eda12ed8b6715ec7feabf4122bde74b26b96c63

healthd: Only use battery values that seems legit

On some devices, there are other sysfs paths having a "capacity"
node than just the battery, which can return a weirdo value. We
make sure we only use legit battery levels (> 0%).

Change-Id: I3caf34e7fed4b89f953ec98c486b53b8786c8f74

healthd/BatteryMonitor.cpp

diff --git a/healthd/BatteryMonitor.cpp b/healthd/BatteryMonitor.cpp
index 688c7ff..7fd51a0 100644
--- a/healthd/BatteryMonitor.cpp
+++ b/healthd/BatteryMonitor.cpp
@@ -331,7 +331,7 @@
                     path.clear();
                     path.appendFormat("%s/%s/capacity", POWER_SUPPLY_SYSFS_PATH,
                                       name);
-                    if (access(path, R_OK) == 0)
+                    if (access(path, R_OK) == 0 && getIntField(path) > 0)
                         mHealthdConfig->batteryCapacityPath = path;
                 }