correctly close jarFile in a finally block

git-svn-id: https://svn.apache.org/repos/asf/archiva/trunk@1358773 13f79535-47bb-0310-9956-ffa450edef68

diff --git a/archiva-modules/archiva-web/archiva-rest/archiva-rest-services/src/main/java/org/apache/archiva/rest/services/DefaultBrowseService.java b/archiva-modules/archiva-web/archiva-rest/archiva-rest-services/src/main/java/org/apache/archiva/rest/services/DefaultBrowseService.java
index 5228ae6216f971162747eb02122142759ab491cc..7896ec50c75628139a5d2a64753951068735452e 100644 (file)
--- a/archiva-modules/archiva-web/archiva-rest/archiva-rest-services/src/main/java/org/apache/archiva/rest/services/DefaultBrowseService.java
+++ b/archiva-modules/archiva-web/archiva-rest/archiva-rest-services/src/main/java/org/apache/archiva/rest/services/DefaultBrowseService.java
@@ -752,6 +752,7 @@ public class DefaultBrowseService
                     }
                     finally
                     {
+                        closeQuietly( jarFile );
                         IOUtils.closeQuietly( inputStream );
                     }
                 }
@@ -782,6 +783,21 @@ public class DefaultBrowseService
         return new ArtifactContent();
     }
 
+    private void closeQuietly( JarFile jarFile )
+    {
+        if ( jarFile != null )
+        {
+            try
+            {
+                jarFile.close();
+            }
+            catch ( IOException e )
+            {
+                log.warn( "ignore error closing jarFile {}", jarFile.getName() );
+            }
+        }
+    }
+
     //---------------------------
     // internals
     //---------------------------