You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@directory.apache.org by el...@apache.org on 2006/12/22 19:22:07 UTC

svn commit: r489712 - /directory/trunks/shared/ldap/src/main/java/org/apache/directory/shared/ldap/schema/syntax/OtherMailboxSyntaxChecker.java

Author: elecharny
Date: Fri Dec 22 10:22:06 2006
New Revision: 489712

URL: http://svn.apache.org/viewvc?view=rev&rev=489712
Log:
fixed the check to handle correctly non escaped $

Modified:
    directory/trunks/shared/ldap/src/main/java/org/apache/directory/shared/ldap/schema/syntax/OtherMailboxSyntaxChecker.java

Modified: directory/trunks/shared/ldap/src/main/java/org/apache/directory/shared/ldap/schema/syntax/OtherMailboxSyntaxChecker.java
URL: http://svn.apache.org/viewvc/directory/trunks/shared/ldap/src/main/java/org/apache/directory/shared/ldap/schema/syntax/OtherMailboxSyntaxChecker.java?view=diff&rev=489712&r1=489711&r2=489712
==============================================================================
--- directory/trunks/shared/ldap/src/main/java/org/apache/directory/shared/ldap/schema/syntax/OtherMailboxSyntaxChecker.java (original)
+++ directory/trunks/shared/ldap/src/main/java/org/apache/directory/shared/ldap/schema/syntax/OtherMailboxSyntaxChecker.java Fri Dec 22 10:22:06 2006
@@ -105,6 +105,12 @@
         
         String mailbox = ( ( dollar < strValue.length() - 1 ) ? 
             strValue.substring( dollar + 1 ) : "" ); 
+        
+        // The mailbox should not contains a '$'
+        if ( mailbox.indexOf( '$' ) != -1 )
+        {
+            return false;
+        }
             
         // Check that the mailboxType is a PrintableString
         if ( !StringTools.isPrintableString( mailboxType ) )