Bug #71249 [Opn]: 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
 Updated by:         [email protected]
 Reported by:        post at rolandgruber dot de
 Summary:            ldap_mod_replace/ldap_mod_add store value as string
                     "Array"
 Status:             Open
 Type:               Bug
 Package:            LDAP related
 Operating System:   Linux
 PHP Version:        7.0.1
 Block user comment: N
 Private report:     N

 New Comment:

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


Previous Comments:
------------------------------------------------------------------------
[2015-12-31 02:27:08] [email protected]

mistake set... restore.

------------------------------------------------------------------------
[2015-12-31 02:26:52] [email protected]

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