Merge "SystemServer: fix an infinite loop in TvInputManagerService [1/1]" am: deb1024b75 am: 839fca8ad5 am: 182b4163ca
Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/1512821 Change-Id: I3f202dcca0c638f341aee705612bcd6d468fc39f
This commit is contained in:
@@ -768,6 +768,7 @@ public final class TvInputManagerService extends SystemService {
|
||||
SessionState sessionState = userState.sessionStateMap.remove(sessionToken);
|
||||
|
||||
if (sessionState == null) {
|
||||
Slog.e(TAG, "sessionState null, no more remove session action!");
|
||||
return;
|
||||
}
|
||||
|
||||
@@ -2525,8 +2526,16 @@ public final class TvInputManagerService extends SystemService {
|
||||
ClientState clientState = userState.clientStateMap.get(clientToken);
|
||||
if (clientState != null) {
|
||||
while (clientState.sessionTokens.size() > 0) {
|
||||
IBinder sessionToken = clientState.sessionTokens.get(0);
|
||||
releaseSessionLocked(
|
||||
clientState.sessionTokens.get(0), Process.SYSTEM_UID, userId);
|
||||
sessionToken, Process.SYSTEM_UID, userId);
|
||||
// the releaseSessionLocked function may return before the sessionToken
|
||||
// is removed if the related sessionState is null. So need to check again
|
||||
// to avoid death curculation.
|
||||
if (clientState.sessionTokens.contains(sessionToken)) {
|
||||
Slog.d(TAG, "remove sessionToken " + sessionToken + " for " + clientToken);
|
||||
clientState.sessionTokens.remove(sessionToken);
|
||||
}
|
||||
}
|
||||
}
|
||||
clientToken = null;
|
||||
|
||||
Reference in New Issue
Block a user