Ensure that immediate disconnection listeners are handled on the UI thread.

This will resolve a crash when disconnecting hosts with "close on
disconnect" set to true. See
https://github.com/connectbot/connectbot/issues/135.

1 files changed, 21 insertions, 17 deletions

diff --git a/app/src/main/java/org/connectbot/service/TerminalBridge.java b/app/src/main/java/org/connectbot/service/TerminalBridge.java
index f7be0ae..6d47f96 100644
--- a/app/src/main/java/org/connectbot/service/TerminalBridge.java
+++ b/app/src/main/java/org/connectbot/service/TerminalBridge.java
@@ -437,8 +437,7 @@ public class TerminalBridge implements VDUDisplay {
 
 		if (immediate || (host.getQuickDisconnect() && !host.getStayConnected())) {
 			awaitingClose = true;
-			if (disconnectListener != null)
-				disconnectListener.onDisconnected(TerminalBridge.this);
+			triggerDisconnectListener();
 		} else {
 			{
 				final String line = manager.res.getString(R.string.alert_disconnect_msg);
@@ -454,21 +453,7 @@ public class TerminalBridge implements VDUDisplay {
 							manager.res.getString(R.string.prompt_host_disconnected));
 					if (result == null || result.booleanValue()) {
 						awaitingClose = true;
-
-						// Tell the TerminalManager that we can be destroyed now.
-						if (disconnectListener != null) {
-							// The disconnect listener should be run on the main thread if possible.
-							if (parent != null) {
-								parent.post(new Runnable() {
-									@Override
-									public void run() {
-										disconnectListener.onDisconnected(TerminalBridge.this);
-									}
-								});
-							} else {
-								disconnectListener.onDisconnected(TerminalBridge.this);
-							}
-						}
+						triggerDisconnectListener();
 					}
 				}
 			});
@@ -478,6 +463,25 @@ public class TerminalBridge implements VDUDisplay {
 		}
 	}
 
+	/**
+	 * Tells the TerminalManager that we can be destroyed now.
+	 */
+	private void triggerDisconnectListener() {
+		if (disconnectListener != null) {
+			// The disconnect listener should be run on the main thread if possible.
+			if (parent != null) {
+				parent.post(new Runnable() {
+					@Override
+					public void run() {
+						disconnectListener.onDisconnected(TerminalBridge.this);
+					}
+				});
+			} else {
+				disconnectListener.onDisconnected(TerminalBridge.this);
+			}
+		}
+	}
+
 	public void setSelectingForCopy(boolean selectingForCopy) {
 		this.selectingForCopy = selectingForCopy;
 	}