From a1397e8e838efa93549047d5600997314cc93b5c Mon Sep 17 00:00:00 2001 From: satayev Date: Thu, 10 Jun 2021 14:16:52 +0100 Subject: [PATCH] Check for permission on every @Override method. Bug: 189862446 Test: presubmit Change-Id: I52c65d831948679eabf4fda10241d25467c608f1 --- .../core/java/com/android/server/compat/PlatformCompat.java | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/services/core/java/com/android/server/compat/PlatformCompat.java b/services/core/java/com/android/server/compat/PlatformCompat.java index 2cd68b0864537..19e858c25c41a 100644 --- a/services/core/java/com/android/server/compat/PlatformCompat.java +++ b/services/core/java/com/android/server/compat/PlatformCompat.java @@ -91,15 +91,17 @@ public class PlatformCompat extends IPlatformCompat.Stub { @Override public void reportChange(long changeId, ApplicationInfo appInfo) { - reportChangeByUid(changeId, appInfo.uid); + checkCompatChangeLogPermission(); + reportChangeInternal(changeId, appInfo.uid, ChangeReporter.STATE_LOGGED); } @Override public void reportChangeByPackageName(long changeId, String packageName, @UserIdInt int userId) { + checkCompatChangeLogPermission(); ApplicationInfo appInfo = getApplicationInfo(packageName, userId); if (appInfo != null) { - reportChangeByUid(changeId, appInfo.uid); + reportChangeInternal(changeId, appInfo.uid, ChangeReporter.STATE_LOGGED); } }