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 ) )