Author: felixk
Date: Wed Sep  7 10:05:20 2011
New Revision: 1166093

URL: http://svn.apache.org/viewvc?rev=1166093&view=rev
Log:
Close inputstream (JAMES-1207)

Modified:
    
james/server/trunk/lifecycle-spring/src/main/java/org/apache/james/container/spring/lifecycle/osgi/OSGIConfigurationProvider.java

Modified: 
james/server/trunk/lifecycle-spring/src/main/java/org/apache/james/container/spring/lifecycle/osgi/OSGIConfigurationProvider.java
URL: 
http://svn.apache.org/viewvc/james/server/trunk/lifecycle-spring/src/main/java/org/apache/james/container/spring/lifecycle/osgi/OSGIConfigurationProvider.java?rev=1166093&r1=1166092&r2=1166093&view=diff
==============================================================================
--- 
james/server/trunk/lifecycle-spring/src/main/java/org/apache/james/container/spring/lifecycle/osgi/OSGIConfigurationProvider.java
 (original)
+++ 
james/server/trunk/lifecycle-spring/src/main/java/org/apache/james/container/spring/lifecycle/osgi/OSGIConfigurationProvider.java
 Wed Sep  7 10:05:20 2011
@@ -36,6 +36,7 @@ public class OSGIConfigurationProvider i
     @Override
     public HierarchicalConfiguration getConfiguration(String beanName) throws 
ConfigurationException {
         XMLConfiguration config = new XMLConfiguration();
+        FileInputStream fis = null;
         config.setDelimiterParsingDisabled(true);
         
         // Don't split attributes which can have bad side-effects with 
matcher-conditions.
@@ -45,9 +46,18 @@ public class OSGIConfigurationProvider i
         // Use InputStream so we are not bound to File implementations of the
         // config
         try {
-            config.load(new FileInputStream("/tmp/" + beanName + ".xml"));
+            fis = new FileInputStream("/tmp/" + beanName + ".xml");
+            config.load(fis);
         } catch (FileNotFoundException e) {
             throw new ConfigurationException("Bean " + beanName);
+        } finally {
+            if (fis != null) {
+                try {
+                    fis.close();
+                } catch (Exception e) {
+                    // Left empty on purpose
+                }
+            }
         }
         
         return config;



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to