From 261b0560100e15e376cb6bcf64eaabb5a79b24c5 Mon Sep 17 00:00:00 2001
From: Andrew Su <asu@redhat.com>
Date: Tue, 8 Mar 2011 10:11:28 -0500
Subject: Close streams after opening them.

---
 ChangeLog                                          |  6 ++++++
 netx/net/sourceforge/jnlp/util/PropertiesFile.java | 24 ++++++++++++++--------
 2 files changed, 22 insertions(+), 8 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index 23861ec..1b2e0ce 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2011-03-08  Andrew Su  <asu@redhat.com>
+
+	* netx/net/sourceforge/jnlp/util/PropertiesFile.java:
+	(load): Closed streams after opening them.
+	(store): Likewise.
+
 2011-03-08  Denis Lila  <dlila@redhat.com>
 
 	* plugin/icedteanp/java/sun/applet/PluginAppletViewer.java
diff --git a/netx/net/sourceforge/jnlp/util/PropertiesFile.java b/netx/net/sourceforge/jnlp/util/PropertiesFile.java
index 32006b6..a3188e0 100644
--- a/netx/net/sourceforge/jnlp/util/PropertiesFile.java
+++ b/netx/net/sourceforge/jnlp/util/PropertiesFile.java
@@ -25,8 +25,6 @@ import java.util.*;
  * file when the first property is requested, but the save method
  * must be called before changes are saved to the file.<p>
  *
- * This class does not report IO exceptions.<p>
- *
  * @author <a href="mailto:jmaxwell@users.sourceforge.net">Jon A. Maxwell (JAM)</a> - initial author
  * @version $Revision: 1.4 $
  */
@@ -110,14 +108,19 @@ public class PropertiesFile extends Properties {
     public void load() {
         loaded = true;
 
+        InputStream s = null;
         try {
             if (!file.exists())
                 return;
 
-            InputStream s = new FileInputStream(file);
-            load(s);
+            try {
+                s = new FileInputStream(file);
+                load(s);
+            } finally {
+                if (s != null) s.close();
+            }
         } catch (IOException ex) {
-            // eat
+            ex.printStackTrace();
         }
     }
 
@@ -128,11 +131,16 @@ public class PropertiesFile extends Properties {
         if (!loaded)
             return; // nothing could have changed so save unnecessary load/save
 
+        OutputStream s = null;
         try {
-            OutputStream s = new FileOutputStream(file);
-            store(s, header);
+            try {
+                s = new FileOutputStream(file);
+                store(s, header);
+            } finally {
+                if (s != null) s.close();
+            }
         } catch (IOException ex) {
-            // eat
+            ex.printStackTrace();
         }
     }
 
-- 
cgit v1.2.3