diff options
| author | 2024-10-22 12:06:16 -0700 | |
|---|---|---|
| committer | 2024-10-24 10:48:34 -0700 | |
| commit | 543e846fc24511e0379ea6decf059db876a2b81e (patch) | |
| tree | 81d3971c1155a9b8679436698b6f8fd131117583 | |
| parent | 0935d9993a1b5027b1bcc69248382495c3bdaa48 (diff) | |
Log ACCOUNT_MANAGER_EVENT__EVENT_TYPE__AUTHENTICATOR_ADDED
Example value from debug log: com.google 10140 removed
Bug: 346563138
Test: manual
Flag: EXEMPT bugfix
Change-Id: I6fe9bb94efb6c7f603921377d276fecd0197e5a5
| -rw-r--r-- | services/core/java/com/android/server/accounts/AccountManagerService.java | 23 |
1 files changed, 22 insertions, 1 deletions
diff --git a/services/core/java/com/android/server/accounts/AccountManagerService.java b/services/core/java/com/android/server/accounts/AccountManagerService.java index 0ca3b56486e3..679c7ac3ceac 100644 --- a/services/core/java/com/android/server/accounts/AccountManagerService.java +++ b/services/core/java/com/android/server/accounts/AccountManagerService.java @@ -112,6 +112,7 @@ import com.android.internal.messages.nano.SystemMessageProto.SystemMessage; import com.android.internal.notification.SystemNotificationChannels; import com.android.internal.util.ArrayUtils; import com.android.internal.util.DumpUtils; +import com.android.internal.util.FrameworkStatsLog; import com.android.internal.util.IndentingPrintWriter; import com.android.internal.util.Preconditions; import com.android.modules.expresslog.Histogram; @@ -1226,7 +1227,17 @@ public class AccountManagerService // been re-enabled (after being updated for example), then we just overwrite the old // values. for (Entry<String, Integer> entry : knownAuth.entrySet()) { - accountsDb.insertOrReplaceMetaAuthTypeAndUid(entry.getKey(), entry.getValue()); + String type = entry.getKey(); + Integer newUid = entry.getValue(); + if (!Objects.equals(metaAuthUid.get(type), newUid)) { + FrameworkStatsLog.write( + FrameworkStatsLog.ACCOUNT_MANAGER_EVENT, + type, + newUid, + FrameworkStatsLog + .ACCOUNT_MANAGER_EVENT__EVENT_TYPE__AUTHENTICATOR_ADDED); + } + accountsDb.insertOrReplaceMetaAuthTypeAndUid(type, newUid); } final Map<Long, Account> accountsMap = accountsDb.findAllDeAccounts(); @@ -1945,6 +1956,11 @@ public class AccountManagerService } accounts.accountsDb.setTransactionSuccessful(); + FrameworkStatsLog.write( + FrameworkStatsLog.ACCOUNT_MANAGER_EVENT, + account.type, + callingUid, + FrameworkStatsLog.ACCOUNT_MANAGER_EVENT__EVENT_TYPE__ACCOUNT_ADDED); logRecord(AccountsDb.DEBUG_ACTION_ACCOUNT_ADD, AccountsDb.TABLE_ACCOUNTS, accountId, accounts, callingUid); @@ -2544,6 +2560,11 @@ public class AccountManagerService } String action = userUnlocked ? AccountsDb.DEBUG_ACTION_ACCOUNT_REMOVE : AccountsDb.DEBUG_ACTION_ACCOUNT_REMOVE_DE; + FrameworkStatsLog.write( + FrameworkStatsLog.ACCOUNT_MANAGER_EVENT, + account.type, + callingUid, + FrameworkStatsLog.ACCOUNT_MANAGER_EVENT__EVENT_TYPE__ACCOUNT_REMOVED); logRecord(action, AccountsDb.TABLE_ACCOUNTS, accountId, accounts); } } |