summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Pierre Imai <imaipi@google.com> 2016-03-10 17:00:50 +0900
committer Pierre Imai <imaipi@google.com> 2016-03-11 10:45:03 +0900
commitbe12d76feb1ba55797b408e5fa13c9f9ca1e06ad (patch)
treefa9196d63aa21d7ce7a10a1baa93ab24fdca7b44
parent349dc61cbaf6876184f659695896158f4991f04d (diff)
Remove an unnecessary InputStream and prevent leaking memory.
BUG: 27441906 Change-Id: Ib0ff36c0adc09e1f303a38e8355c84382cbc63fb
-rw-r--r--services/core/java/com/android/server/connectivity/NetworkMonitor.java3
1 files changed, 1 insertions, 2 deletions
diff --git a/services/core/java/com/android/server/connectivity/NetworkMonitor.java b/services/core/java/com/android/server/connectivity/NetworkMonitor.java
index 73da427c47c4..bce77337d581 100644
--- a/services/core/java/com/android/server/connectivity/NetworkMonitor.java
+++ b/services/core/java/com/android/server/connectivity/NetworkMonitor.java
@@ -688,12 +688,11 @@ public class NetworkMonitor extends StateMachine {
// Time how long it takes to get a response to our request
long requestTimestamp = SystemClock.elapsedRealtime();
- urlConnection.getInputStream();
+ httpResponseCode = urlConnection.getResponseCode();
// Time how long it takes to get a response to our request
long responseTimestamp = SystemClock.elapsedRealtime();
- httpResponseCode = urlConnection.getResponseCode();
validationLog("isCaptivePortal: ret=" + httpResponseCode +
" headers=" + urlConnection.getHeaderFields());
// NOTE: We may want to consider an "HTTP/1.0 204" response to be a captive