From b633f9d01f484d1a17ebfcd5aa4d6000dff53773 Mon Sep 17 00:00:00 2001 From: Fyodor Kupolov Date: Wed, 16 Nov 2016 10:15:45 -0800 Subject: [PATCH] Log when priv permission is not whitelisted Log violations for now, we will start enforcing it later. Test: Manual - device boots Bug: 32935038 Bug: 32935364 Change-Id: I1957c334f98852a5a87e093e24c72dd2c223c08b --- .../java/com/android/server/pm/PackageManagerService.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/services/core/java/com/android/server/pm/PackageManagerService.java b/services/core/java/com/android/server/pm/PackageManagerService.java index de801a5198708..f9759d92f8d1c 100644 --- a/services/core/java/com/android/server/pm/PackageManagerService.java +++ b/services/core/java/com/android/server/pm/PackageManagerService.java @@ -10407,9 +10407,9 @@ public class PackageManagerService extends IPackageManager.Stub { .getPrivAppPermissions(pkg.packageName); boolean whitelisted = wlPermissions != null && wlPermissions.contains(perm); if (!whitelisted) { - Slog.e(TAG, "Not granting privileged permission " + perm + " for package " + // Log for now. TODO Enforce permissions + Slog.w(TAG, "Privileged permission " + perm + " for package " + pkg.packageName + " - not in privapp-permissions whitelist"); - return false; } } boolean allowed = (compareSignatures(