diff options
author | Stefan Metzmacher <metze@samba.org> | 2023-02-16 16:31:34 +0100 |
---|---|---|
committer | Jule Anger <janger@samba.org> | 2023-03-30 15:10:10 +0000 |
commit | 148d5ad7698d154731dfa092b0c6314c71f38086 (patch) | |
tree | fe3ba017d6d0ba378ca56016f42be493f9c97d80 | |
parent | cb204cfc69bbc9a850b3c8783465d295f3a3d99f (diff) | |
download | samba-148d5ad7698d154731dfa092b0c6314c71f38086.tar.gz |
winbindd: don't call set_domain_online_request() in the idmap child
Most idmap backends don't need access to the domain controllers.
And the related code is not needed for the backends.
Commit 17c86a2c5a5a5e2b194362e5f36f0f99910222c5 changed
the logic of set_domain_online_request() completely!
Instead of triggering a dc probe in the background,
it is now doing a blocking connection.
And doing this in the idmap child is completely useless.
BUG: https://bugzilla.samba.org/show_bug.cgi?id=15317
Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
(cherry picked from commit ad242a20643c930eb00a8b700f7bd9638f8821a8)
-rw-r--r-- | source3/winbindd/winbindd_dual.c | 7 |
1 files changed, 0 insertions, 7 deletions
diff --git a/source3/winbindd/winbindd_dual.c b/source3/winbindd/winbindd_dual.c index fe855045fa8..3ab3e63dc4c 100644 --- a/source3/winbindd/winbindd_dual.c +++ b/source3/winbindd/winbindd_dual.c @@ -1810,13 +1810,6 @@ static bool fork_domain_child(struct winbindd_child *child) } } - /* - * We are in idmap child, bring primary domain online. - */ - if (is_idmap_child(child)) { - set_domain_online_request(primary_domain); - } - /* We might be in the idmap child...*/ if (child->domain && !(child->domain->internal) && lp_winbind_offline_logon()) { |