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 2009/01/29 22:43:09 UTC

svn commit: r739030 - /maven/shared/trunk/maven-verifier/src/main/java/org/apache/maven/it/Verifier.java

Author: bentmann
Date: Thu Jan 29 21:43:09 2009
New Revision: 739030

URL: http://svn.apache.org/viewvc?rev=739030&view=rev
Log:
o Fixed file handle leak

Modified:
    maven/shared/trunk/maven-verifier/src/main/java/org/apache/maven/it/Verifier.java

Modified: maven/shared/trunk/maven-verifier/src/main/java/org/apache/maven/it/Verifier.java
URL: http://svn.apache.org/viewvc/maven/shared/trunk/maven-verifier/src/main/java/org/apache/maven/it/Verifier.java?rev=739030&r1=739029&r2=739030&view=diff
==============================================================================
--- maven/shared/trunk/maven-verifier/src/main/java/org/apache/maven/it/Verifier.java (original)
+++ maven/shared/trunk/maven-verifier/src/main/java/org/apache/maven/it/Verifier.java Thu Jan 29 21:43:09 2009
@@ -306,14 +306,20 @@
     {
         Properties properties = new Properties();
 
-        FileInputStream fis;
         try
         {
             File propertiesFile = new File( getBasedir(), filename );
             if ( propertiesFile.exists() )
             {
-                fis = new FileInputStream( propertiesFile );
-                properties.load( fis );
+                FileInputStream fis = new FileInputStream( propertiesFile );
+                try
+                {
+                    properties.load( fis );
+                }
+                finally
+                {
+                    fis.close();
+                }
             }
         }
         catch ( FileNotFoundException e )