[SSSD] [PATCH] LDAP: Fix retrieving a group with no members
Pavel Reichl
preichl at redhat.com
Fri Jun 27 04:50:01 UTC 2014
Well, I think I did so in my previous mail, anyway:
> * sssd-devel, subjects:
> LDAP: group_split_members returns incorrectly ENOMEM
> sudo: return after tevent_req_error
I might be that the title of the 2nd patch is misleading I didn't want to start a new thread for almost same simple patch.
Thanks,
Pavel Reichl
----- Original Message -----
From: "Jakub Hrozek" <jhrozek at redhat.com>
To: "Development of the System Security Services Daemon" <sssd-devel at lists.fedorahosted.org>
Sent: Thursday, June 26, 2014 10:48:22 PM
Subject: Re: [SSSD] [PATCH] LDAP: Fix retrieving a group with no members
On 26 Jun 2014, at 19:49, Pavel Reichl <preichl at redhat.com> wrote:
> On Mon, 2014-06-23 at 19:34 +0200, Jakub Hrozek wrote:
>> Hi,
>>
>> our handling of empty nested groups only worked because one bug was
>> shadowed with another. Please see the attached patch.
>> _______________________________________________
>> sssd-devel mailing list
>> sssd-devel at lists.fedorahosted.org
>> https://lists.fedorahosted.org/mailman/listinfo/sssd-devel
>
>
> Hello,
>
> I do believe that the patch is itself OK (ACK), but in my test
> environment it caused some subsequent problems.
>
> Could we wait with pushing this patch to master until/if my patches*
> make it to master?
>
Sure!
Full disclosure — Pavel showed me his patches while he was looking at the nested groups code. I agree they are important.
Just for the record, can you paste the title(s) of the thread(s) that contain the patches?
> Thanks,
>
> Pavel Reichl
>
> * sssd-devel, subjects:
> LDAP: group_split_members returns incorrectly ENOMEM
> sudo: return after tevent_req_error
>
> _______________________________________________
> sssd-devel mailing list
> sssd-devel at lists.fedorahosted.org
> https://lists.fedorahosted.org/mailman/listinfo/sssd-devel
_______________________________________________
sssd-devel mailing list
sssd-devel at lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/sssd-devel
More information about the sssd-devel
mailing list