Attachment 8724 Details for Bug 9699 – v4-0-test patch

[patch] v4-0-test patch

0001-BUG-9699-Fix-adding-case-sensitive-spn.patch (text/plain), 1.58 KB, created by Andreas Schneider on 2013-04-04 07:39:57 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Andreas Schneider

Created: 2013-04-04 07:39:57 UTC

Size: 1.58 KB

patch

obsolete

>From 2646da0efdd58a72fb670d1a02e669088f2643cb Mon Sep 17 00:00:00 2001
>From: Andreas Schneider <asn@samba.org>
>Date: Wed, 3 Apr 2013 15:46:00 +0200
>Subject: [PATCH] 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>
>
>Autobuild-User(master): Jeremy Allison <jra@samba.org>
>Autobuild-Date(master): Wed Apr  3 23:57:32 CEST 2013 on sn-devel-104
>---
> source3/libads/ldap.c | 14 ++------------
> 1 file changed, 2 insertions(+), 12 deletions(-)
>
>diff --git a/source3/libads/ldap.c b/source3/libads/ldap.c
>index ca5962c..2a4a83b 100644
>--- a/source3/libads/ldap.c
>+++ b/source3/libads/ldap.c
>@@ -1949,12 +1949,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);
> 	}
>-	if (!strupper_m(psp1)) {
>-		ret = ADS_ERROR(LDAP_NO_MEMORY);
>-		goto out;
>-	}
>-
>-	if (!strlower_m(&psp1[strlen(spn)])) {
>+	if (!strlower_m(&psp1[strlen(spn) + 1])) {
> 		ret = ADS_ERROR(LDAP_NO_MEMORY);
> 		goto out;
> 	}
>@@ -1970,12 +1965,7 @@ ADS_STATUS ads_add_service_principal_name(ADS_STRUCT *ads, const char *machine_n
> 		ret = ADS_ERROR(LDAP_NO_MEMORY);
> 		goto out;
> 	}
>-	if (!strupper_m(psp2)) {
>-		ret = ADS_ERROR(LDAP_NO_MEMORY);
>-		goto out;
>-	}
>-
>-	if (!strlower_m(&psp2[strlen(spn)])) {
>+	if (!strlower_m(&psp2[strlen(spn) + 1])) {
> 		ret = ADS_ERROR(LDAP_NO_MEMORY);
> 		goto out;
> 	}
>-- 
>1.8.2
>

Flags:

jra: review+

Actions: View

Attachments on bug 9699: 8718 | 8724 | 8725 | 8732 | 8748