You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@archiva.apache.org by ol...@apache.org on 2013/12/19 01:51:34 UTC
svn commit: r1552191 -
/archiva/trunk/archiva-modules/archiva-web/archiva-web-common/src/main/java/org/apache/archiva/web/api/DefaultFileUploadService.java
Author: olamy
Date: Thu Dec 19 00:51:34 2013
New Revision: 1552191
URL: http://svn.apache.org/r1552191
Log:
correctly close streams
Modified:
archiva/trunk/archiva-modules/archiva-web/archiva-web-common/src/main/java/org/apache/archiva/web/api/DefaultFileUploadService.java
Modified: archiva/trunk/archiva-modules/archiva-web/archiva-web-common/src/main/java/org/apache/archiva/web/api/DefaultFileUploadService.java
URL: http://svn.apache.org/viewvc/archiva/trunk/archiva-modules/archiva-web/archiva-web-common/src/main/java/org/apache/archiva/web/api/DefaultFileUploadService.java?rev=1552191&r1=1552190&r2=1552191&view=diff
==============================================================================
--- archiva/trunk/archiva-modules/archiva-web/archiva-web-common/src/main/java/org/apache/archiva/web/api/DefaultFileUploadService.java (original)
+++ archiva/trunk/archiva-modules/archiva-web/archiva-web-common/src/main/java/org/apache/archiva/web/api/DefaultFileUploadService.java Thu Dec 19 00:51:34 2013
@@ -575,17 +575,19 @@ public class DefaultFileUploadService
private void copyFile( File sourceFile, File targetPath, String targetFilename, boolean fixChecksums )
throws IOException
{
- FileOutputStream out = new FileOutputStream( new File( targetPath, targetFilename ) );
- FileInputStream input = new FileInputStream( sourceFile );
+ FileOutputStream out = null;
+ FileInputStream input = null;
try
{
+ out = new FileOutputStream( new File( targetPath, targetFilename ) );
+ input = new FileInputStream( sourceFile );
IOUtils.copy( input, out );
}
finally
{
- out.close();
- input.close();
+ IOUtils.closeQuietly( out );
+ IOUtils.closeQuietly( input );
}
if ( fixChecksums )