Attachment 10644 Details for Bug 11059 – patchset for v4-0-test branch, slightly differs from master (extra cli_init_creds() call)

[patch] patchset for v4-0-test branch, slightly differs from master (extra cli_init_creds() call)

bso11059_dfs_ref_workgroup_40t.patchset (text/plain), 3.63 KB, created by David Disseldorp on 2015-01-21 13:41:38 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: David Disseldorp

Created: 2015-01-21 13:41:38 UTC

Size: 3.63 KB

patch

obsolete

>From 06e7bb99dcd2b88ed2ada704c99740282a956cb1 Mon Sep 17 00:00:00 2001
>From: David Disseldorp <ddiss@samba.org>
>Date: Fri, 16 Jan 2015 16:21:23 +0100
>Subject: [PATCH 1/2] libsmb: provide authinfo domain for DFS referral auth
>
>libsmbclient uses the smbc_init->smbc_get_auth_data_fn() provided
>workgroup/domain in initial connections, but then switches to the
>default smb.conf workgroup/domain when handling DFS referrals.
>
>Bug: https://bugzilla.samba.org/show_bug.cgi?id=11059
>
>Signed-off-by: David Disseldorp <ddiss@samba.org>
>Reviewed-by: Jeremy Allison <jra@samba.org>
>(cherry picked from commit 6c9de0cd056afc0b478c02f1bdb0e06532388037)
>[ddiss@samba.org: 4.0 rebase with cli_init_creds() call]
>---
> source3/libsmb/clidfs.c | 13 +++++++++----
> 1 file changed, 9 insertions(+), 4 deletions(-)
>
>diff --git a/source3/libsmb/clidfs.c b/source3/libsmb/clidfs.c
>index b2e2e9e..7a2beac 100644
>--- a/source3/libsmb/clidfs.c
>+++ b/source3/libsmb/clidfs.c
>@@ -95,6 +95,7 @@ static NTSTATUS do_connect(TALLOC_CTX *ctx,
> 	char *newserver, *newshare;
> 	const char *username;
> 	const char *password;
>+	const char *domain;
> 	NTSTATUS status;
> 	int flags = 0;
> 
>@@ -162,11 +163,15 @@ static NTSTATUS do_connect(TALLOC_CTX *ctx,
> 
> 	username = get_cmdline_auth_info_username(auth_info);
> 	password = get_cmdline_auth_info_password(auth_info);
>+	domain = get_cmdline_auth_info_domain(auth_info);
>+	if ((domain == NULL) || (domain[0] == '\0')) {
>+		domain = lp_workgroup();
>+	}
> 
> 	status = cli_session_setup(c, username,
> 				   password, strlen(password),
> 				   password, strlen(password),
>-				   lp_workgroup());
>+				   domain);
> 	if (!NT_STATUS_IS_OK(status)) {
> 		/* If a password was not supplied then
> 		 * try again with a null username. */
>@@ -187,7 +192,7 @@ static NTSTATUS do_connect(TALLOC_CTX *ctx,
> 		d_printf("Anonymous login successful\n");
> 		status = cli_init_creds(c, "", lp_workgroup(), "");
> 	} else {
>-		status = cli_init_creds(c, username, lp_workgroup(), password);
>+		status = cli_init_creds(c, username, domain, password);
> 	}
> 
> 	if (!NT_STATUS_IS_OK(status)) {
>@@ -218,7 +223,7 @@ static NTSTATUS do_connect(TALLOC_CTX *ctx,
> 				force_encrypt,
> 				username,
> 				password,
>-				lp_workgroup())) {
>+				domain)) {
> 		cli_shutdown(c);
> 		return do_connect(ctx, newserver,
> 				newshare, auth_info, false,
>@@ -240,7 +245,7 @@ static NTSTATUS do_connect(TALLOC_CTX *ctx,
> 		status = cli_cm_force_encryption(c,
> 					username,
> 					password,
>-					lp_workgroup(),
>+					domain,
> 					sharename);
> 		if (!NT_STATUS_IS_OK(status)) {
> 			cli_shutdown(c);
>-- 
>2.1.2
>
>
>From 9bfb0c56ce2bcb0f5ccb977ceafc64784f57ee13 Mon Sep 17 00:00:00 2001
>From: David Disseldorp <ddiss@samba.org>
>Date: Mon, 19 Jan 2015 13:39:35 +0100
>Subject: [PATCH 2/2] libsmb: provide authinfo domain for encrypted session
> referrals
>
>6c9de0cd056afc0b478c02f1bdb0e06532388037 requires this extra change.
>
>Bug: https://bugzilla.samba.org/show_bug.cgi?id=11059
>
>Signed-off-by: David Disseldorp <ddiss@samba.org>
>Reviewed-by: Jeremy Allison <jra@samba.org>
>
>Autobuild-User(master): Jeremy Allison <jra@samba.org>
>Autobuild-Date(master): Wed Jan 21 04:29:06 CET 2015 on sn-devel-104
>
>(cherry picked from commit 6da86012a2ca521efe0cf1bf05fcd04c3099b190)
>---
> source3/libsmb/clidfs.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/source3/libsmb/clidfs.c b/source3/libsmb/clidfs.c
>index 7a2beac..979661b 100644
>--- a/source3/libsmb/clidfs.c
>+++ b/source3/libsmb/clidfs.c
>@@ -1082,7 +1082,7 @@ bool cli_check_msdfs_proxy(TALLOC_CTX *ctx,
> 		status = cli_cm_force_encryption(cli,
> 					username,
> 					password,
>-					lp_workgroup(),
>+					domain,
> 					"IPC$");
> 		if (!NT_STATUS_IS_OK(status)) {
> 			return false;
>-- 
>2.1.2
>

Flags:

jra: review+

Actions: View

Attachments on bug 11059: 10626 | 10631 | 10642 | 10643 | 10644 | 10645