On Tue, 2011-12-20 at 16:16 +0100, Jakub Hrozek wrote:
On Tue, Dec 20, 2011 at 01:45:59PM +0100, Jan Zelený wrote:
> > The sdap_save_* refactoring brought one bug where instead of saving into
> > SYSDB_ORIG_MEMBEROF we would save into SYSDB_MEMBEROF. I rechecked all
> > other attribute names and this was the only one that was changed.
>
> Nack,
>
> the issue is deeper than it looks. In the original code, there was conversion
> from SYSDB_MEMBEROF to SYSDB_ORIG_MEMBEROF, hence this fix should be done not
> only here, but also in user map. Also please review if the memberof attribute
> is mapped correctly in other maps, I think I saw some similar code on other
> places.
>
> Thanks
> Jan
Thanks, you're right. New patch attached.
Ack