Re: [PATCH] Plug two leaks in python-ldap

[<mi...@st...>]

Xin LI wrote:
>=20
> 12:16 +0200=EF=BC=8CMichael Str=C3=B6der=EF=BC=9A
>=20
>>Thanks for submitting the patch.
>>Do you have Python code demonstrating the leak?
>=20
> By searching a non-existent entry DN from LDAP server, it would end up
> with the error path and got the leak.
>=20
>>Do you add the ldap_msgfree(msg) because LDAPmessage_to_python() is not
>>called in these error cases?
>=20
> 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.