[jboss-cvs] JBossAS SVN: r58309 - trunk/server/src/main/org/jboss/deployment

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

Author: wolfc
Date: 2006-11-13 16:39:35 -0500 (Mon, 13 Nov 2006)
New Revision: 58309

Modified:
   trunk/server/src/main/org/jboss/deployment/JBossApplicationObjectFactory.java
Log:
fixed NPE (TCK ejb30/sec/stateful/lsecp)

Modified: trunk/server/src/main/org/jboss/deployment/JBossApplicationObjectFactory.java
===================================================================
--- trunk/server/src/main/org/jboss/deployment/JBossApplicationObjectFactory.java	2006-11-13 21:39:32 UTC (rev 58308)
+++ trunk/server/src/main/org/jboss/deployment/JBossApplicationObjectFactory.java	2006-11-13 21:39:35 UTC (rev 58309)
@@ -158,6 +158,13 @@
       {
          J2eeApplicationMetaData metaData = activeMetaData.get();
          SecurityRoleMetaData srmd = metaData.getSecurityRole(role.getRoleName());
+         // Wolf: there could be no application.xml and thus no srmd
+         if(srmd == null)
+         {
+            srmd = new SecurityRoleMetaData();
+            srmd.setRoleName(role.getRoleName());
+            metaData.addSecurityRole(srmd);
+         }
          srmd.addPrincipalName(value);
       }
    }