[jboss-cvs] JBossAS SVN: r58200 - trunk/server/src/main/org/jboss/web/deployers

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

Author: anil.saldhana at jboss.com
Date: 2006-11-08 13:20:05 -0500 (Wed, 08 Nov 2006)
New Revision: 58200

Modified:
   trunk/server/src/main/org/jboss/web/deployers/AbstractWarDeployment.java
Log:
add null checks

Modified: trunk/server/src/main/org/jboss/web/deployers/AbstractWarDeployment.java
===================================================================
--- trunk/server/src/main/org/jboss/web/deployers/AbstractWarDeployment.java	2006-11-08 16:36:02 UTC (rev 58199)
+++ trunk/server/src/main/org/jboss/web/deployers/AbstractWarDeployment.java	2006-11-08 18:20:05 UTC (rev 58200)
@@ -247,6 +247,8 @@
     */
    public String getDefaultSecurityDomain()
    {
+	  if(defaultSecurityDomain == null)
+		  throw new IllegalStateException("Default Security Domain is null");
       return defaultSecurityDomain;
    }
 
@@ -431,6 +433,8 @@
    protected void processEnc(ClassLoader loader, WebMetaData metaData)
       throws Exception
    {
+	  if(loader == null)
+		  throw new IllegalArgumentException("Classloader passed to process ENC refs is null");
       log.debug("AbstractWebContainer.parseWebAppDescriptors, Begin");
       InitialContext iniCtx = new InitialContext();
       Context envCtx = null;