summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMichael Kuhn <[email protected]>2012-05-05 20:07:40 +0200
committerAndrew Dolgov <[email protected]>2012-05-05 22:23:51 +0400
commit299682bb6d4e2e9f2b0f0553f6f475f7f7585a99 (patch)
tree646424729eccdeb23d2c4f1c146106a160864dc5
parentc13dd67f4392ba9fd5cb544b930a5d0a7dce98b6 (diff)
Fix AndroidHttpClient leaks.
-rw-r--r--src/org/fox/ttrss/ApiRequest.java7
-rw-r--r--src/org/fox/ttrss/FeedsFragment.java5
2 files changed, 8 insertions, 4 deletions
diff --git a/src/org/fox/ttrss/ApiRequest.java b/src/org/fox/ttrss/ApiRequest.java
index 837aaa0c..6526ca28 100644
--- a/src/org/fox/ttrss/ApiRequest.java
+++ b/src/org/fox/ttrss/ApiRequest.java
@@ -132,10 +132,12 @@ public class ApiRequest extends AsyncTask<HashMap<String,String>, Integer, JsonE
} catch (IllegalArgumentException e) {
m_lastError = ApiError.INVALID_URL;
e.printStackTrace();
+ client.close();
return null;
} catch (Exception e) {
m_lastError = ApiError.OTHER_ERROR;
e.printStackTrace();
+ client.close();
return null;
}
@@ -153,6 +155,7 @@ public class ApiRequest extends AsyncTask<HashMap<String,String>, Integer, JsonE
} catch (MalformedURLException e) {
m_lastError = ApiError.INVALID_URL;
e.printStackTrace();
+ client.close();
return null;
}
@@ -218,8 +221,6 @@ public class ApiRequest extends AsyncTask<HashMap<String,String>, Integer, JsonE
}
}
- client.close();
-
return null;
case 401:
m_lastError = ApiError.HTTP_UNAUTHORIZED;
@@ -238,6 +239,7 @@ public class ApiRequest extends AsyncTask<HashMap<String,String>, Integer, JsonE
break;
}
+ client.close();
return null;
} catch (javax.net.ssl.SSLPeerUnverifiedException e) {
m_lastError = ApiError.SSL_REJECTED;
@@ -253,6 +255,7 @@ public class ApiRequest extends AsyncTask<HashMap<String,String>, Integer, JsonE
e.printStackTrace();
}
+ client.close();
return null;
}
}
diff --git a/src/org/fox/ttrss/FeedsFragment.java b/src/org/fox/ttrss/FeedsFragment.java
index b8ac66be..02ac2104 100644
--- a/src/org/fox/ttrss/FeedsFragment.java
+++ b/src/org/fox/ttrss/FeedsFragment.java
@@ -512,6 +512,7 @@ public class FeedsFragment extends Fragment implements OnItemClickListener, OnSh
targetUrl = new URL(fetchUrl);
} catch (MalformedURLException e) {
e.printStackTrace();
+ client.close();
return;
}
@@ -543,12 +544,12 @@ public class FeedsFragment extends Fragment implements OnItemClickListener, OnSh
fos.close();
is.close();
-
- client.close();
} catch (Exception e) {
e.printStackTrace();
}
+
+ client.close();
}
protected void onPostExecute(Integer result) {