From e52e9ebd6507893daf6897cc8d2913f2861e1026 Mon Sep 17 00:00:00 2001 From: Aseem Kumar Date: Thu, 10 Nov 2022 20:42:28 +0000 Subject: [PATCH] Fix NPE in AccountManagerService. Test: atest AccountManagerServiceTest Bug: 243969466 Change-Id: I59ad96dfd6894dda21d2a8c86b18ee88dc7f7a00 --- .../com/android/server/accounts/AccountManagerService.java | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/services/core/java/com/android/server/accounts/AccountManagerService.java b/services/core/java/com/android/server/accounts/AccountManagerService.java index 416de0af84a09..c7c2655a991d8 100644 --- a/services/core/java/com/android/server/accounts/AccountManagerService.java +++ b/services/core/java/com/android/server/accounts/AccountManagerService.java @@ -1870,7 +1870,7 @@ public class AccountManagerService } if (accounts.accountsDb.findAllDeAccounts().size() > 100) { Log.w(TAG, "insertAccountIntoDatabase: " + account.toSafeString() - + ", skipping since more than 50 accounts on device exist"); + + ", skipping since more than 100 accounts on device exist"); return false; } long accountId = accounts.accountsDb.insertCeAccount(account, password); @@ -3520,10 +3520,10 @@ public class AccountManagerService @Override protected String toDebugString(long now) { - String requiredFeaturesStr = TextUtils.join(",", requiredFeatures); return super.toDebugString(now) + ", startAddAccountSession" + ", accountType " + accountType + ", requiredFeatures " - + (requiredFeatures != null ? requiredFeaturesStr : null); + + (requiredFeatures != null + ? TextUtils.join(",", requiredFeatures) : "null"); } }.bind(); } finally {