diff options
| author | Andrew Bartlett <abartlet@samba.org> | 2002-09-06 11:46:59 +0000 |
|---|---|---|
| committer | Andrew Bartlett <abartlet@samba.org> | 2002-09-06 11:46:59 +0000 |
| commit | eec38ee3bb4bceeaa82abf8df1cce92b6a5781ce (patch) | |
| tree | 505c354bdb503f732dce311a8e823af2415efe38 /source3/libads/ldap.c | |
| parent | c4d1f9a1e8961c051eb1cee2aa9d93b1b120d836 (diff) | |
| download | samba-eec38ee3bb4bceeaa82abf8df1cce92b6a5781ce.tar.gz samba-eec38ee3bb4bceeaa82abf8df1cce92b6a5781ce.tar.bz2 samba-eec38ee3bb4bceeaa82abf8df1cce92b6a5781ce.zip | |
Patch from "Stefan (metze) Metzmacher" <metze@metzemix.de>
to extend the ADS_STATUS system to include NTSTATUS, and to provide a better
general infrustructure for his sam_ads work.
I've also added some extra failure mode DEBUG()s to parts of the code.
NOTE: The ADS_ERR_OK() macro is rather sensitive to braketing issues - without
the final set of brakets, the test is essentially inverted - causing some
intersting 'error = success' messages...
Andrew Bartlett
(This used to be commit 5b9a7ab901bc311f3ad08462a8a68d133c34a8b4)
Diffstat (limited to 'source3/libads/ldap.c')
| -rw-r--r-- | source3/libads/ldap.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/source3/libads/ldap.c b/source3/libads/ldap.c index 428fc02065..2f70d3a285 100644 --- a/source3/libads/ldap.c +++ b/source3/libads/ldap.c @@ -1648,6 +1648,7 @@ ADS_STATUS ads_server_info(ADS_STRUCT *ads) if (!p) { ldap_value_free(values); ldap_msgfree(res); + DEBUG(1, ("ads_server_info: returned ldap server name did not contain a ':' so was deemed invalid\n")); return ADS_ERROR(LDAP_DECODING_ERROR); } @@ -1659,6 +1660,7 @@ ADS_STATUS ads_server_info(ADS_STRUCT *ads) ldap_value_free(values); ldap_msgfree(res); SAFE_FREE(ads->config.ldap_server_name); + DEBUG(1, ("ads_server_info: returned ldap server name did not contain '$@' so was deemed invalid\n")); return ADS_ERROR(LDAP_DECODING_ERROR); } |