From 673cc5650ea34df3057f9cc7ae6f887c90dd19f1 Mon Sep 17 00:00:00 2001
From: Stanislav Zholnin <zholnin@google.com>
Date: Sat, 22 May 2021 15:50:04 +0100
Subject: [PATCH] Remove filtering of DiscreteRegistry events by uid.

Bug: 184597401
Test: manually verified
Change-Id: I6c02f0ec64aa31211ffffa13d8ad0b5facbb9bbd
---
 .../android/server/appop/DiscreteRegistry.java    | 15 ++-------------
 1 file changed, 2 insertions(+), 13 deletions(-)

diff --git a/services/core/java/com/android/server/appop/DiscreteRegistry.java b/services/core/java/com/android/server/appop/DiscreteRegistry.java
index 99daa240fbd5b..10cfddfb3ae46 100644
--- a/services/core/java/com/android/server/appop/DiscreteRegistry.java
+++ b/services/core/java/com/android/server/appop/DiscreteRegistry.java
@@ -43,7 +43,6 @@ import android.os.AsyncTask;
 import android.os.Build;
 import android.os.Environment;
 import android.os.FileUtils;
-import android.os.Process;
 import android.provider.DeviceConfig;
 import android.util.ArrayMap;
 import android.util.AtomicFile;
@@ -229,7 +228,7 @@ final class DiscreteRegistry {
     void recordDiscreteAccess(int uid, String packageName, int op, @Nullable String attributionTag,
             @AppOpsManager.OpFlags int flags, @AppOpsManager.UidState int uidState, long accessTime,
             long accessDuration) {
-        if (!isDiscreteOp(op, uid, flags)) {
+        if (!isDiscreteOp(op, flags)) {
             return;
         }
         synchronized (mInMemoryLock) {
@@ -1046,26 +1045,16 @@ final class DiscreteRegistry {
         return result;
     }
 
-    private static boolean isDiscreteOp(int op, int uid, @AppOpsManager.OpFlags int flags) {
+    private static boolean isDiscreteOp(int op, @AppOpsManager.OpFlags int flags) {
         if (!ArrayUtils.contains(sDiscreteOps, op)) {
             return false;
         }
-        if (!isDiscreteUid(uid)) {
-            return false;
-        }
         if ((flags & (sDiscreteFlags)) == 0) {
             return false;
         }
         return true;
     }
 
-    private static boolean isDiscreteUid(int uid) {
-        if (uid < Process.FIRST_APPLICATION_UID) {
-            return false;
-        }
-        return true;
-    }
-
     void setDebugMode(boolean debugMode) {
         this.mDebugMode = debugMode;
     }