[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