From 6251b7938db4ce3b4214cd862ed5ff2a1e8e239b Mon Sep 17 00:00:00 2001 From: Garming Sam Date: Fri, 21 Oct 2016 15:50:09 +1300 Subject: 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 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 Reviewed-by: Andrew Bartlett BUG: https://bugzilla.samba.org/show_bug.cgi?id=12385 Autobuild-User(master): Andrew Bartlett Autobuild-Date(master): Thu Nov 3 01:46:43 CET 2016 on sn-devel-144 (cherry picked from commit ef7e46d68a6596be6e904caaa04e917c576dd9d3) --- selftest/knownfail | 1 - 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_buf_str); + ";%s", + guid_buf_str, + dsdb_dn_get_linearized(mem_ctx, dn)); cleanup_val = data_blob_string_const(guid_search_str); talloc_free(dn); -- cgit v1.2.1