Add sanity check for battery capacity range. (38387c36) · Commits · e / os / android_frameworks_base

services/jni/com_android_server_BatteryService.cpp

+20 −1

Original line number	Diff line number	Diff line
		@@ -181,6 +181,25 @@ static void setIntField(JNIEnv* env, jobject obj, const char* path, jfieldID fie
		env->SetIntField(obj, fieldID, value);
		}

		static void setPercentageField(JNIEnv* env, jobject obj, const char* path, jfieldID fieldID)
		{
		const int SIZE = 128;
		char buf[SIZE];

		jint value = 0;
		if (readFromFile(path, buf, SIZE) > 0) {
		value = atoi(buf);
		}
		/* sanity check for buggy drivers that provide bogus values, e.g. 103% */
		if (value < 0) {
		value = 0;
		} else if (value > 100) {
		value = 100;
		}

		env->SetIntField(obj, fieldID, value);
		}

		static void setVoltageField(JNIEnv* env, jobject obj, const char* path, jfieldID fieldID)
		{
		const int SIZE = 128;
		@@ -201,7 +220,7 @@ static void android_server_BatteryService_update(JNIEnv* env, jobject obj)
		setBooleanField(env, obj, gPaths.usbOnlinePath, gFieldIds.mUsbOnline);
		setBooleanField(env, obj, gPaths.batteryPresentPath, gFieldIds.mBatteryPresent);

		setIntField(env, obj, gPaths.batteryCapacityPath, gFieldIds.mBatteryLevel);
		setPercentageField(env, obj, gPaths.batteryCapacityPath, gFieldIds.mBatteryLevel);
		setVoltageField(env, obj, gPaths.batteryVoltagePath, gFieldIds.mBatteryVoltage);
		setIntField(env, obj, gPaths.batteryTemperaturePath, gFieldIds.mBatteryTemperature);