summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Dmitry Dementyev <dementyev@google.com> 2023-07-21 22:13:41 +0000
committer Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> 2023-07-21 22:13:41 +0000
commit923b35a933590690b7be47b1687dce303d11c4db (patch)
tree2618d30e87fd6a600942d3efb9b73539e03dde26
parent1150c8894357c11f945bd4630ba11e9172809b1b (diff)
parentb6c6ce7ee6bd1a0fed535f72fd6d0e998f5b0bf9 (diff)
Merge "Fix the concurrency issue when 'linkToDeath' in AccountManagerService that leads to JE" into main am: e7627bd732 am: b6c6ce7ee6
Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/2664259 Change-Id: I5cdd85222b365b9d88cd7334ed1966fe28f2b5c3 Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
-rw-r--r--services/core/java/com/android/server/accounts/AccountManagerService.java3
1 files changed, 1 insertions, 2 deletions
diff --git a/services/core/java/com/android/server/accounts/AccountManagerService.java b/services/core/java/com/android/server/accounts/AccountManagerService.java
index 193515585aa8..5fec8494fb92 100644
--- a/services/core/java/com/android/server/accounts/AccountManagerService.java
+++ b/services/core/java/com/android/server/accounts/AccountManagerService.java
@@ -4905,7 +4905,6 @@ public class AccountManagerService
if (accountType == null) throw new IllegalArgumentException("accountType is null");
mAccounts = accounts;
mStripAuthTokenFromResult = stripAuthTokenFromResult;
- mResponse = response;
mAccountType = accountType;
mExpectActivityLaunch = expectActivityLaunch;
mCreationTime = SystemClock.elapsedRealtime();
@@ -4919,8 +4918,8 @@ public class AccountManagerService
if (response != null) {
try {
response.asBinder().linkToDeath(this, 0 /* flags */);
+ mResponse = response;
} catch (RemoteException e) {
- mResponse = null;
binderDied();
}
}