[Pkg-openldap-devel] back..

[Russ Allbery]

Steve Langasek <vorlon at debian.org> writes:

> The only thing I'm not clear on is why this particular check:

> +			if( connections[i].c_struct_state == SLAP_C_USED ) {

> is moved inside the mutex, given that there's another check against
> c_struct_state immediately preceding the mutex.

The check outside the mutex is on whether that connection has been
initialized, so my assumption was that if the outer check failed, there
was no mutex to use.  So as much of the check as possible is inside the
mutex; the outer check can't similarly be moved.

-- 
Russ Allbery (rra at debian.org)               <http://www.eyrie.org/~eagle/>