From 4dc50527d87a0f5a05a35caf5398eb7aec162213 Mon Sep 17 00:00:00 2001 From: Jeff Sharkey Date: Tue, 17 Oct 2017 15:29:41 -0600 Subject: [PATCH] Relax checks on dynamic permissions. We'll have to circle back to enforce this once we do more investigation. Test: none Bug: 67371907 Change-Id: Ib2f73b6e52d56cf080f3a6f224ab4bf452d91a48 --- .../android/server/pm/permission/PermissionManagerService.java | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/services/core/java/com/android/server/pm/permission/PermissionManagerService.java b/services/core/java/com/android/server/pm/permission/PermissionManagerService.java index 062aa3310c21..d2d857caa240 100644 --- a/services/core/java/com/android/server/pm/permission/PermissionManagerService.java +++ b/services/core/java/com/android/server/pm/permission/PermissionManagerService.java @@ -417,8 +417,8 @@ public class PermissionManagerService { bp = new BasePermission(info.name, tree.getSourcePackageName(), BasePermission.TYPE_DYNAMIC); } else if (bp.isDynamic()) { - throw new SecurityException( - "Not allowed to modify non-dynamic permission " + // TODO: switch this back to SecurityException + Slog.wtf(TAG, "Not allowed to modify non-dynamic permission " + info.name); } changed = bp.addToTree(fixedLevel, info, tree); @@ -444,8 +444,8 @@ public class PermissionManagerService { return; } if (bp.isDynamic()) { - throw new SecurityException( - "Not allowed to modify non-dynamic permission " + // TODO: switch this back to SecurityException + Slog.wtf(TAG, "Not allowed to modify non-dynamic permission " + permName); } mSettings.removePermissionLocked(permName); -- 2.11.0