You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@accumulo.apache.org by vi...@apache.org on 2013/02/26 18:34:29 UTC

svn commit: r1450297 - in /accumulo/trunk: ./ test/src/main/java/org/apache/accumulo/test/randomwalk/security/WalkingSecurity.java

Author: vines
Date: Tue Feb 26 17:34:28 2013
New Revision: 1450297

URL: http://svn.apache.org/r1450297
Log:
ACCUMULO-1111 - fixed infinite loops clobering the stack



Modified:
    accumulo/trunk/   (props changed)
    accumulo/trunk/test/src/main/java/org/apache/accumulo/test/randomwalk/security/WalkingSecurity.java

Propchange: accumulo/trunk/
------------------------------------------------------------------------------
  Merged /accumulo/branches/1.5:r1450274-1450296

Modified: accumulo/trunk/test/src/main/java/org/apache/accumulo/test/randomwalk/security/WalkingSecurity.java
URL: http://svn.apache.org/viewvc/accumulo/trunk/test/src/main/java/org/apache/accumulo/test/randomwalk/security/WalkingSecurity.java?rev=1450297&r1=1450296&r2=1450297&view=diff
==============================================================================
--- accumulo/trunk/test/src/main/java/org/apache/accumulo/test/randomwalk/security/WalkingSecurity.java (original)
+++ accumulo/trunk/test/src/main/java/org/apache/accumulo/test/randomwalk/security/WalkingSecurity.java Tue Feb 26 17:34:28 2013
@@ -17,6 +17,7 @@
 package org.apache.accumulo.test.randomwalk.security;
 
 import java.io.IOException;
+import java.nio.charset.Charset;
 import java.util.HashMap;
 import java.util.Map;
 import java.util.Properties;
@@ -281,7 +282,7 @@ public class WalkingSecurity extends Sec
   public byte[] getUserPassword(String user) {
     Object obj = state.get(getSysUserName() + userPass);
     if (obj instanceof PasswordToken) {
-      return ((PasswordToken)obj).getPassword();
+      return ((PasswordToken) obj).getPassword();
     }
     return null;
   }
@@ -289,7 +290,7 @@ public class WalkingSecurity extends Sec
   public byte[] getSysPassword() {
     Object obj = state.get(getSysUserName() + userPass);
     if (obj instanceof PasswordToken) {
-      return ((PasswordToken)obj).getPassword();
+      return ((PasswordToken) obj).getPassword();
     }
     return null;
   }
@@ -372,14 +373,16 @@ public class WalkingSecurity extends Sec
       throw tse;
     }
   }
-
+  
   @Override
   public SecurityToken login(Properties properties) throws AccumuloSecurityException {
-    return authenticator.login(properties);
+    if (properties.containsKey("password"))
+      return new PasswordToken().setPassword(properties.getProperty("password").getBytes(Charset.forName("UTF-8")));
+    throw new AccumuloSecurityException(properties.getProperty("user"), SecurityErrorCode.INSUFFICIENT_PROPERTIES);
   }
-
+  
   @Override
   public boolean validTokenClass(String tokenClass) {
-    return authenticator.validTokenClass(tokenClass);
+    return tokenClass.equals(PasswordToken.class.getCanonicalName());
   }
 }