diff options
| -rwxr-xr-x | services/core/java/com/android/server/tv/TvInputManagerService.java | 11 |
1 files changed, 10 insertions, 1 deletions
diff --git a/services/core/java/com/android/server/tv/TvInputManagerService.java b/services/core/java/com/android/server/tv/TvInputManagerService.java index 323ac7b8806e..bedcf9c45139 100755 --- a/services/core/java/com/android/server/tv/TvInputManagerService.java +++ b/services/core/java/com/android/server/tv/TvInputManagerService.java @@ -714,6 +714,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; } @@ -2278,8 +2279,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; |