commit be12d76feb1ba55797b408e5fa13c9f9ca1e06ad
author Pierre Imai <imaipi@google.com> Thu Mar 10 17:00:50 2016 +0900
committer Pierre Imai <imaipi@google.com> Fri Mar 11 10:45:03 2016 +0900
tree fa9196d63aa21d7ce7a10a1baa93ab24fdca7b44
parent 349dc61cbaf6876184f659695896158f4991f04d

Remove an unnecessary InputStream and prevent leaking memory.

BUG: 27441906
Change-Id: Ib0ff36c0adc09e1f303a38e8355c84382cbc63fb

services/core/java/com/android/server/connectivity/NetworkMonitor.java

diff --git a/services/core/java/com/android/server/connectivity/NetworkMonitor.java b/services/core/java/com/android/server/connectivity/NetworkMonitor.java
index 73da427..bce7733 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 @@
             // 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