diff options
author | Stefan Metzmacher <metze@samba.org> | 2023-02-16 16:31:34 +0100 |
---|---|---|
committer | Jule Anger <janger@samba.org> | 2023-04-05 10:40:13 +0000 |
commit | 5c68985b2788330026df687d5439f4e86f90a77c (patch) | |
tree | 0ecabeabcb5f5f0b488c42dec31b5bc919a8047a | |
parent | 5db0223764d392e17df40a6519ea8516e94a94ad (diff) | |
download | samba-5c68985b2788330026df687d5439f4e86f90a77c.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 8cb59b2f752..d053977033f 100644 --- a/source3/winbindd/winbindd_dual.c +++ b/source3/winbindd/winbindd_dual.c @@ -1811,13 +1811,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()) { |