Attachment 8748 Details for Bug 9699 – v3-6-test patch

[patch] v3-6-test patch

0001-BUG-9699-Fix-adding-case-sensitive-spn.patch (text/plain), 1.41 KB, created by Andreas Schneider on 2013-04-10 08:03:53 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Andreas Schneider

Created: 2013-04-10 08:03:53 UTC

Size: 1.41 KB

patch

obsolete

>From 00e3ac8a4ef72049cb0b684643a3daec4f61f5b9 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>
>---
> source3/libads/ldap.c | 6 ++----
> 1 file changed, 2 insertions(+), 4 deletions(-)
>
>diff --git a/source3/libads/ldap.c b/source3/libads/ldap.c
>index e62a4b4..b841c84 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", 
>-- 
>1.8.2
>

Flags:

jra: review+

Actions: View

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