diff options
| author | Yasuma Takeda <yasuma@osstech.co.jp> | 2009-02-11 14:10:21 -0800 |
|---|---|---|
| committer | Jeremy Allison <jra@samba.org> | 2009-02-11 14:10:21 -0800 |
| commit | 58331a118dd6a7fb56e70afe6cf93ef7cfff7e81 (patch) | |
| tree | c5ef1981fbbb80af6b6add428114f3783a437306 | |
| parent | db4a435d235bedf48d668a0f4418dd46f38044ed (diff) | |
| download | samba-58331a118dd6a7fb56e70afe6cf93ef7cfff7e81.tar.gz samba-58331a118dd6a7fb56e70afe6cf93ef7cfff7e81.tar.xz samba-58331a118dd6a7fb56e70afe6cf93ef7cfff7e81.zip | |
Fix bug #6098 - When the DNS server is invalid, the ads_find_dc() does not work correctly with "security = domain"
1. If DNS server is invalid, the get_sorted_dc_list() is called with
realm(FQDN) and it fails.
2. On the next step, the get_sorted_dc_list() is called with realm(FQDN) again.
I think "again" is wrong place.
On the 2nd step, get_sorted_dc_list() should be called with realm(WORKGROUP).
| -rw-r--r-- | source/libads/ldap.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/source/libads/ldap.c b/source/libads/ldap.c index 36e86936180..65f13cd7810 100644 --- a/source/libads/ldap.c +++ b/source/libads/ldap.c @@ -275,6 +275,8 @@ static NTSTATUS ads_find_dc(ADS_STRUCT *ads) if (c_realm && *c_realm) got_realm = True; + again: + /* we need to try once with the realm name and fallback to the netbios domain name if we fail (if netbios has not been disabled */ @@ -326,8 +328,6 @@ static NTSTATUS ads_find_dc(ADS_STRUCT *ads) sitename = sitename_fetch(realm); - again: - DEBUG(6,("ads_find_dc: (cldap) looking for %s '%s'\n", (got_realm ? "realm" : "domain"), realm)); |