Not using _free_ldap_result destructor for le_result_entry, leads to

crashes due to freeing same memory several times in some cases

diff --git a/ext/ldap/ldap.c b/ext/ldap/ldap.c
index 696646e1b5ccad9b7359dcbbb7409a5362181f0a..8562b527532763793a967a5cc10a1cd20be524c7 100644 (file)
--- a/ext/ldap/ldap.c
+++ b/ext/ldap/ldap.c
@@ -207,7 +207,7 @@ PHP_MINIT_FUNCTION(ldap)
 
        le_result = zend_register_list_destructors_ex(_free_ldap_result, NULL, "ldap result", module_number);
        le_link = zend_register_list_destructors_ex(_close_ldap_link, NULL, "ldap link", module_number);
-       le_result_entry = zend_register_list_destructors_ex(_free_ldap_result, NULL, "ldap result entry", module_number);
+       le_result_entry = zend_register_list_destructors_ex(NULL, NULL, "ldap result entry", module_number);
 
        ldap_module_entry.type = type;