[Date Prev][Date Next]
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 :-)