Don't modify arrays passed by value

Author: cmb69

ext/ldap/ldap.c

@@ -1480,7 +1480,7 @@ static void php_ldap_do_search(INTERNAL_FUNCTION_PARAMETERS, int scope)
 	int old_ldap_sizelimit = -1, old_ldap_timelimit = -1, old_ldap_deref = -1;
 	int num_attribs = 0, ret = 1, i, errno, argcount = ZEND_NUM_ARGS();
 
-	if (zend_parse_parameters(argcount, "zzz|alllla/", &link, &base_dn, &filter, &attrs, &attrsonly,
+	if (zend_parse_parameters(argcount, "zzz|a/lllla/", &link, &base_dn, &filter, &attrs, &attrsonly,
 		&sizelimit, &timelimit, &deref, &serverctrls) == FAILURE) {
 		return;
 	}

ext/ldap/tests/ldap_read_variation1.phpt

@@ -0,0 +1,21 @@
+--TEST--
+ldap_read() does not modify $attributes array
+--SKIPIF--
+<?php
+if (!extension_loaded('ldap')) die('ldap extension not available');
+?>
+--FILE--
+<?php
+$array = [123, 456, 789];
+@ldap_read(null, null, null, $array);
+var_dump($array);
+?>
+--EXPECT--
+array(3) {
+  [0]=>
+  int(123)
+  [1]=>
+  int(456)
+  [2]=>
+  int(789)
+}