[jboss-cvs] jboss-seam/src/main/org/jboss/seam/security ...

[Shane Bryzak]

  User: sbryzak2
  Date: 07/11/06 10:12:24

  Modified:    src/main/org/jboss/seam/security  Identity.java
  Log:
  don't allow null roles to be added
  
  Revision  Changes    Path
  1.103     +2 -0      jboss-seam/src/main/org/jboss/seam/security/Identity.java
  
  (In the diff below, changes in quantity of whitespace are not shown.)
  
  Index: Identity.java
  ===================================================================
  RCS file: /cvsroot/jboss/jboss-seam/src/main/org/jboss/seam/security/Identity.java,v
  retrieving revision 1.102
  retrieving revision 1.103
  diff -u -b -r1.102 -r1.103
  --- Identity.java	6 Nov 2007 08:56:24 -0000	1.102
  +++ Identity.java	6 Nov 2007 15:12:24 -0000	1.103
  @@ -364,6 +364,8 @@
       */
      public boolean addRole(String role)
      {
  +      if (role == null || "".equals(role)) return false;
  +      
         if (!isLoggedIn())
         {
            preAuthenticationRoles.add(role);