mirror/openldap
2
0
Fork 0
mirror of https://git.openldap.org/openldap/openldap.git synced 2025-01-24 13:24:56 +08:00
Code Issues Packages Projects Releases Wiki Activity

connection_get:

Browse Source 
Connection could be closed before acquiring connections_mutex.
	if c->c_struct_state == SLAP_C_UNUSED, return NULL.
This commit is contained in:
Kurt Zeilenga 1999-06-12 00:03:39 +00:00
parent 0dda34c9e4
commit 35e91be482
1 changed files with 14 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
servers/slapd/connection.c
View File

@ -156,13 +156,15 @@ static Connection* connection_get( int s )
#ifndef HAVE_WINSOCK
c = &connections[s];
assert( c->c_struct_state == SLAP_C_USED );
assert( c->c_struct_state != SLAP_C_UNINITIALIZED );
ldap_pvt_thread_mutex_lock( &c->c_mutex );
if( c->c_struct_state != SLAP_C_USED ) {
/* connection must have been closed */
assert( c->c_conn_state == SLAP_C_INVALID );
assert( !ber_pvt_sb_in_use( c->c_sb ) );
assert( c->c_struct_state == SLAP_C_USED );
assert( c->c_conn_state != SLAP_C_INVALID );
assert( ber_pvt_sb_in_use( c->c_sb ) );
return NULL;
}
#else
c = NULL;
@ -188,7 +190,6 @@ static Connection* connection_get( int s )
if( ber_pvt_sb_get_desc( connections[i].c_sb ) == s ) {
c = &connections[i];
ldap_pvt_thread_mutex_lock( &c->c_mutex );
break;
}
}
@ -196,11 +197,18 @@ static Connection* connection_get( int s )
#endif
if( c != NULL ) {
ldap_pvt_thread_mutex_lock( &c->c_mutex );
/* we do this AFTER locking to aid in debugging */
Debug( LDAP_DEBUG_TRACE,
"connection_get(%d): got connid=%ld\n",
s, c->c_connid, 0 );
assert( c->c_struct_state == SLAP_C_USED );
assert( c->c_conn_state != SLAP_C_INVALID );
assert( ber_pvt_sb_in_use( c->c_sb ) );
}
return c;
}