From: Samuel Cabrero <scabrero@samba.org>
Date: Thu, 6 Feb 2025 14:18:26 +0000 (+0100)
Subject: winbind: Add a check for "winbind varlink service"
X-Git-Tag: tevent-0.17.0~724
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=11591ba54496be67eb095776311dbbe3a72910e7;p=thirdparty%2Fsamba.git

winbind: Add a check for "winbind varlink service"

Print a warning in winbindd startup if the option is enabled
but samba was built without systemd's userdb support.

Signed-off-by: Samuel Cabrero <scabrero@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
---

diff --git a/source3/winbindd/winbindd.c b/source3/winbindd/winbindd.c
index 64891976882..fa150368fc6 100644
--- a/source3/winbindd/winbindd.c
+++ b/source3/winbindd/winbindd.c
@@ -1084,15 +1084,20 @@ static bool winbindd_setup_listeners(void)
 	}
 	tevent_fd_set_auto_close(fde);
 
-#if defined(WITH_SYSTEMD_USERDB)
 	if (lp_winbind_varlink_service()) {
+#if defined(WITH_SYSTEMD_USERDB)
 		/* Setup varlink socket */
 		if (!winbind_setup_varlink(global_event_context(),
 					   global_event_context())) {
 			goto failed;
 		}
-	}
+#else
+		DBG_WARNING("\"winbind varlink service\" is enabled but "
+			    "samba was built without systemd's userdb "
+			    "support. This option will not have any "
+			    "effect\n");
 #endif
+	}
 
 	winbindd_scrub_clients_handler(global_event_context(), NULL,
 				       timeval_current(), NULL);