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

Attachment: signature.asc
Description: This is a digitally signed message part

_______________________________________________
sssd-devel mailing list
sssd-devel@lists.fedorahosted.org
https://fedorahosted.org/mailman/listinfo/sssd-devel

Reply via email to