ext/ldap: Check array key does not have any nul bytes

Author: Girgias

ext/ldap/ldap.c

@@ -2239,6 +2239,11 @@ static void php_ldap_do_modify(INTERNAL_FUNCTION_PARAMETERS, int oper, int ext)
 			RETVAL_FALSE;
 			goto cleanup;
 		}
+		if (zend_str_has_nul_byte(attribute)) {
+			zend_argument_value_error(3, "key must not contain any null bytes");
+			RETVAL_FALSE;
+			goto cleanup;
+		}
 
 		ldap_mods[attribute_index] = emalloc(sizeof(LDAPMod));
 		ldap_mods[attribute_index]->mod_op = oper | LDAP_MOD_BVALUES;

ext/ldap/tests/ldap_add_modify_delete_programming_errors.phpt

@@ -137,9 +137,7 @@ ValueError: ldap_add(): Argument #3 ($entry) must be an associative array of att
 
 Warning: ldap_add(): Add: Can't contact LDAP server in %s on line %d
 bool(false)
-
-Warning: ldap_add(): Add: Can't contact LDAP server in %s on line %d
-bool(false)
+ValueError: ldap_add(): Argument #3 ($entry) key must not contain any null bytes
 Error: Object of class stdClass could not be converted to string
 ValueError: ldap_add(): Argument #3 ($entry) list of attribute values must not be empty
 ValueError: ldap_add(): Argument #3 ($entry) must be a list of attribute values