Attachment 17050 Details for Bug 14902 – Patch for v4-13-test

[patch] Patch for v4-13-test

tmp413.diff.txt (text/plain), 2.82 KB, created by Stefan Metzmacher on 2021-12-07 10:27:47 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Stefan Metzmacher

Created: 2021-12-07 10:27:47 UTC

Size: 2.82 KB

patch

obsolete

>From 38cc724574d3f7b45e21b06433334f8cb8596f71 Mon Sep 17 00:00:00 2001
>From: Andrew Bartlett <abartlet@samba.org>
>Date: Fri, 12 Nov 2021 12:44:44 +1300
>Subject: [PATCH] dsdb: Use DSDB_SEARCH_SHOW_EXTENDED_DN when searching for the
> local replicated object
>
>This may allow further processing when the DN normalisation has changed
>which changes the indexing, such as seen after fixes for bug 14656.
>
>BUG: https://bugzilla.samba.org/show_bug.cgi?id=14656
>BUG: https://bugzilla.samba.org/show_bug.cgi?id=14902
>
>Signed-off-by: Andrew Bartlett <abartlet@samba.org>
>Reviewed-by: Stefan Metzmacher <metze@samba.org>
>(cherry picked from commit f621317e3b25a8925ab6e448068264488a0a47c7)
>---
> source4/dsdb/samdb/ldb_modules/operational.c    |  2 +-
> source4/dsdb/samdb/ldb_modules/repl_meta_data.c | 13 ++++++++++++-
> 2 files changed, 13 insertions(+), 2 deletions(-)
>
>diff --git a/source4/dsdb/samdb/ldb_modules/operational.c b/source4/dsdb/samdb/ldb_modules/operational.c
>index 5eaebf981417..4e60feaf14f2 100644
>--- a/source4/dsdb/samdb/ldb_modules/operational.c
>+++ b/source4/dsdb/samdb/ldb_modules/operational.c
>@@ -1399,7 +1399,7 @@ static const struct op_attributes_replace search_sub[] = {
> 	{ "tokenGroups", "primaryGroupID", objectSid_attr, construct_token_groups },
> 	{ "tokenGroupsNoGCAcceptable", "primaryGroupID", objectSid_attr, construct_token_groups_no_gc},
> 	{ "tokenGroupsGlobalAndUniversal", "primaryGroupID", objectSid_attr, construct_global_universal_token_groups },
>-	{ "parentGUID", NULL, NULL, construct_parent_guid },
>+	{ "parentGUID", "objectGUID", NULL, construct_parent_guid },
> 	{ "subSchemaSubEntry", NULL, NULL, construct_subschema_subentry },
> 	{ "msDS-isRODC", "objectClass", objectCategory_attr, construct_msds_isrodc },
> 	{ "msDS-KeyVersionNumber", "replPropertyMetaData", NULL, construct_msds_keyversionnumber },
>diff --git a/source4/dsdb/samdb/ldb_modules/repl_meta_data.c b/source4/dsdb/samdb/ldb_modules/repl_meta_data.c
>index 58c04da5f53f..870185ee1d3d 100644
>--- a/source4/dsdb/samdb/ldb_modules/repl_meta_data.c
>+++ b/source4/dsdb/samdb/ldb_modules/repl_meta_data.c
>@@ -6923,7 +6923,18 @@ static int replmd_replicated_apply_next(struct replmd_replicated_request *ar)
> 				   ar->req);
> 	LDB_REQ_SET_LOCATION(search_req);
> 
>-	ret = dsdb_request_add_controls(search_req, DSDB_SEARCH_SHOW_RECYCLED);
>+	/*
>+	 * We set DSDB_SEARCH_SHOW_EXTENDED_DN to get the GUID on the
>+	 * DN.  This in turn helps our operational module find the
>+	 * record by GUID, not DN lookup which is more error prone if
>+	 * DN indexing changes.  We prefer to keep chasing GUIDs
>+	 * around if possible, even within a transaction.
>+	 *
>+	 * The aim here is to keep replication moving and allow a
>+	 * reindex later.
>+	 */
>+	ret = dsdb_request_add_controls(search_req, DSDB_SEARCH_SHOW_RECYCLED
>+					|DSDB_SEARCH_SHOW_EXTENDED_DN);
> 
> 	if (ret != LDB_SUCCESS) {
> 		return ret;
>-- 
>2.25.1
>

Flags:

slow: review+

Actions: View

Attachments on bug 14902: 17048 | 17049 | 17050