Stabilize open InputStream's / Closeable's: Decorate w/ try-finally and close within the latter

2 files changed, 26 insertions, 2 deletions

diff --git a/src/java/com/jogamp/common/util/IOUtil.java b/src/java/com/jogamp/common/util/IOUtil.java
index 9e803cb..0463c37 100644
--- a/src/java/com/jogamp/common/util/IOUtil.java
+++ b/src/java/com/jogamp/common/util/IOUtil.java
@@ -30,6 +30,7 @@ package com.jogamp.common.util;
 
 import java.io.BufferedInputStream;
 import java.io.BufferedOutputStream;
+import java.io.Closeable;
 import java.io.File;
 import java.io.FileOutputStream;
 import java.io.IOException;
@@ -585,4 +586,19 @@ public class IOUtil {
        }
        throw new IOException("Could not create temp directory @ "+tempRoot.getAbsolutePath()+tmpDirPrefix+"_*");        
     }
+    
+    public static void close(Closeable stream, boolean throwRuntimeException) throws RuntimeException {
+        if(null != stream) {
+            try {
+                stream.close();
+            } catch (IOException e) {
+                if(throwRuntimeException) {
+                    throw new RuntimeException(e);
+                } else if(DEBUG) {
+                    System.err.println("Catched Exception: ");
+                    e.printStackTrace();
+                }
+            }
+        }        
+    }
 }
diff --git a/src/java/com/jogamp/common/util/VersionUtil.java b/src/java/com/jogamp/common/util/VersionUtil.java
index 157edaa..30b23f2 100644
--- a/src/java/com/jogamp/common/util/VersionUtil.java
+++ b/src/java/com/jogamp/common/util/VersionUtil.java
@@ -32,6 +32,7 @@ import com.jogamp.common.os.AndroidVersion;
 import com.jogamp.common.os.Platform;
 
 import java.io.IOException;
+import java.io.InputStream;
 import java.net.URL;
 import java.util.Enumeration;
 import java.util.Iterator;
@@ -90,10 +91,17 @@ public class VersionUtil {
      * @return the requested manifest or null when not found.
      */
     public static Manifest getManifest(ClassLoader cl, String extension) {
+         
         try {
-            Enumeration resources = cl.getResources("META-INF/MANIFEST.MF");
+            Enumeration<URL> resources = cl.getResources("META-INF/MANIFEST.MF");
             while (resources.hasMoreElements()) {
-                Manifest manifest = new Manifest(((URL)resources.nextElement()).openStream());
+                final InputStream is = resources.nextElement().openStream();
+                final Manifest manifest;
+                try {
+                    manifest = new Manifest(is);
+                } finally {
+                    IOUtil.close(is, false);
+                }
                 Attributes attributes = manifest.getMainAttributes();
                 if(attributes != null && extension.equals(attributes.getValue(Attributes.Name.EXTENSION_NAME))) {
                     return manifest;