[jboss-cvs] JBossAS SVN: r57799 - projects/security/trunk/src/main/org/jboss/security

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

Author: anil.saldhana at jboss.com
Date: 2006-10-24 12:46:03 -0400 (Tue, 24 Oct 2006)
New Revision: 57799

Modified:
   projects/security/trunk/src/main/org/jboss/security/Util.java
Log:
Additional check for null subject

Modified: projects/security/trunk/src/main/org/jboss/security/Util.java
===================================================================
--- projects/security/trunk/src/main/org/jboss/security/Util.java	2006-10-24 16:45:38 UTC (rev 57798)
+++ projects/security/trunk/src/main/org/jboss/security/Util.java	2006-10-24 16:46:03 UTC (rev 57799)
@@ -628,6 +628,8 @@
     */ 
    public static Group getSubjectRoles(Subject theSubject)
    {
+      if(theSubject == null)
+         throw new IllegalArgumentException("Subject is null");
       Set subjectGroups = theSubject.getPrincipals(Group.class);
       Iterator iter = subjectGroups.iterator();
       Group roles = null;