diff options
| author | Andrew Tridgell <tridge@samba.org> | 2004-12-06 06:45:51 +0000 |
|---|---|---|
| committer | Gerald (Jerry) Carter <jerry@samba.org> | 2007-10-10 13:06:22 -0500 |
| commit | a42dba94d0d6d8203d845f771817d4e48b8c5205 (patch) | |
| tree | 112ad4bd4338c28eadc2283979b07d9e60a650bb /source4/lib/ldb/ldb_tdb | |
| parent | f99c93ec57691a393b4ae5ba57176b98f33efc17 (diff) | |
| download | samba-a42dba94d0d6d8203d845f771817d4e48b8c5205.tar.gz samba-a42dba94d0d6d8203d845f771817d4e48b8c5205.tar.bz2 samba-a42dba94d0d6d8203d845f771817d4e48b8c5205.zip | |
r4071: - ldap does allow adding additional attribute values with a modify
operation, but not if the value already exists
- fixed syntax of test.ldif for ldap backend
(This used to be commit 29225d0bec39038e42e68849bd9378898f062081)
Diffstat (limited to 'source4/lib/ldb/ldb_tdb')
| -rw-r--r-- | source4/lib/ldb/ldb_tdb/ldb_tdb.c | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/source4/lib/ldb/ldb_tdb/ldb_tdb.c b/source4/lib/ldb/ldb_tdb/ldb_tdb.c index 4049900a23..6623fd0052 100644 --- a/source4/lib/ldb/ldb_tdb/ldb_tdb.c +++ b/source4/lib/ldb/ldb_tdb/ldb_tdb.c @@ -516,8 +516,13 @@ int ltdb_modify_internal(struct ldb_module *module, const struct ldb_message *ms already exists */ ret = find_element(&msg2, msg->elements[i].name); if (ret != -1) { - ltdb->last_err_string = "Attribute exists"; - goto failed; + for (j=0;j<msg->elements[i].num_values;j++) { + if (ldb_msg_find_val(&msg2.elements[ret], + &msg->elements[i].values[j])) { + ltdb->last_err_string = "Type or value exists"; + goto failed; + } + } } if (msg_add_element(ldb, &msg2, &msg->elements[i]) != 0) { goto failed; |