From: Xinchen Hui <laruence@gmail.com>
Date: Fri, 1 Jan 2016 15:06:58 +0000 (-0800)
Subject: Fixed bug #71249 (ldap_mod_replace/ldap_mod_add store value as string "Array")
X-Git-Tag: php-7.0.3RC1~90
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=87958927b0cef17ba06a8739c7febca8e920c6de;p=php

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

diff --git a/NEWS b/NEWS
index 372f47cdc3..d1a82fe2ab 100644
--- a/NEWS
+++ b/NEWS
@@ -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).
diff --git a/ext/ldap/ldap.c b/ext/ldap/ldap.c
index e0faf24b16..b1cb6bf65e 100644
--- 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 {