Loop control variable was not being incremented. I also converted a goto loop into a do...while loop to make it easier to follow the logic.
This should fix https://fedorahosted.org/sssd/ticket/211
I tested with a single LOCAL domain, specified as in the ticket, and also with a second, LDAP domain to verify that groups were still enumerated for further domains.
On Tue, 2009-09-29 at 07:37 -0400, Stephen Gallagher wrote:
Loop control variable was not being incremented. I also converted a goto loop into a do...while loop to make it easier to follow the logic.
This should fix https://fedorahosted.org/sssd/ticket/211
I tested with a single LOCAL domain, specified as in the ticket, and also with a second, LDAP domain to verify that groups were still enumerated for further domains.
ACK.
Simo.
-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1
On 09/29/2009 07:42 AM, Simo Sorce wrote:
On Tue, 2009-09-29 at 07:37 -0400, Stephen Gallagher wrote:
Loop control variable was not being incremented. I also converted a goto loop into a do...while loop to make it easier to follow the logic.
This should fix https://fedorahosted.org/sssd/ticket/211
I tested with a single LOCAL domain, specified as in the ticket, and also with a second, LDAP domain to verify that groups were still enumerated for further domains.
ACK.
Simo.
sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://fedorahosted.org/mailman/listinfo/sssd-devel
Pushed to master.
- -- Stephen Gallagher RHCE 804006346421761
Looking to carve out IT costs? www.redhat.com/carveoutcosts/
On Tue, Sep 29, 2009 at 4:37 AM, Stephen Gallagher sgallagh@redhat.com wrote:
Loop control variable was not being incremented. I also converted a goto loop into a do...while loop to make it easier to follow the logic.
Will this be in a 0.6.1 release or should we just cherry pick fixes like this into 0.6 and roll packages ourselves?
This should fix https://fedorahosted.org/sssd/ticket/211
I tested with a single LOCAL domain, specified as in the ticket, and also with a second, LDAP domain to verify that groups were still enumerated for further domains. -- Stephen Gallagher RHCE 804006346421761
Looking to carve out IT costs? www.redhat.com/carveoutcosts/
sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://fedorahosted.org/mailman/listinfo/sssd-devel
-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1
On 09/29/2009 08:09 AM, Jeff Schroeder wrote:
On Tue, Sep 29, 2009 at 4:37 AM, Stephen Gallagher sgallagh@redhat.com wrote:
Loop control variable was not being incremented. I also converted a goto loop into a do...while loop to make it easier to follow the logic.
Will this be in a 0.6.1 release or should we just cherry pick fixes like this into 0.6 and roll packages ourselves?
This should fix https://fedorahosted.org/sssd/ticket/211
I tested with a single LOCAL domain, specified as in the ticket, and also with a second, LDAP domain to verify that groups were still enumerated for further domains. -- Stephen Gallagher RHCE 804006346421761
Looking to carve out IT costs? www.redhat.com/carveoutcosts/
sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://fedorahosted.org/mailman/listinfo/sssd-devel
This will be in a 0.6.1 release soon, but I'm also cherry-picking it into the 0.6.0 RPM that I'm building for Fedora 12 (and will shortly backport it into Fedora 11)
If you're building your own RPMs, you may want to cherry-pick it for now.
- -- Stephen Gallagher RHCE 804006346421761
Looking to carve out IT costs? www.redhat.com/carveoutcosts/
On Tue, Sep 29, 2009 at 5:11 AM, Stephen Gallagher sgallagh@redhat.com wrote:
On 09/29/2009 08:09 AM, Jeff Schroeder wrote:
On Tue, Sep 29, 2009 at 4:37 AM, Stephen Gallagher sgallagh@redhat.com wrote:
Loop control variable was not being incremented. I also converted a goto loop into a do...while loop to make it easier to follow the logic.
Will this be in a 0.6.1 release or should we just cherry pick fixes like this into 0.6 and roll packages ourselves?
This should fix https://fedorahosted.org/sssd/ticket/211
I tested with a single LOCAL domain, specified as in the ticket, and also with a second, LDAP domain to verify that groups were still enumerated for further domains. -- Stephen Gallagher RHCE 804006346421761
Looking to carve out IT costs? www.redhat.com/carveoutcosts/
This will be in a 0.6.1 release soon, but I'm also cherry-picking it into the 0.6.0 RPM that I'm building for Fedora 12 (and will shortly backport it into Fedora 11)
If you're building your own RPMs, you may want to cherry-pick it for now.
Waiting for Fedora 12 blessed rpms and then rebuilding them for F10 is fine.
Stephen Gallagher RHCE 804006346421761
Looking to carve out IT costs? www.redhat.com/carveoutcosts/
sssd-devel@lists.fedorahosted.org