[Date Prev][Date Next]
[Chronological]
[Thread]
[Top]
Re: (ITS#6684) Patch for autogroup overlay
moenoel@informatik.uni-bremen.de wrote:
>> 1) it should simply be comparing the AttributeDescription pointers
>> 2) since the "memberof" attribute is actually configurable in the memberof
>> overlay, there's no guarantee that this is the correct attribute to be looking
>> for. It should also be configurable in your patch.
>>
>> You're using strcasecmp, but your inputs are already normalized values. You
>> should just use ber_bvcmp.
>>
>
> Since I am also interested in this, I took some time to make a new
> patch. I took Norberts original patch, applied it to a current checkout
> from HEAD and tried to fix the issues mentioned by Howard. My initial
> tests are looking good.
>
> My C skills are rather mediocre, though, so I hope I didn't slaughter
> the thing :-)
>
> http://www.informatik.uni-bremen.de/~moenoel/ldap/christian-manal-autogroup-27012011.tgz
This patch looks pretty good. There are only one or two minor issues with it,
which I will clean up. (E.g., config actions require no mutexes; slapd is
always single-threaded when processing config changes.)
--
-- Howard Chu
CTO, Symas Corp. http://www.symas.com
Director, Highland Sun http://highlandsun.com/hyc/
Chief Architect, OpenLDAP http://www.openldap.org/project/