You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@aries.apache.org by mn...@apache.org on 2010/01/20 16:16:39 UTC

svn commit: r901229 - /incubator/aries/trunk/application/application-management/src/main/java/org/apache/aries/application/management/impl/AriesApplicationImpl.java

Author: mnuttall
Date: Wed Jan 20 15:16:38 2010
New Revision: 901229

URL: http://svn.apache.org/viewvc?rev=901229&view=rev
Log:
Failed to close an output stream in a finally{} block

Modified:
    incubator/aries/trunk/application/application-management/src/main/java/org/apache/aries/application/management/impl/AriesApplicationImpl.java

Modified: incubator/aries/trunk/application/application-management/src/main/java/org/apache/aries/application/management/impl/AriesApplicationImpl.java
URL: http://svn.apache.org/viewvc/incubator/aries/trunk/application/application-management/src/main/java/org/apache/aries/application/management/impl/AriesApplicationImpl.java?rev=901229&r1=901228&r2=901229&view=diff
==============================================================================
--- incubator/aries/trunk/application/application-management/src/main/java/org/apache/aries/application/management/impl/AriesApplicationImpl.java (original)
+++ incubator/aries/trunk/application/application-management/src/main/java/org/apache/aries/application/management/impl/AriesApplicationImpl.java Wed Jan 20 15:16:38 2010
@@ -152,10 +152,13 @@
 
     // Write the migrated bundles out
     if (_modifiedBundles != null) { 
-      for (Map.Entry<String, InputStream> modifiedBundle : _modifiedBundles.entrySet()) { 
-        out = IOUtils.getOutputStream(tempDir, modifiedBundle.getKey());
-        IOUtils.copy(modifiedBundle.getValue(), out);
-        IOUtils.close(out);
+      for (Map.Entry<String, InputStream> modifiedBundle : _modifiedBundles.entrySet()) {
+        try { 
+          out = IOUtils.getOutputStream(tempDir, modifiedBundle.getKey());
+          IOUtils.copy(modifiedBundle.getValue(), out);
+        } finally { 
+          IOUtils.close(out);
+        }
       }
     }