[PATCH] Plug two leaks in python-ldap

Michael Ströder michael at stroeder.com
Tue Apr 4 12:28:37 CEST 2006


Xin LI wrote:
> 
> 12:16 +0200,Michael Ströder:
> 
>>Thanks for submitting the patch.
>>Do you have Python code demonstrating the leak?
> 
> By searching a non-existent entry DN from LDAP server, it would end up
> with the error path and got the leak.
> 
>>Do you add the ldap_msgfree(msg) because LDAPmessage_to_python() is not
>>called in these error cases?
> 
> Yes.  It takes me some minutes to figure out that the ldap message was
> actually free'ed by LDAPmessage_to_python() in the usual path :-)

Ok, will commit your patch ASAP.

Ciao, Michael.



More information about the python-ldap mailing list