summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAndrew Tridgell <tridge@samba.org>2002-07-31 02:00:30 +0000
committerAndrew Tridgell <tridge@samba.org>2002-07-31 02:00:30 +0000
commit81b69dc79cccd575d33b62ddf7941c5b3c44fdfe (patch)
treeffd24d81ce5af2d78aa63db1491dd88768d256bb
parenta5b6fbdf6086d0cf7772a2abde80a66739f26d11 (diff)
downloadsamba-81b69dc79cccd575d33b62ddf7941c5b3c44fdfe.tar.gz
samba-81b69dc79cccd575d33b62ddf7941c5b3c44fdfe.tar.bz2
samba-81b69dc79cccd575d33b62ddf7941c5b3c44fdfe.zip
the ads_connect() here doesn't need to actually succeed, as its only
needed to find the DC IP. Just don't check its return value! (This used to be commit ab144cd8af1622894d446ce48dde99babeb30bd6)
-rw-r--r--source3/auth/auth_domain.c4
1 files changed, 1 insertions, 3 deletions
diff --git a/source3/auth/auth_domain.c b/source3/auth/auth_domain.c
index 72c216581e..b37ce2cc91 100644
--- a/source3/auth/auth_domain.c
+++ b/source3/auth/auth_domain.c
@@ -52,9 +52,7 @@ static NTSTATUS ads_resolve_dc(fstring remote_machine,
/* a full ads_connect() is actually overkill, as we don't srictly need
to do the SASL auth in order to get the info we need, but libads
doesn't offer a better way right now */
- if (!ADS_ERR_OK(ads_connect(ads))) {
- return NT_STATUS_NO_LOGON_SERVERS;
- }
+ ads_connect(ads);
#endif
fstrcpy(remote_machine, ads->ldap_server_name);