You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by ec...@apache.org on 2013/11/26 23:34:06 UTC
svn commit: r1545877 -
/hbase/branches/0.96/hbase-server/src/main/java/org/apache/hadoop/hbase/ZNodeClearer.java
Author: eclark
Date: Tue Nov 26 22:34:05 2013
New Revision: 1545877
URL: http://svn.apache.org/r1545877
Log:
HBASE-10038 Fix potential Resource Leak in ZNodeCleaner
Modified:
hbase/branches/0.96/hbase-server/src/main/java/org/apache/hadoop/hbase/ZNodeClearer.java
Modified: hbase/branches/0.96/hbase-server/src/main/java/org/apache/hadoop/hbase/ZNodeClearer.java
URL: http://svn.apache.org/viewvc/hbase/branches/0.96/hbase-server/src/main/java/org/apache/hadoop/hbase/ZNodeClearer.java?rev=1545877&r1=1545876&r2=1545877&view=diff
==============================================================================
--- hbase/branches/0.96/hbase-server/src/main/java/org/apache/hadoop/hbase/ZNodeClearer.java (original)
+++ hbase/branches/0.96/hbase-server/src/main/java/org/apache/hadoop/hbase/ZNodeClearer.java Tue Nov 26 22:34:05 2013
@@ -94,10 +94,14 @@ public class ZNodeClearer {
throw new FileNotFoundException("No filename; set environment variable HBASE_ZNODE_FILE");
}
FileReader znodeFile = new FileReader(fileName);
- BufferedReader br = new BufferedReader(znodeFile);
- String file_content = br.readLine();
- br.close();
- return file_content;
+ BufferedReader br = null;
+ try {
+ br = new BufferedReader(znodeFile);
+ String file_content = br.readLine();
+ return file_content;
+ } finally {
+ if (br != null) br.close();
+ }
}
/**