directory-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From akaras...@apache.org
Subject svn commit: r688578 - /directory/apacheds/trunk/protocol-ldap/src/main/java/org/apache/directory/server/ldap/LdapProtocolHandler.java
Date Sun, 24 Aug 2008 23:23:39 GMT
Author: akarasulu
Date: Sun Aug 24 16:23:39 2008
New Revision: 688578

URL: http://svn.apache.org/viewvc?rev=688578&view=rev
Log:
race conditions can naturally occur where a null LDAP session is handed off due to something
else removing the ldap session before we cleanup

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

Modified: directory/apacheds/trunk/protocol-ldap/src/main/java/org/apache/directory/server/ldap/LdapProtocolHandler.java
URL: http://svn.apache.org/viewvc/directory/apacheds/trunk/protocol-ldap/src/main/java/org/apache/directory/server/ldap/LdapProtocolHandler.java?rev=688578&r1=688577&r2=688578&view=diff
==============================================================================
--- directory/apacheds/trunk/protocol-ldap/src/main/java/org/apache/directory/server/ldap/LdapProtocolHandler.java
(original)
+++ directory/apacheds/trunk/protocol-ldap/src/main/java/org/apache/directory/server/ldap/LdapProtocolHandler.java
Sun Aug 24 16:23:39 2008
@@ -104,6 +104,12 @@
      */
     private void cleanUpSession( LdapSession ldapSession )
     {
+        if ( ldapSession == null )
+        {
+            LOG.warn( "Null LdapSession given to cleanUpSession." );
+            return;
+        }
+        
         if ( ldapSession != null )
         {
             ldapSession.abandonAllOutstandingRequests();



Mime
View raw message