mirror/openldap
2
0
Fork 0
mirror of https://git.openldap.org/openldap/openldap.git synced 2025-01-06 10:46:21 +08:00
Code Issues Packages Projects Releases Wiki Activity

More proper criticality handling. Implementation is a kludge.

Browse Source
This commit is contained in:
Kurt Zeilenga 2005-01-25 18:30:55 +00:00
parent d68041ce50
commit 0bd2dafd53
2 changed files with 10 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
servers/slapd/backend.c
View File

@ -803,22 +803,20 @@ backend_check_controls(
if( ctrls ) {
for( ; *ctrls != NULL ; ctrls++ ) {
/* KLUDGE: ldctl_iscritical munged by controls.c:get_ctrls()
* to ensure this check is enabled/disabled appropriately.
*/
if( (*ctrls)->ldctl_iscritical && !ldap_charray_inlist(
op->o_bd->be_controls, (*ctrls)->ldctl_oid ) )
{
/* Per RFC 2251 (and LDAPBIS discussions), if the control
* is recognized and appropriate for the operation (which
* we've already verified), then the server should make
* use of the control when performing the operation
* (regardless of the criticality of the control).
* use of the control when performing the operation.
*
* Here we find that operation extended by the control
* is not unavailable in a particular context, hence the
* return of unwillingToPerform.
*
* FIXME: As noted above, this check should be done
* regardless of the criticality of the control. The
* frontend infrastructure doesn't (yet) support this.
*/
rs->sr_text = "control unavailable in context";
rs->sr_err = LDAP_UNWILLING_TO_PERFORM;

8
servers/slapd/controls.c
View File

@ -613,14 +613,18 @@ int get_ctrls(
}
if ( sc->sc_mask & SLAP_CTRL_FRONTEND ) {
/* kludge to disable backend_control() check */
/* KLUDGE: disable backend_control() check */
c->ldctl_iscritical = 0;
} else if ( tagmask == SLAP_CTRL_SEARCH &&
sc->sc_mask & SLAP_CTRL_FRONTEND_SEARCH )
{
/* kludge to disable backend_control() check */
/* KLUDGE: disable backend_control() check */
c->ldctl_iscritical = 0;
} else {
/* KLUDGE: enable backend_control() check */
c->ldctl_iscritical = 1;
}
} else if( c->ldctl_iscritical ) {