[Date Prev][Date Next] [Chronological] [Thread] [Top]

Re: (ITS#6684) Patch for autogroup overlay



>    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


Regards,
Christian Manal