On Tue, Mar 04, 2014 at 04:15:37PM +0100, Jakub Hrozek wrote:
On Tue, Mar 04, 2014 at 03:50:21PM +0100, Jakub Hrozek wrote:
> Hi,
>
> the attached patch fixes
>
https://bugzilla.redhat.com/show_bug.cgi?id=1071578
>
> The problem was that the intermediate computing we did added an
> originalMemberUser attribute to sysdb_attrs that was never intended to
> be stored in sysdb. But because we stored the attributes after the
> computing, we also stored originalMemberUser.
>
> When sssd was online, this wasn't a problem as the cache got purged and
> recreated again, but when offline, adding another originalMemberUser
> added a duplicate attribute so storing the rules failed with EEXIST and
> the user was assigned a default rule.
>
> Another improvement might be to only delete and store the rules when
> online -- right now the logic is a bit strange when offline as we read
> rules from the cache, delete them and then store the same rules again.
Attached is also a version for 1.11 that differs in DEBUG messages only.
... and changed sysdb calls :-)
I wasn't able to reproduce the issue with the patch applied so ACK.
bye,
Sumit