diff options
author | Andreas Schneider <asn@samba.org> | 2013-04-03 15:46:00 +0200 |
---|---|---|
committer | Karolin Seeger <kseeger@samba.org> | 2013-04-11 09:43:02 +0200 |
commit | 6848fb121a3a16b2d87b2bf2f7cca8364a1343f1 (patch) | |
tree | 983dc2f2598a75c13f0ddd872b8b3adc6bbbf1bf /source3 | |
parent | 70107fc911570bbbc1cd613e9c594f5481e5685e (diff) | |
download | samba-6848fb121a3a16b2d87b2bf2f7cca8364a1343f1.tar.gz |
BUG 9699: Fix adding case sensitive spn.
We should be able to define the case of the spn cause it is important
for some services like nfs. 'net ads keytab add "nfs"' should not
result in an uppercase spn.
Signed-off-by: Andreas Schneider <asn@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
Diffstat (limited to 'source3')
-rw-r--r-- | source3/libads/ldap.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/source3/libads/ldap.c b/source3/libads/ldap.c index e62a4b4137d..b841c843c63 100644 --- a/source3/libads/ldap.c +++ b/source3/libads/ldap.c @@ -1920,8 +1920,7 @@ ADS_STATUS ads_add_service_principal_name(ADS_STRUCT *ads, const char *machine_n ads_msgfree(ads, res); return ADS_ERROR(LDAP_NO_MEMORY); } - strupper_m(psp1); - strlower_m(&psp1[strlen(spn)]); + strlower_m(&psp1[strlen(spn) + 1]); servicePrincipalName[0] = psp1; DEBUG(5,("ads_add_service_principal_name: INFO: Adding %s to host %s\n", @@ -1934,8 +1933,7 @@ ADS_STATUS ads_add_service_principal_name(ADS_STRUCT *ads, const char *machine_n ret = ADS_ERROR(LDAP_NO_MEMORY); goto out; } - strupper_m(psp2); - strlower_m(&psp2[strlen(spn)]); + strlower_m(&psp2[strlen(spn) + 1]); servicePrincipalName[1] = psp2; DEBUG(5,("ads_add_service_principal_name: INFO: Adding %s to host %s\n", |