commit e9050bbcb7ee43cc6870c054b884a0bace2f156e
author Hai Zhang <zhanghai@google.com> Wed Jul 03 19:29:39 2024 +0000
committer Hai Zhang <zhanghai@google.com> Wed Jul 03 19:29:56 2024 +0000
tree dde75002da1b1dbbe2b4ba2cbb5df6c448d199bc
parent 0cc6ccb5fc4a12086747c7d7d2758531fe003ad9

Kill apps when GIDs changed instead of only when added

Bug: 350847329
Flag: EXEMPT bugfix
Test: presubmit
Change-Id: I4da60ad0803cb8e8691fa2474d47cff9fb3b198d

services/permission/java/com/android/server/permission/access/permission/PermissionService.kt

diff --git a/services/permission/java/com/android/server/permission/access/permission/PermissionService.kt b/services/permission/java/com/android/server/permission/access/permission/PermissionService.kt
index 78dbc60..0b7438c 100644
--- a/services/permission/java/com/android/server/permission/access/permission/PermissionService.kt
+++ b/services/permission/java/com/android/server/permission/access/permission/PermissionService.kt
@@ -2687,7 +2687,7 @@
                 runtimePermissionChangedUidDevices.getOrPut(uid) { mutableSetOf() } += deviceId
             }
 
-            if (permission.hasGids && !wasPermissionGranted && isPermissionGranted) {
+            if (permission.hasGids && (wasPermissionGranted != isPermissionGranted)) {
                 gidsChangedUids += uid
             }
         }