Merge pull request #1 from werktools/fix-inputstream-memory-leak-android

Fix memory leak issue where private InputStream(s) were not closed. F…
werktools · Aug 28, 2023 · 940cce4 · 940cce4
2 parents ab7dfc0 + 1194f34
commit 940cce4
Showing 1 changed file with 16 additions and 0 deletions.
diff --git a/src/android/com/ionicframework/cordova/webview/WebViewLocalServer.java b/src/android/com/ionicframework/cordova/webview/WebViewLocalServer.java
@@ -598,6 +598,14 @@ public int available() throws IOException {
       return (is != null) ? is.available() : 0;
     }
 
+    @Override
+    public void close() throws IOException {
+      if (is != null) {
+        is.close();
+      }
+      super.close();
+    }
+
     @Override
     public int read() throws IOException {
       InputStream is = getInputStream();
@@ -633,6 +641,14 @@ public LollipopLazyInputStream(PathHandler handler, Uri uri) {
       this.uri = uri;
     }
 
+    @Override
+    public void close() throws IOException {
+      if (is != null) {
+        is.close();
+      }
+      super.close();
+    }
+
     @Override
     protected InputStream handle() {
       return handler.handle(uri);