Re: [Freeipa-devel] [PATCH 0050] Fix memory leak in configuration with multiple LDAP instances

2012-09-05 Thread Petr Spacek
On 09/05/2012 01:02 PM, Adam Tkac wrote: On Tue, Aug 14, 2012 at 04:00:21PM +0200, Petr Spacek wrote: Hello, this patch fixes $SUBJ$. Adam, please double-check correctness of this change. I had two assumptions: - all locking is done inside dns_db_(un)register() functions - LDAP instances are

Re: [Freeipa-devel] [PATCH 0050] Fix memory leak in configuration with multiple LDAP instances

2012-09-05 Thread Adam Tkac
On Tue, Aug 14, 2012 at 04:00:21PM +0200, Petr Spacek wrote: > Hello, > > this patch fixes $SUBJ$. > > Adam, please double-check correctness of this change. > > I had two assumptions: > - all locking is done inside dns_db_(un)register() functions > - LDAP instances are decommissioned before dyna