diff options
author | Garming Sam <garming@catalyst.net.nz> | 2016-10-21 15:50:09 +1300 |
---|---|---|
committer | Andrew Bartlett <abartlet@samba.org> | 2016-11-03 01:46:43 +0100 |
commit | ef7e46d68a6596be6e904caaa04e917c576dd9d3 (patch) | |
tree | e758aeaa5762eb55017e092857f17f253174917c /source4/dsdb | |
parent | dba624364cde1c885640386c4e2bca17e9d5297c (diff) | |
download | samba-ef7e46d68a6596be6e904caaa04e917c576dd9d3.tar.gz |
collect_tombstones: Allow links to recycled objects to be deleted
The reason we choose to provide the string DN is because extended_dn_in
will try to correct the <GUID=...> by searching on it (despite the fact
it does not exist and then failing on a ldb_dn_validate in
objectclass_attrs).
We can now also remove the dangling link test from the knownfail.
Signed-off-by: Garming Sam <garming@catalyst.net.nz>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
BUG: https://bugzilla.samba.org/show_bug.cgi?id=12385
Autobuild-User(master): Andrew Bartlett <abartlet@samba.org>
Autobuild-Date(master): Thu Nov 3 01:46:43 CET 2016 on sn-devel-144
Diffstat (limited to 'source4/dsdb')
-rw-r--r-- | source4/dsdb/kcc/garbage_collect_tombstones.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/source4/dsdb/kcc/garbage_collect_tombstones.c b/source4/dsdb/kcc/garbage_collect_tombstones.c index ad14d5e55bb..1909cfeff76 100644 --- a/source4/dsdb/kcc/garbage_collect_tombstones.c +++ b/source4/dsdb/kcc/garbage_collect_tombstones.c @@ -193,8 +193,9 @@ static NTSTATUS garbage_collect_tombstones_part(TALLOC_CTX *mem_ctx, guid_buf_str = GUID_buf_string(&guid, &buf_guid); guid_search_str = talloc_asprintf(mem_ctx, - "<GUID=%s>", - guid_buf_str); + "<GUID=%s>;%s", + guid_buf_str, + dsdb_dn_get_linearized(mem_ctx, dn)); cleanup_val = data_blob_string_const(guid_search_str); talloc_free(dn); |