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);
+ }
}
}