Commit 82bb89b9 authored Dec 09, 2014 by Diogo Ferreira

appops: Fix an off-by-one error that causes the location icon to stay on

Commit bdfde4fc added Bluetooth AppOps
capabilities, but it contained a subtle bug where the string representation
was added in the wrong place, offseting every operation after it by 1. This
broke the toString functionality but didn't have any consequence since this
was used for logging purposes only.

Commit b01b0892 changed AppOps so that
operations are persisted by name instead of by index, this caused the string
representation offset to affect functionality and, among other things, one
of the consequences was that coarse location requests were being interpreted
by AppOps as high-energy location requests.

Change-Id: I319f1ac1c5910098689b4cb8645eb92c0e3538cb

parent 45255743

core/java/android/app/AppOpsManager.java

+1 −1

Original line number	Diff line number	Diff line
		@@ -490,7 +490,6 @@ public class AppOpsManager {
		null,
		null,
		null,
		null,
		OPSTR_MONITOR_LOCATION,
		OPSTR_MONITOR_HIGH_POWER_LOCATION,
		OPSTR_GET_USAGE_STATS,
		@@ -509,6 +508,7 @@ public class AppOpsManager {
		null,
		null,
		null,
		null,
		};

		/**