From 19f4a29fa44008703738fd5217366365f14f2649 Mon Sep 17 00:00:00 2001 From: Svetoslav Ganov Date: Wed, 12 Sep 2012 20:26:27 -0700 Subject: [PATCH] Enforcing BIND_ACCESSIBILITY_SERVICE for connecting to an accessibility service. 1. This change enforces an accessibility service to require the system defined BIND_ACCESSIBILITY_SERVICE permission. bug:6507771 Change-Id: If5e16bb4fa97891be0ccbb35e343773712e33b98 --- .../server/accessibility/AccessibilityManagerService.java | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/services/java/com/android/server/accessibility/AccessibilityManagerService.java b/services/java/com/android/server/accessibility/AccessibilityManagerService.java index f6354bb606c25..99ec1d2f3060b 100644 --- a/services/java/com/android/server/accessibility/AccessibilityManagerService.java +++ b/services/java/com/android/server/accessibility/AccessibilityManagerService.java @@ -682,11 +682,7 @@ public class AccessibilityManagerService extends IAccessibilityManager.Stub { for (int i = 0, count = installedServices.size(); i < count; i++) { ResolveInfo resolveInfo = installedServices.get(i); ServiceInfo serviceInfo = resolveInfo.serviceInfo; - // For now we are enforcing this if the target version is JellyBean or - // higher and in a later release we will enforce this for everyone. - if (serviceInfo.applicationInfo.targetSdkVersion >= Build.VERSION_CODES.JELLY_BEAN - && !android.Manifest.permission.BIND_ACCESSIBILITY_SERVICE.equals( - serviceInfo.permission)) { + if (!android.Manifest.permission.BIND_ACCESSIBILITY_SERVICE.equals(serviceInfo.permission)) { Slog.w(LOG_TAG, "Skipping accessibilty service " + new ComponentName( serviceInfo.packageName, serviceInfo.name).flattenToShortString() + ": it does not require the permission "