[PATCH] Plug two leaks in python-ldap

Xin LI delphij at delphij.net
Tue Apr 4 12:22:27 CEST 2006


Hi, Michael,

在 2006-04-04二的 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 :-)

> > BTW.  Is there any plan to release a new version soon?
> 
> I could release a new version very soon. (Despite that SF sucks and
> again I can't contact the CVS repository via SSH at the moment.)
> 
> Note that as already stated in CVS version of CHANGES upcoming
> python-ldap 2.2.0 will require OpenLDAP libs 2.2.x or later for the build.

Glad to hear that :-)  Thanks!

Cheers,
-- 
Xin LI <delphij delphij net>    http://www.delphij.net/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: ???????????
URL: <http://mail.python.org/pipermail/python-ldap/attachments/20060404/7a5f824d/attachment.pgp>


More information about the python-ldap mailing list