summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Fyodor Kupolov <fkupolov@google.com> 2017-10-19 11:45:29 -0700
committer Fyodor Kupolov <fkupolov@google.com> 2017-10-19 11:47:47 -0700
commit4f41e849f1b7d6d037439d60eace1b134165a148 (patch)
tree0b16bab28ea84481008e74be8b8a48fbae89259a
parent89b377d8a3c76000d26a1a9297222a37c13766ab (diff)
Fail only if CONTROL_PRIVAPP_PERMISSIONS_ENFORCE
Previously the error was also thrown if ro.control_privapp_permissions=log Test: manual Bug: 64693550 Change-Id: Ia87984e2e5902dd885feeb37f676956fa56b421c
-rw-r--r--services/core/java/com/android/server/pm/PackageManagerService.java3
1 files changed, 2 insertions, 1 deletions
diff --git a/services/core/java/com/android/server/pm/PackageManagerService.java b/services/core/java/com/android/server/pm/PackageManagerService.java
index 275db1fa543d..717461229d10 100644
--- a/services/core/java/com/android/server/pm/PackageManagerService.java
+++ b/services/core/java/com/android/server/pm/PackageManagerService.java
@@ -12428,7 +12428,8 @@ public class PackageManagerService extends IPackageManager.Stub
.getPrivAppDenyPermissions(pkg.packageName);
final boolean permissionViolation =
deniedPermissions == null || !deniedPermissions.contains(perm);
- if (permissionViolation) {
+ if (permissionViolation
+ && RoSystemProperties.CONTROL_PRIVAPP_PERMISSIONS_ENFORCE) {
if (mPrivappPermissionsViolations == null) {
mPrivappPermissionsViolations = new ArraySet<>();
}