]> git.sur5r.net Git - openldap/commitdiff
ITS#6361 remove asserts
authorHoward Chu <hyc@openldap.org>
Fri, 13 Nov 2009 23:48:51 +0000 (23:48 +0000)
committerHoward Chu <hyc@openldap.org>
Fri, 13 Nov 2009 23:48:51 +0000 (23:48 +0000)
servers/slapd/connection.c
servers/slapd/daemon.c

index e1f651bd2b8aea769d8c98d741ff906fd16e596c..43faf736a511b65f30d575dcbf813fbade6c81ed 100644 (file)
@@ -783,7 +783,9 @@ void connection_closing( Connection *c, const char *why )
 {
        assert( connections != NULL );
        assert( c != NULL );
-       assert( c->c_struct_state == SLAP_C_USED );
+
+       if ( c->c_struct_state != SLAP_C_USED ) return;
+
        assert( c->c_conn_state != SLAP_C_INVALID );
 
        /* c_mutex must be locked by caller */
@@ -816,7 +818,9 @@ connection_close( Connection *c )
 {
        assert( connections != NULL );
        assert( c != NULL );
-       assert( c->c_struct_state == SLAP_C_USED );
+
+       if ( c->c_struct_state != SLAP_C_USED ) return;
+
        assert( c->c_conn_state == SLAP_C_CLOSING );
 
        /* NOTE: c_mutex should be locked by caller */
index 8027f2a78847d7289e4fc57201d01d3f96520e48..1acc659712ffeab3cd14bbd5087bc785cadfb0a7 100644 (file)
@@ -989,13 +989,17 @@ slapd_clr_read( ber_socket_t s, int wake )
 void
 slapd_set_read( ber_socket_t s, int wake )
 {
+       int do_wake = 1;
        ldap_pvt_thread_mutex_lock( &slap_daemon.sd_mutex );
 
-       assert( SLAP_SOCK_IS_ACTIVE( s ));
-       if (!SLAP_SOCK_IS_READ( s )) SLAP_SOCK_SET_READ( s );
-
+       if( SLAP_SOCK_IS_ACTIVE( s ) && !SLAP_SOCK_IS_READ( s )) {
+               SLAP_SOCK_SET_READ( s );
+       } else {
+               do_wake = 0;
+       }
        ldap_pvt_thread_mutex_unlock( &slap_daemon.sd_mutex );
-       WAKE_LISTENER(wake);
+       if ( do_wake )
+               WAKE_LISTENER(wake);
 }
 
 time_t