close http download connection after download

1 files changed, 6 insertions, 1 deletions

diff --git a/src/main/java/de/pixart/messenger/http/HttpDownloadConnection.java b/src/main/java/de/pixart/messenger/http/HttpDownloadConnection.java
index 722c1dea9..dbbfec861 100644
--- a/src/main/java/de/pixart/messenger/http/HttpDownloadConnection.java
+++ b/src/main/java/de/pixart/messenger/http/HttpDownloadConnection.java
@@ -227,6 +227,7 @@ public class HttpDownloadConnection implements Transferable {
                 }
                 return Long.parseLong(contentLength, 10);
             } catch (IOException e) {
+                Log.d(Config.LOGTAG, "io exception during HEAD " + e.getMessage());
                 throw e;
             } catch (NumberFormatException e) {
                 throw new IOException();
@@ -269,10 +270,10 @@ public class HttpDownloadConnection implements Transferable {
 
         private void download() throws Exception {
             InputStream is = null;
+            HttpURLConnection connection = null;
             PowerManager.WakeLock wakeLock = mHttpConnectionManager.createWakeLock("http_download_" + message.getUuid());
             try {
                 wakeLock.acquire();
-                HttpURLConnection connection;
                 if (mUseTor) {
                     connection = (HttpURLConnection) mUrl.openConnection(mHttpConnectionManager.getProxy());
                 } else {
@@ -332,10 +333,14 @@ public class HttpDownloadConnection implements Transferable {
                     throw new FileWriterException();
                 }
             } catch (CancellationException | IOException e) {
+                Log.d(Config.LOGTAG, "http download failed " + e.getMessage());
                 throw e;
             } finally {
                 FileBackend.close(os);
                 FileBackend.close(is);
+                if (connection != null) {
+                    connection.disconnect();
+                }
                 wakeLock.release();
             }
         }