Re: [SSSD] [PATCH] Fix memory hierarchy in subdomains discovery

2012-10-11 Thread Sumit Bose
On Wed, Oct 10, 2012 at 10:14:22PM +0200, Jakub Hrozek wrote: The attached patch fixes the user-after-free crash I was seeing occasionally. https://fedorahosted.org/sssd/ticket/1571 The patch changes the subdomains discovery to use the tevent_req style. Previously, the code violated

Re: [SSSD] [PATCH] Fix memory hierarchy in subdomains discovery

2012-10-11 Thread Jakub Hrozek
On Thu, Oct 11, 2012 at 01:35:49PM +0200, Sumit Bose wrote: On Wed, Oct 10, 2012 at 10:14:22PM +0200, Jakub Hrozek wrote: The attached patch fixes the user-after-free crash I was seeing occasionally. https://fedorahosted.org/sssd/ticket/1571 The patch changes the subdomains

[SSSD] [PATCH] Fix memory hierarchy in subdomains discovery

2012-10-10 Thread Jakub Hrozek
The attached patch fixes the user-after-free crash I was seeing occasionally. https://fedorahosted.org/sssd/ticket/1571 The patch changes the subdomains discovery to use the tevent_req style. Previously, the code violated several rules which made the code very unreadable and led to memory