svn commit: r530528 - /directory/apacheds/branches/1.0/core/src/main/java/org/apache/directory/server/core/authz/TupleCache.java

[ak...@apache.org]

Author: akarasulu
Date: Thu Apr 19 12:48:45 2007
New Revision: 530528

URL: http://svn.apache.org/viewvc?view=rev&rev=530528
Log:
DIRSERVER-884:  Authorization, Prescriptive ACI Bug - Server start fails on bad ACI Entry

Modified:
    directory/apacheds/branches/1.0/core/src/main/java/org/apache/directory/server/core/authz/TupleCache.java

Modified: directory/apacheds/branches/1.0/core/src/main/java/org/apache/directory/server/core/authz/TupleCache.java
URL: http://svn.apache.org/viewvc/directory/apacheds/branches/1.0/core/src/main/java/org/apache/directory/server/core/authz/TupleCache.java?view=diff&rev=530528&r1=530527&r2=530528
==============================================================================
--- directory/apacheds/branches/1.0/core/src/main/java/org/apache/directory/server/core/authz/TupleCache.java (original)
+++ directory/apacheds/branches/1.0/core/src/main/java/org/apache/directory/server/core/authz/TupleCache.java Thu Apr 19 12:48:45 2007
@@ -89,7 +89,7 @@
      *
      * @param factoryCfg the context factory configuration for the server
      */
-    public TupleCache(DirectoryServiceConfiguration factoryCfg) throws NamingException
+    public TupleCache( DirectoryServiceConfiguration factoryCfg ) throws NamingException
     {
     	normalizerMap = factoryCfg.getGlobalRegistries().getAttributeTypeRegistry().getNormalizerMapping();
         this.nexus = factoryCfg.getPartitionNexus();
@@ -187,8 +187,15 @@
             }
             catch ( ParseException e )
             {
-                String msg = "ACIItem parser failure on '" + item + "'. Cannnot add ACITuples to TupleCache.";
-                log.warn( msg, e );
+                String msg = "ACIItem parser failure on \n'" + item + "'\ndue to syntax error. " +
+                        "Cannnot add ACITuples to TupleCache.\n" +
+                        "Check that the syntax of the ACI item is correct. \nUntil this error " +
+                        "is fixed your security settings will not be as expected.";
+                log.error( msg, e );
+                
+                // do not process this ACI Item because it will be null
+                // continue on to process the next ACI item in the entry
+                continue;
             }
 
             entryTuples.addAll( item.toTuples() );