From aa292fdb699e51ac8021e551707404c96e2fc11f Mon Sep 17 00:00:00 2001 From: Marco Nelissen Date: Fri, 2 Jun 2017 12:16:44 -0700 Subject: [PATCH] Close connection before retrying Otherwise the (CTS) server might run out of connections. Bug: 38391487 Bug: 22771132 Test: build, run CTS, stream music Change-Id: I92c782a6799ab36eec8df3f7c3217bea667b838a (cherry picked from commit 46132afee7783665ae12538edd024770dbd93ecb) --- media/java/android/media/MediaHTTPConnection.java | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/media/java/android/media/MediaHTTPConnection.java b/media/java/android/media/MediaHTTPConnection.java index d6bf421ffa9f..dda5ce6fd083 100644 --- a/media/java/android/media/MediaHTTPConnection.java +++ b/media/java/android/media/MediaHTTPConnection.java @@ -135,7 +135,13 @@ public class MediaHTTPConnection extends IMediaHTTPConnection.Stub { private void teardownConnection() { if (mConnection != null) { - mInputStream = null; + if (mInputStream != null) { + try { + mInputStream.close(); + } catch (IOException e) { + } + mInputStream = null; + } mConnection.disconnect(); mConnection = null; @@ -297,8 +303,7 @@ public class MediaHTTPConnection extends IMediaHTTPConnection.Stub { mCurrentOffset = offset; } catch (IOException e) { mTotalSize = -1; - mInputStream = null; - mConnection = null; + teardownConnection(); mCurrentOffset = -1; throw e; -- 2.11.0