Fixed bug #71249 (ldap_mod_replace/ldap_mod_add store value as string "Array")

10 years ago · 87958927b0
2 changed files with 5 additions and 0 deletions
--- a/4
+++ b/4
@ -11,6 +11,10 @@ PHP                                                                        NEWS
 - CURL:
  . Fixed bug #71225 (curl_setopt() fails to set CURLOPT_POSTFIELDS with
    reference to CURLFile). (Laruence)
+
+- LDAP:
+  . Fixed bug #71249 (ldap_mod_replace/ldap_mod_add store value as string
+    "Array"). (Laruence)
  
 - SPL:
  . Fixed bug #71204 (segfault if clean spl_autoload_funcs while autoloading).
--- a/ext/ldap/ldap.c
+++ b/ext/ldap/ldap.c
@ -1460,6 +1460,7 @@ static void php_ldap_do_modify(INTERNAL_FUNCTION_PARAMETERS, int oper)

 		value = zend_hash_get_current_data(Z_ARRVAL_P(entry));

+		ZVAL_DEREF(value);
 		if (Z_TYPE_P(value) != IS_ARRAY) {
 			num_values = 1;
 		} else {