Prevent HandlerThread leak of AWSIotMqttManager

hirofumi · minbi · commit 3d18823bc793 · 2018-12-12T13:51:53.000-08:00
Otherwise leak occurs on reconnection as follows. https://github.com/hirofumi/aws-sdk-android-ht-leak
diff --git a/aws-android-sdk-iot/src/main/java/com/amazonaws/mobileconnectors/iot/AWSIotMqttManager.java b/aws-android-sdk-iot/src/main/java/com/amazonaws/mobileconnectors/iot/AWSIotMqttManager.java
@@ -1053,6 +1053,7 @@ public void run() {
                     if (mqttClient != null && !mqttClient.isConnected()) {
                         reconnectToSession();
                     }
+                    ht.quit();
                 }
             }, MILLIS_IN_ONE_SECOND * currentReconnectRetryTime);
             currentReconnectRetryTime = Math.min(currentReconnectRetryTime * 2, maxReconnectRetryTime);

Original file line number	Diff line number	Diff line change
`@@ -1053,6 +1053,7 @@ public void run() {`
`1053`	`1053`	`if (mqttClient != null && !mqttClient.isConnected()) {`
`1054`	`1054`	`reconnectToSession();`
`1055`	`1055`	`}`
	`1056`	`+ ht.quit();`
`1056`	`1057`	`}`
`1057`	`1058`	`}, MILLIS_IN_ONE_SECOND * currentReconnectRetryTime);`
`1058`	`1059`	`currentReconnectRetryTime = Math.min(currentReconnectRetryTime * 2, maxReconnectRetryTime);`