diff options
author | Andrew Tridgell <tridge@samba.org> | 2002-02-07 02:44:37 +0000 |
---|---|---|
committer | Andrew Tridgell <tridge@samba.org> | 2002-02-07 02:44:37 +0000 |
commit | 24c7e7a3849df3a3378f7e7f20099de048f0b7bd (patch) | |
tree | 9030a8dd782933583102e5461beeb881ed626f21 /source/libads | |
parent | fd4d3496e142574e2bd25faab5c8f1534a21558f (diff) | |
download | samba-24c7e7a3849df3a3378f7e7f20099de048f0b7bd.tar.gz samba-24c7e7a3849df3a3378f7e7f20099de048f0b7bd.tar.xz samba-24c7e7a3849df3a3378f7e7f20099de048f0b7bd.zip |
when a trusted domain is down an ADS server will return a success on a
get trusted domains query but leave the domain SID blank - we need to
fail the add of the trusted domain in winbindd in that case
Diffstat (limited to 'source/libads')
-rw-r--r-- | source/libads/ldap.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/source/libads/ldap.c b/source/libads/ldap.c index 4567a42c0fc..a13bda5fa9f 100644 --- a/source/libads/ldap.c +++ b/source/libads/ldap.c @@ -855,8 +855,9 @@ ADS_STATUS ads_trusted_domains(ADS_STRUCT *ads, TALLOC_CTX *mem_ctx, for (i=0, msg = ads_first_entry(ads, res); msg; msg = ads_next_entry(ads, msg)) { (*names)[i] = ads_pull_string(ads, mem_ctx, msg, "flatName"); - ads_pull_sid(ads, msg, "securityIdentifier", &(*sids)[i]); - i++; + if (ads_pull_sid(ads, msg, "securityIdentifier", &(*sids)[i])) { + i++; + } } ads_msgfree(ads, res); |