You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by te...@apache.org on 2013/11/26 20:43:26 UTC

svn commit: r1545796 - /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/fs/HFileSystem.java

Author: tedyu
Date: Tue Nov 26 19:43:25 2013
New Revision: 1545796

URL: http://svn.apache.org/r1545796
Log:
HBASE-10029 Addendum checks for args against null


Modified:
    hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/fs/HFileSystem.java

Modified: hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/fs/HFileSystem.java
URL: http://svn.apache.org/viewvc/hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/fs/HFileSystem.java?rev=1545796&r1=1545795&r2=1545796&view=diff
==============================================================================
--- hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/fs/HFileSystem.java (original)
+++ hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/fs/HFileSystem.java Tue Nov 26 19:43:25 2013
@@ -264,7 +264,7 @@ public class HFileSystem extends FilterF
               public Object invoke(Object proxy, Method method,
                                    Object[] args) throws Throwable {
                 try {
-                  if (args.length == 0 && "close".equals(method.getName())) {
+                  if ((args == null || args.length == 0) && "close".equals(method.getName())) {
                     if (cp instanceof Closeable) {
                       ((Closeable)cp).close();
                     } else {