Re: commit: ldap/servers/slapd acl.c

["Kurt D. Zeilenga" <Kurt@OpenLDAP.org>]

At 08:36 AM 2002-01-14, Pierangelo Masarati wrote:
>> Update of /repo/OpenLDAP/pkg/ldap/servers/slapd
>> 
>> Modified Files:
>>       acl.c  1.123 -> 1.124
>> 
>> Log Message:
>> Add a default case with assert() just in case.
>
>little off topic, but I'm trying to cache the test 
>for rootdn/updatedn/any special dn; I found a good place 
>to cache is the Connection.

A simple way to handle this is just to set a flag
in the Connection/Operation structure for each of
these special cases and then to test the flag instead
of the DNs...  This will work well for rootdn and
updatedn.  (Note needed for anonymous as this is already
a simple check)

>However I note there might 
>be places where access_allowed gets called with NULL
>connection; moreover, can I safely assume op->o_ndn
>is the same as conn->c_ndn ?

No.