You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@directory.apache.org by ka...@apache.org on 2010/03/08 12:59:50 UTC

svn commit: r920285 - /directory/apacheds/trunk/protocol-ldap/src/main/java/org/apache/directory/server/ldap/handlers/LdapRequestHandler.java

Author: kayyagari
Date: Mon Mar  8 11:59:49 2010
New Revision: 920285

URL: http://svn.apache.org/viewvc?rev=920285&view=rev
Log:
fix for DIRSERVER-1404

Modified:
    directory/apacheds/trunk/protocol-ldap/src/main/java/org/apache/directory/server/ldap/handlers/LdapRequestHandler.java

Modified: directory/apacheds/trunk/protocol-ldap/src/main/java/org/apache/directory/server/ldap/handlers/LdapRequestHandler.java
URL: http://svn.apache.org/viewvc/directory/apacheds/trunk/protocol-ldap/src/main/java/org/apache/directory/server/ldap/handlers/LdapRequestHandler.java?rev=920285&r1=920284&r2=920285&view=diff
==============================================================================
--- directory/apacheds/trunk/protocol-ldap/src/main/java/org/apache/directory/server/ldap/handlers/LdapRequestHandler.java (original)
+++ directory/apacheds/trunk/protocol-ldap/src/main/java/org/apache/directory/server/ldap/handlers/LdapRequestHandler.java Mon Mar  8 11:59:49 2010
@@ -128,6 +128,14 @@
     {
         LdapSession ldapSession = ldapServer.getLdapSessionManager().getLdapSession( session );
         
+        if( ldapSession == null )
+        {
+            // in some cases the session is becoming null though the client is sending the UnbindRequest
+            // before closing
+            LOG.info( "ignoring the message {} received from null session", message  );
+            return;
+        }
+        
         // First check that the client hasn't issued a previous BindRequest, unless it
         // was a SASL BindRequest
         if ( ldapSession.isAuthPending() )