[SSSD] [PATCH] Save original memberof, not memberof

Stephen Gallagher sgallagh at redhat.com
Tue Dec 20 18:56:40 UTC 2011


On Tue, 2011-12-20 at 13:18 -0500, Stephen Gallagher wrote:
> 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

Pushed to master.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20111220/c453142b/attachment.sig>


More information about the sssd-devel mailing list