accumulo-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From medi...@apache.org
Subject svn commit: r1384121 - /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/util/shell/commands/UserPermissionsCommand.java
Date Wed, 12 Sep 2012 21:00:11 GMT
Author: medined
Date: Wed Sep 12 21:00:11 2012
New Revision: 1384121

URL: http://svn.apache.org/viewvc?rev=1384121&view=rev
Log:
fixed incorrect null pointer check

Modified:
    accumulo/trunk/core/src/main/java/org/apache/accumulo/core/util/shell/commands/UserPermissionsCommand.java

Modified: accumulo/trunk/core/src/main/java/org/apache/accumulo/core/util/shell/commands/UserPermissionsCommand.java
URL: http://svn.apache.org/viewvc/accumulo/trunk/core/src/main/java/org/apache/accumulo/core/util/shell/commands/UserPermissionsCommand.java?rev=1384121&r1=1384120&r2=1384121&view=diff
==============================================================================
--- accumulo/trunk/core/src/main/java/org/apache/accumulo/core/util/shell/commands/UserPermissionsCommand.java
(original)
+++ accumulo/trunk/core/src/main/java/org/apache/accumulo/core/util/shell/commands/UserPermissionsCommand.java
Wed Sep 12 21:00:11 2012
@@ -39,7 +39,7 @@ public class UserPermissionsCommand exte
     String delim = "";
     shellState.getReader().printString("System permissions: ");
     for (SystemPermission p : SystemPermission.values()) {
-      if (shellState.getConnector().securityOperations().hasSystemPermission(user, p) &
p != null) {
+      if (p != null && shellState.getConnector().securityOperations().hasSystemPermission(user,
p)) {
         shellState.getReader().printString(delim + "System." + p.name());
         delim = ", ";
       }



Mime
View raw message