upstream/openldap
1
0
Fork 0
mirror of https://git.openldap.org/openldap/openldap.git synced 2025-12-24 08:39:37 -05:00
Code Issues Projects Releases Packages Wiki Activity Actions

#6684 cleanup unneeded agi_mutex locks

Browse source
This commit is contained in:
Howard Chu 2011-01-29 21:23:22 +00:00
parent 7b5dbc0b61
commit 01e1b1e842
1 changed files with 0 additions and 21 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

21
contrib/slapd-modules/autogroup/autogroup.c
View file

@ -1588,8 +1588,6 @@ ag_cfgen( ConfigArgs *c )
if ( c->op == SLAP_CONFIG_EMIT ) {
ldap_pvt_thread_mutex_lock( &agi->agi_mutex );
switch( c->type ){
case AG_ATTRSET:
for ( i = 0 ; agd ; i++, agd = agd->agd_next ) {
@ -1624,8 +1622,6 @@ ag_cfgen( ConfigArgs *c )
return 1;
}
ldap_pvt_thread_mutex_unlock( &agi->agi_mutex );
return rc;
}else if ( c->op == LDAP_MOD_DELETE ) {
@ -1635,8 +1631,6 @@ ag_cfgen( ConfigArgs *c )
autogroup_filter_t *agf = age->age_filter,
*agf_next;
ldap_pvt_thread_mutex_lock( &agi->agi_mutex );
for ( agd_next = agd; agd_next; agd = agd_next ) {
agd_next = agd->agd_next;
@ -1664,9 +1658,6 @@ ag_cfgen( ConfigArgs *c )
ch_free( age );
}
ldap_pvt_thread_mutex_unlock( &agi->agi_mutex );
ldap_pvt_thread_mutex_destroy( &agi->agi_mutex );
ch_free( agi );
on->on_bi.bi_private = NULL;
@ -1676,8 +1667,6 @@ ag_cfgen( ConfigArgs *c )
autogroup_filter_t *agf,
*agf_next;
ldap_pvt_thread_mutex_lock( &agi->agi_mutex );
for ( i = 0, agdp = &agi->agi_def;
i < c->valx; i++ )
{
@ -1722,7 +1711,6 @@ ag_cfgen( ConfigArgs *c )
ch_free( agd );
agd = agi->agi_def;
ldap_pvt_thread_mutex_unlock( &agi->agi_mutex );
}
@ -1784,8 +1772,6 @@ ag_cfgen( ConfigArgs *c )
return 1;
}
ldap_pvt_thread_mutex_lock( &agi->agi_mutex );
for ( agdp = &agi->agi_def ; *agdp ; agdp = &(*agdp)->agd_next ) {
/* The same URL attribute / member attribute pair
* cannot be repeated */
@ -1815,7 +1801,6 @@ ag_cfgen( ConfigArgs *c )
Debug( LDAP_DEBUG_ANY, "%s: %s.\n",
c->log, c->cr_msg, 0 );
ldap_pvt_thread_mutex_unlock( &agi->agi_mutex );
return 1;
}
agdp = &(*agdp)->agd_next;
@ -1835,8 +1820,6 @@ ag_cfgen( ConfigArgs *c )
(*agdp)->agd_member_ad = member_ad;
(*agdp)->agd_next = agd_next;
ldap_pvt_thread_mutex_unlock( &agi->agi_mutex );
} break;
case AG_MEMBER_OF_AD: {
@ -1866,12 +1849,8 @@ ag_cfgen( ConfigArgs *c )
return 1;
}
ldap_pvt_thread_mutex_lock( &agi->agi_mutex );
agi->agi_memberof_ad = memberof_ad;
ldap_pvt_thread_mutex_unlock( &agi->agi_mutex );
} break;
default: