Bug #71249 [Fbk->Asn]: ldap_mod_replace/ldap_mod_add store value as string "Array"

Edit report at https://bugs.php.net/bug.php?id=71249&edit=1 ID: 71249 User updated by: post at rolandgruber dot de Reported by: post at rolandgruber dot de Summary: ldap_mod_replace/ldap_mod_add store value as string "Array" -Status: Feedback +Status: Assigned Type: Bug Package: LDAP related Operating System: Linux PHP Version: 7.0.1 Assigned To: laruence Block user comment: N Private report: N New Comment: I added the line and now it works. :) The attribute gets the correct value. Please add this to the next release. Thanks a lot for your fast help and have a great start into 2016! Previous Comments: ------------------------------------------------------------------------ [2015-12-31 02:28:36] laruence@php.net could you please try with the following quick fix? diff --git a/ext/ldap/ldap.c b/ext/ldap/ldap.c index e0faf24..b1cb6bf 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 { thanks ------------------------------------------------------------------------ [2015-12-31 02:27:08] laruence@php.net mistake set... restore. ------------------------------------------------------------------------ [2015-12-31 02:26:52] laruence@php.net Thank you for this bug report. To properly diagnose the problem, we need a short but complete example script to be able to reproduce this bug ourselves. A proper reproducing script starts with <?php and ends with ?>, is max. 10-20 lines long and does not require any external resources such as databases, etc. If the script requires a database to demonstrate the issue, please make sure it creates all necessary tables, stored procedures etc. Please avoid embedding huge scripts into the report. ------------------------------------------------------------------------ [2015-12-30 20:11:28] post at rolandgruber dot de Description: ------------ The following parameter $entry results in storing the plain string "Array" in LDAP: Array ( [initials] => Array ( [0] => 11 ) ) Looks like the functions no longer check the type of the array values. The same call in PHP 5.6 stores the correct value "11". Expected result: ---------------- Instead of setting LDAP attribute to "Array" the value should be "11". Actual result: -------------- LDAP attribute gets value "Array" ------------------------------------------------------------------------ -- Edit this bug report at https://bugs.php.net/bug.php?id=71249&edit=1