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/08/29 00:22:32 UTC
svn commit: r437864 -
/directory/branches/shared/0.9.5/ldap/src/main/java/org/apache/directory/shared/ldap/codec/LdapMessageGrammar.java
Author: elecharny
Date: Mon Aug 28 15:22:31 2006
New Revision: 437864
URL: http://svn.apache.org/viewvc?rev=437864&view=rev
Log:
Added a comment to clarify an exception
Modified:
directory/branches/shared/0.9.5/ldap/src/main/java/org/apache/directory/shared/ldap/codec/LdapMessageGrammar.java
Modified: directory/branches/shared/0.9.5/ldap/src/main/java/org/apache/directory/shared/ldap/codec/LdapMessageGrammar.java
URL: http://svn.apache.org/viewvc/directory/branches/shared/0.9.5/ldap/src/main/java/org/apache/directory/shared/ldap/codec/LdapMessageGrammar.java?rev=437864&r1=437863&r2=437864&view=diff
==============================================================================
--- directory/branches/shared/0.9.5/ldap/src/main/java/org/apache/directory/shared/ldap/codec/LdapMessageGrammar.java (original)
+++ directory/branches/shared/0.9.5/ldap/src/main/java/org/apache/directory/shared/ldap/codec/LdapMessageGrammar.java Mon Aug 28 15:22:31 2006
@@ -99,6 +99,7 @@
if ( tlv.getLength().getLength() == 0 )
{
log.error( "The LdapMessage has a zero length. This is not allowed" );
+ // This will generate a PROTOCOL_ERROR
throw new DecoderException( "The LdapMessage should not be empty" );
}
@@ -142,6 +143,8 @@
if ( tlv.getLength().getLength() == 0 )
{
log.error( "The messageId has a zero length. This is not allowed" );
+
+ // This will generate a PROTOCOL_ERROR
throw new DecoderException( "The messageId should not be null" );
}
@@ -163,6 +166,7 @@
log.error( "The Message Id " + StringTools.dumpBytes( value.getData() ) + " is invalid : "
+ ide.getMessage() + ". The message ID must be between (0 .. 2 147 483 647)" );
+ // This will generate a PROTOCOL_ERROR
throw new DecoderException( ide.getMessage() );
}