diff options
| author | Garming Sam <garming@catalyst.net.nz> | 2016-10-21 15:50:09 +1300 |
|---|---|---|
| committer | Karolin Seeger <kseeger@samba.org> | 2016-11-30 12:19:33 +0100 |
| commit | 6251b7938db4ce3b4214cd862ed5ff2a1e8e239b (patch) | |
| tree | 8426171640d3b9ba4d07c33a0389475b794a146e | |
| parent | b5d5d85d5160ac82b24d73e7cd1e757c0bfd7835 (diff) | |
| download | samba-6251b7938db4ce3b4214cd862ed5ff2a1e8e239b.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
(cherry picked from commit ef7e46d68a6596be6e904caaa04e917c576dd9d3)
| -rw-r--r-- | selftest/knownfail | 1 | ||||
| -rw-r--r-- | source4/dsdb/kcc/garbage_collect_tombstones.c | 5 |
2 files changed, 3 insertions, 3 deletions
diff --git a/selftest/knownfail b/selftest/knownfail index 1b543b97a87..c1899da9053 100644 --- a/selftest/knownfail +++ b/selftest/knownfail @@ -292,4 +292,3 @@ #ntvfs server blocks copychunk with execute access on read handle ^samba4.smb2.ioctl.copy_chunk_bad_access ^samba4.drs.getnc_exop.python.*getnc_exop.DrsReplicaPrefixMapTestCase.test_regular_prefix_map_ex_attid.* -^samba4.blackbox.tombstones-expunge.release-4-5-0-pre1.tombstones_expunge 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); |