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/12/17 00:46:06 UTC

svn commit: r1551408 - /hbase/branches/0.98/hbase-client/src/main/java/org/apache/hadoop/hbase/security/access/TablePermission.java

Author: tedyu
Date: Mon Dec 16 23:46:06 2013
New Revision: 1551408

URL: http://svn.apache.org/r1551408
Log:
HBASE-10178 Potential null object dereference in TablePermission#equals()


Modified:
    hbase/branches/0.98/hbase-client/src/main/java/org/apache/hadoop/hbase/security/access/TablePermission.java

Modified: hbase/branches/0.98/hbase-client/src/main/java/org/apache/hadoop/hbase/security/access/TablePermission.java
URL: http://svn.apache.org/viewvc/hbase/branches/0.98/hbase-client/src/main/java/org/apache/hadoop/hbase/security/access/TablePermission.java?rev=1551408&r1=1551407&r2=1551408&view=diff
==============================================================================
--- hbase/branches/0.98/hbase-client/src/main/java/org/apache/hadoop/hbase/security/access/TablePermission.java (original)
+++ hbase/branches/0.98/hbase-client/src/main/java/org/apache/hadoop/hbase/security/access/TablePermission.java Mon Dec 16 23:46:06 2013
@@ -324,7 +324,7 @@ public class TablePermission extends Per
         ((qualifier == null && other.getQualifier() == null) ||
          Bytes.equals(qualifier, other.getQualifier())) &&
         ((namespace == null && other.getNamespace() == null) ||
-         namespace.equals(other.getNamespace()))
+         (namespace != null && namespace.equals(other.getNamespace())))
        )) {
       return false;
     }