Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit be12d76f authored by Pierre Imai's avatar Pierre Imai
Browse files

Remove an unnecessary InputStream and prevent leaking memory. 

BUG: 27441906
Change-Id: Ib0ff36c0adc09e1f303a38e8355c84382cbc63fb
parent 349dc61c

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

+1 −2
Original line number Diff line number Diff line
@@ -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