Think have to 0-terminate mod_values strings, got trailing garbage,

this fixes bug #5001 I think.

diff --git a/ext/ldap/ldap.c b/ext/ldap/ldap.c
index 0149f1e9e7cf38fa937c15a76165213df821b3b9..3aa8cf5273351a99b999c5fa686d3089aac973d8 100644 (file)
--- a/ext/ldap/ldap.c
+++ b/ext/ldap/ldap.c
@@ -1151,6 +1151,7 @@ static void php_ldap_do_modify(INTERNAL_FUNCTION_PARAMETERS, int oper)
                 if ((num_values == 1) && ((*value)->type != IS_ARRAY)) {
                        convert_to_string_ex(value);
                        ldap_mods[i]->mod_values[0] = (*value)->value.str.val;
+                       ldap_mods[i]->mod_values[0][(*value)->value.str.len] = '\0';
                } else {        
                        for(j=0; j<num_values; j++) {
                                zend_hash_index_find((*value)->value.ht,j, (void **) &ivalue);