[jboss-cvs] Picketbox SVN: r36 - trunk/security-jboss-sx/jbosssx/src/main/java/org/jboss/security.

[jboss-cvs-commits at lists.jboss.org]

Author: anil.saldhana at jboss.com
Date: 2010-02-18 17:27:30 -0500 (Thu, 18 Feb 2010)
New Revision: 36

Modified:
   trunk/security-jboss-sx/jbosssx/src/main/java/org/jboss/security/SimplePrincipal.java
Log:
use security actions

Modified: trunk/security-jboss-sx/jbosssx/src/main/java/org/jboss/security/SimplePrincipal.java
===================================================================
--- trunk/security-jboss-sx/jbosssx/src/main/java/org/jboss/security/SimplePrincipal.java	2010-02-17 11:44:18 UTC (rev 35)
+++ trunk/security-jboss-sx/jbosssx/src/main/java/org/jboss/security/SimplePrincipal.java	2010-02-18 22:27:30 UTC (rev 36)
@@ -22,9 +22,7 @@
 package org.jboss.security;
 
 import java.io.Serializable;
-import java.security.AccessController;
 import java.security.Principal;
-import java.security.PrivilegedAction;
 
 /**
  * A simple String based implementation of Principal. Typically a SimplePrincipal is created given a userID which is
@@ -49,20 +47,14 @@
     * org.jboss.security.simpleprincipal.equals.override is set to true will only
     * compare instances of SimplePrincipals.
     * @return true if name equals another.getName();
-    */
+    */ 
    @Override
    public boolean equals(Object another)
    {
       if (!(another instanceof Principal))
          return false;
-      if ("true".equals(AccessController.doPrivileged(new PrivilegedAction()
+      if ("true".equals(SecurityActions.getProperty(OVERRIDE_EQUALS_BEHAVIOR, "false")))
       {
-         public Object run()
-         {
-            return System.getProperty(OVERRIDE_EQUALS_BEHAVIOR, "false");
-         }
-      })))
-      {
          if (!(another instanceof SimplePrincipal))
             return false;
       }