diff options
author | Jeremy Allison <jra@samba.org> | 2003-10-22 23:38:20 +0000 |
---|---|---|
committer | Jeremy Allison <jra@samba.org> | 2003-10-22 23:38:20 +0000 |
commit | bb0598faf58679a7ad26a1caab8eadb154a07ae2 (patch) | |
tree | beee6b610df4ad7660cb174c86773bd9c747682c /source3/libads/ldap.c | |
parent | 3531647574ba61ce69969c501fbc82937a4d4dfa (diff) | |
download | samba-bb0598faf58679a7ad26a1caab8eadb154a07ae2.tar.gz samba-bb0598faf58679a7ad26a1caab8eadb154a07ae2.tar.bz2 samba-bb0598faf58679a7ad26a1caab8eadb154a07ae2.zip |
Put strcasecmp/strncasecmp on the banned list (except for needed calls
in iconv.c and nsswitch/). Using them means you're not thinking about multibyte at
all and I really want to discourage that.
Jeremy.
(This used to be commit d7e35dfb9283d560d0ed2ab231f36ed92767dace)
Diffstat (limited to 'source3/libads/ldap.c')
-rw-r--r-- | source3/libads/ldap.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/source3/libads/ldap.c b/source3/libads/ldap.c index 8c3185ea5e..b3706cb240 100644 --- a/source3/libads/ldap.c +++ b/source3/libads/ldap.c @@ -972,7 +972,7 @@ ADS_STATUS ads_del_dn(ADS_STRUCT *ads, char *del_dn) **/ char *ads_ou_string(const char *org_unit) { - if (!org_unit || !*org_unit || strcasecmp(org_unit, "Computers") == 0) { + if (!org_unit || !*org_unit || strequal(org_unit, "Computers")) { return strdup("cn=Computers"); } @@ -1970,8 +1970,8 @@ ADS_STATUS ads_workgroup_name(ADS_STRUCT *ads, TALLOC_CTX *mem_ctx, char **workg prefix_length = strlen(prefix); for (i=0;principles[i]; i++) { - if (strncasecmp(principles[i], prefix, prefix_length) == 0 && - strcasecmp(ads->config.realm, principles[i]+prefix_length) != 0 && + if (strnequal(principles[i], prefix, prefix_length) && + !strequal(ads->config.realm, principles[i]+prefix_length) && !strchr(principles[i]+prefix_length, '.')) { /* found an alternate (short) name for the domain. */ DEBUG(3,("Found alternate name '%s' for realm '%s'\n", |