From: Kurt Zeilenga <kurt@openldap.org>
Date: Thu, 10 Jun 1999 22:06:05 +0000 (+0000)
Subject: Close inactive streams instead of asserting condition.
X-Git-Tag: OPENLDAP_REL_ENG_2_BP~345
X-Git-Url: https://git.sur5r.net/?a=commitdiff_plain;h=368628e9d52b09e7e52c4d2141c7259ec881677e;p=openldap

Close inactive streams instead of asserting condition.
---

diff --git a/servers/slapd/daemon.c b/servers/slapd/daemon.c
index 2d3e8f9d6b..82d18b019e 100644
--- a/servers/slapd/daemon.c
+++ b/servers/slapd/daemon.c
@@ -532,78 +532,89 @@ slapd_daemon_task(
 
 		/* loop through the writers */
 #ifdef HAVE_WINSOCK
-		for ( i = 0; i < writefds.fd_count; i++ ) {
-			int wd = writefds.fd_array[i];
+		for ( i = 0; i < writefds.fd_count; i++ )
+#else
+		for ( i = 0; i < nfds; i++ )
+#endif
+		{
+			int wd, active;
+
+#ifdef HAVE_WINSOCK
+			wd = writefds.fd_array[i];
+#else
+			if( ! FD_ISSET( i, &writefds ) ) {
+				continue;
+			}
+			wd = i;
+#endif
 
 			if ( wd == tcps ) {
 				continue;
 			}
 
 			Debug( LDAP_DEBUG_CONNS,
-				"daemon: signalling write waiter on %d\n",
+				"daemon: write active on %d\n",
 				wd, 0, 0 );
 
+			ldap_pvt_thread_mutex_lock( &slap_daemon.sd_mutex );
+			active = FD_ISSET( wd, &slap_daemon.sd_actives );
+			ldap_pvt_thread_mutex_unlock( &slap_daemon.sd_mutex );
+
+			if( ! active ) {
+				/* descriptor no longer in FD set, should be closed */
+				Debug( LDAP_DEBUG_CONNS,
+			   		"daemon: write %d inactive, closing.\n", wd, 0, 0 );
+				tcp_close( wd );
+				continue;
+			}
+
 			slapd_clr_write( wd, 0 );
 			if ( connection_write( wd ) < 0 ) {
 				FD_CLR( (unsigned) wd, &readfds );
 				slapd_close( wd );
 			}
 		}
+
+#ifdef HAVE_WINSOCK
+		for ( i = 0; i < readfds.fd_count; i++ )
 #else
-		for ( i = 0; i < nfds; i++ ) {
-			if ( i == tcps ) {
+		for ( i = 0; i < nfds; i++ )
+#endif
+		{
+			int rd, active;
+
+#ifdef HAVE_WINSOCK
+			rd = readfds.fd_array[i];
+#else
+			if( ! FD_ISSET( i, &readfds ) ) {
 				continue;
 			}
-			if ( FD_ISSET( i, &writefds ) ) {
-				Debug( LDAP_DEBUG_CONNS,
-				    "daemon: signaling write waiter on %d\n", i, 0, 0 );
-
-				/* clear the write flag */
-				slapd_clr_write( i, 0 );
-				
-				if( connection_write( i ) < 0 ) { 
-					FD_CLR( i, &readfds );
-					slapd_close( i );
-				}
-			}
-		}
+			rd = i;
 #endif
 
-#ifdef HAVE_WINSOCK
-		for ( i = 0; i < readfds.fd_count; i++ ) {
-			int rd = readfds.fd_array[i];
 			if ( rd == tcps ) {
 				continue;
 			}
+
 			Debug ( LDAP_DEBUG_CONNS,
 				"daemon: read activity on %d\n", rd, 0, 0 );
 
 			ldap_pvt_thread_mutex_lock( &slap_daemon.sd_mutex );
-			assert( FD_ISSET( rd, &slap_daemon.sd_actives) );
+			active = FD_ISSET( rd, &slap_daemon.sd_actives );
 			ldap_pvt_thread_mutex_unlock( &slap_daemon.sd_mutex );
 
-			if ( connection_read( rd ) < 0 ) {
-				slapd_close( rd );
-			}
-		}
-#else
-		for ( i = 0; i < nfds; i++ ) {
-			if ( i == tcps ) {
+			if( ! active ) {
+				/* descriptor no longer in FD set, should be closed */
+				Debug( LDAP_DEBUG_CONNS,
+			   		"daemon: read %d inactive, closing.\n", rd, 0, 0 );
+				tcp_close( rd );
 				continue;
 			}
 
-			if ( FD_ISSET( i, &readfds ) ) {
-				Debug( LDAP_DEBUG_CONNS,
-				    "daemon: read activity on %d\n", i, 0, 0 );
-
-				assert( FD_ISSET( i, &slap_daemon.sd_actives) );
-
-				if( connection_read( i ) < 0) {
-					slapd_close( i );
-				}
+			if ( connection_read( rd ) < 0 ) {
+				slapd_close( rd );
 			}
 		}
-#endif
 		ldap_pvt_thread_yield();
 	}