You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by be...@apache.org on 2008/03/14 20:27:39 UTC
svn commit: r637219 -
/maven/shared/trunk/maven-verifier/src/main/java/org/apache/maven/it/util/ResourceExtractor.java
Author: bentmann
Date: Fri Mar 14 12:27:38 2008
New Revision: 637219
URL: http://svn.apache.org/viewvc?rev=637219&view=rev
Log:
[MNG-3465] verifier doesn't close outputstreams
o Ensured streams are closed on error, too
Modified:
maven/shared/trunk/maven-verifier/src/main/java/org/apache/maven/it/util/ResourceExtractor.java
Modified: maven/shared/trunk/maven-verifier/src/main/java/org/apache/maven/it/util/ResourceExtractor.java
URL: http://svn.apache.org/viewvc/maven/shared/trunk/maven-verifier/src/main/java/org/apache/maven/it/util/ResourceExtractor.java?rev=637219&r1=637218&r2=637219&view=diff
==============================================================================
--- maven/shared/trunk/maven-verifier/src/main/java/org/apache/maven/it/util/ResourceExtractor.java (original)
+++ maven/shared/trunk/maven-verifier/src/main/java/org/apache/maven/it/util/ResourceExtractor.java Fri Mar 14 12:27:38 2008
@@ -77,15 +77,21 @@
destFile.mkdirs();
} else {
FileOutputStream fos = new FileOutputStream(destFile);
- IOUtil.copy(z.getInputStream(ze), fos);
- fos.close();
+ try {
+ IOUtil.copy(z.getInputStream(ze), fos);
+ } finally {
+ IOUtil.close(fos);
+ }
}
}
}
} else {
FileOutputStream fos = new FileOutputStream(dest);
- IOUtil.copy(cl.getResourceAsStream(resourcePath), fos);
- fos.close();
+ try {
+ IOUtil.copy(cl.getResourceAsStream(resourcePath), fos);
+ } finally {
+ IOUtil.close(fos);
+ }
}
}