[Archive-access-cvs] SF.net SVN: archive-access: [1961] trunk/archive-access/projects/wayback/ wayback-core/src/main/java/org/archive/wayback/util/flatfile/FlatFile.java

[<bra...@us...>]

Revision: 1961
          http://archive-access.svn.sourceforge.net/archive-access/?rev=1961&view=rev
Author:   bradtofel
Date:     2007-08-28 17:11:16 -0700 (Tue, 28 Aug 2007)

Log Message:
-----------
BUGFIX: (unreported) was not closing file descriptor in getReverseRecordIterator() when no results were found, which was (often.. : ) the case with empty .cdx files.

Modified Paths:
--------------
    trunk/archive-access/projects/wayback/wayback-core/src/main/java/org/archive/wayback/util/flatfile/FlatFile.java

Modified: trunk/archive-access/projects/wayback/wayback-core/src/main/java/org/archive/wayback/util/flatfile/FlatFile.java
===================================================================
--- trunk/archive-access/projects/wayback/wayback-core/src/main/java/org/archive/wayback/util/flatfile/FlatFile.java	2007-08-27 21:36:41 UTC (rev 1960)
+++ trunk/archive-access/projects/wayback/wayback-core/src/main/java/org/archive/wayback/util/flatfile/FlatFile.java	2007-08-29 00:11:16 UTC (rev 1961)
@@ -168,6 +168,7 @@
 		RandomAccessFile raf = new RandomAccessFile(file,"r");
 		long offset = findKeyOffset(raf,prefix);
 		if(offset < 1) {
+			raf.close();
 			return new ReverseRecordIterator(null);
 		}
 		raf.seek(raf.getFilePointer()-1);


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.