Attachment 12547 Details for Bug 12298 – patch backported to 4.5 (with cherrypick marker)

[patch] patch backported to 4.5 (with cherrypick marker)

dom-patch-4.5.patch (text/plain), 7.14 KB, created by Noel Power on 2016-10-06 10:23:43 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Noel Power

Created: 2016-10-06 10:23:43 UTC

Size: 7.14 KB

patch

obsolete

>From 9e4abd826d0ee105e222bc8899ddb23595e1bc25 Mon Sep 17 00:00:00 2001
>From: Noel Power <noel.power@suse.com>
>Date: Thu, 29 Sep 2016 16:50:58 +0100
>Subject: [PATCH 1/2] Add a blackbox tests for id & getent to test domain@realm
> type credentials
>
>Using domain@realm credentials has been problematic when
>global conf setting "winbind use default domain" is enabled, this patch
>creates a new s4member_dflt_domain environment (where
>"winbind use default domain" is enabled) and runs getent & id against the
>normal s4member & and new s4member_dflt_domain environments
>
>BUG: https://bugzilla.samba.org/show_bug.cgi?id=12298
>
>Signed-off-by: Noel Power <noel.power@suse.com>
>Reviewed-by: Jeremy Allison <jra@samba.org>
>Reviewed-by: Andreas Schneider <asn@samba.org>
>(cherry picked from commit ebfe3c85d0bbdf5d5f7459ddd61e3b44c3ec2bd3)
>---
> selftest/target/Samba.pm        |  1 +
> selftest/target/Samba4.pm       | 34 ++++++++++++++++++++++++++++++----
> source4/selftest/tests.py       |  7 +++++++
> testprogs/blackbox/dom_parse.sh | 27 +++++++++++++++++++++++++++
> 4 files changed, 65 insertions(+), 4 deletions(-)
> create mode 100755 testprogs/blackbox/dom_parse.sh
>
>diff --git a/selftest/target/Samba.pm b/selftest/target/Samba.pm
>index 69b956d..a8f3a9b 100644
>--- a/selftest/target/Samba.pm
>+++ b/selftest/target/Samba.pm
>@@ -300,6 +300,7 @@ sub get_interface($)
>     $interfaces{"fileserver"} = 35;
>     $interfaces{"fakednsforwarder1"} = 36;
>     $interfaces{"fakednsforwarder2"} = 37;
>+    $interfaces{"s4member_dflt"} = 38;
> 
>     # update lib/socket_wrapper/socket_wrapper.c
>     #  #define MAX_WRAPPED_INTERFACES 40
>diff --git a/selftest/target/Samba4.pm b/selftest/target/Samba4.pm
>index ed88ac5..f1de4b9 100755
>--- a/selftest/target/Samba4.pm
>+++ b/selftest/target/Samba4.pm
>@@ -954,9 +954,9 @@ $extra_smbconf_shares
> 	return $self->provision_raw_step2($ctx, $ret);
> }
> 
>-sub provision_s4member($$$)
>+sub provision_s4member($$$$$)
> {
>-	my ($self, $prefix, $dcvars) = @_;
>+	my ($self, $prefix, $dcvars, $hostname, $more_conf) = @_;
> 	print "PROVISIONING MEMBER...\n";
> 	my $extra_smb_conf = "
>         passdb backend = samba_dsdb
>@@ -975,9 +975,12 @@ rpc_server:spoolss = embedded
> rpc_daemon:spoolssd = embedded
> rpc_server:tcpip = no
> ";
>+	if ($more_conf) {
>+		$extra_smb_conf = $extra_smb_conf . $more_conf . "\n";
>+	}
> 	my $ret = $self->provision($prefix,
> 				   "member server",
>-				   "s4member",
>+				   $hostname,
> 				   "SAMBADOMAIN",
> 				   "samba.example.com",
> 				   "2008",
>@@ -1967,6 +1970,11 @@ sub setup_env($$$)
> 			$self->setup_ad_dc_ntvfs("$path/ad_dc_ntvfs");
> 		}
> 		return $self->setup_subdom_dc("$path/subdom_dc", $self->{vars}->{ad_dc_ntvfs});
>+	} elsif ($envname eq "s4member_dflt_domain") {
>+		if (not defined($self->{vars}->{ad_dc_ntvfs})) {
>+			$self->setup_ad_dc_ntvfs("$path/ad_dc_ntvfs");
>+		}
>+		return $self->setup_s4member_dflt_domain("$path/s4member_dflt_domain", $self->{vars}->{ad_dc_ntvfs});
> 	} elsif ($envname eq "s4member") {
> 		if (not defined($self->{vars}->{ad_dc_ntvfs})) {
> 			$self->setup_ad_dc_ntvfs("$path/ad_dc_ntvfs");
>@@ -2005,7 +2013,7 @@ sub setup_s4member($$$)
> {
> 	my ($self, $path, $dc_vars) = @_;
> 
>-	my $env = $self->provision_s4member($path, $dc_vars);
>+	my $env = $self->provision_s4member($path, $dc_vars, "s4member");
> 
> 	if (defined $env) {
> 	        if (not defined($self->check_or_start($env, "standard"))) {
>@@ -2018,6 +2026,24 @@ sub setup_s4member($$$)
> 	return $env;
> }
> 
>+sub setup_s4member_dflt_domain($$$)
>+{
>+	my ($self, $path, $dc_vars) = @_;
>+
>+	my $env = $self->provision_s4member($path, $dc_vars, "s4member_dflt",
>+					    "winbind use default domain = yes");
>+
>+	if (defined $env) {
>+	        if (not defined($self->check_or_start($env, "standard"))) {
>+		        return undef;
>+		}
>+
>+		$self->{vars}->{s4member_dflt_domain} = $env;
>+	}
>+
>+	return $env;
>+}
>+
> sub setup_rpc_proxy($$$)
> {
> 	my ($self, $path, $dc_vars) = @_;
>diff --git a/source4/selftest/tests.py b/source4/selftest/tests.py
>index e4af5f8..ef11a70 100755
>--- a/source4/selftest/tests.py
>+++ b/source4/selftest/tests.py
>@@ -515,6 +515,13 @@ for env in ["nt4_dc", "nt4_member", "ad_dc", "ad_member", "s4member", "chgdcpass
> 
>     plantestsuite("samba.ntlm_auth.(%s:local)" % env, "%s:local" % env, [os.path.join(samba3srcdir, "script/tests/test_ntlm_auth_s3.sh"), valgrindify(python), samba3srcdir, ntlm_auth3,  '$DOMAIN', '$DC_USERNAME', '$DC_PASSWORD', configuration])
> 
>+for env in ["s4member_dflt_domain", "s4member"]:
>+    for cmd in ["id", "getent"]:
>+        users = ["$DC_USERNAME", "$DC_USERNAME@$REALM"]
>+        if env == "s4member":
>+            users = ["$DOMAIN/$DC_USERNAME", "$DC_USERNAME@$REALM"]
>+        for usr in users:
>+            plantestsuite("samba4.winbind.dom_name_parse.cmd", env, "%s/dom_parse.sh %s %s" % (bbdir,cmd,usr))
> 
> nsstest4 = binpath("nsstest")
> for env in ["ad_dc:local", "s4member:local", "nt4_dc:local", "ad_member:local", "nt4_member:local"]:
>diff --git a/testprogs/blackbox/dom_parse.sh b/testprogs/blackbox/dom_parse.sh
>new file mode 100755
>index 0000000..dd14f0d
>--- /dev/null
>+++ b/testprogs/blackbox/dom_parse.sh
>@@ -0,0 +1,27 @@
>+#!/bin/sh
>+# Blackbox wrapper for nsstest
>+# Copyright (C) 2006-2007 Jelmer Vernooij <jelmer@samba.org>
>+# Copyright (C) 2006-2008 Andrew Bartlett <abartlet@samba.org>
>+
>+if [ $# -lt 2 ]; then
>+cat <<EOF
>+Usage: dom_parse.sh [id|getent] $USER
>+EOF
>+exit 1;
>+fi
>+
>+USER=$2
>+CMD=$1
>+EXTRA=""
>+shift 2
>+failed=0
>+
>+. `dirname $0`/subunit.sh
>+
>+if [ "$CMD" = "getent" ]; then
>+    EXTRA="passwd"
>+fi
>+
>+testit "samba4.winbind.dom_name_parse.cmd.$CMD" $CMD $EXTRA $USER || failed=`expr $failed + 1`
>+
>+exit $failed
>-- 
>2.1.4
>
>
>From 35d3f3bbaed8805d6f302022f8ed63da4cbc4505 Mon Sep 17 00:00:00 2001
>From: Noel Power <noel.power@suse.com>
>Date: Tue, 20 Sep 2016 11:49:49 +0100
>Subject: [PATCH 2/2] s3/winbindd: using default domain with user@domain.com
> format fails
>
>For example for samba client joined to a windows AD DC the following
>commands fail if 'winbind use default domain = yes'
>   getent passwd user@domain.com
>   ssh -o user=user@domain.com localhost
>
>The same commands succeed if the setting above has the default 'no' value
>
>BUG: https://bugzilla.samba.org/show_bug.cgi?id=12298
>
>Signed-off-by: Noel Power <noel.power@suse.com>
>Reviewed-by: Jeremy Allison <jra@samba.org>
>Reviewed-by: Andreas Schneider <asn@samba.org>
>
>Autobuild-User(master): Jeremy Allison <jra@samba.org>
>Autobuild-Date(master): Mon Oct  3 23:37:44 CEST 2016 on sn-devel-144
>
>(cherry picked from commit 7c786f89820dc1f8e2a7e8da1b80042dd69b7188)
>---
> source3/winbindd/winbindd_util.c | 5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)
>
>diff --git a/source3/winbindd/winbindd_util.c b/source3/winbindd/winbindd_util.c
>index f0344b1..d707173 100644
>--- a/source3/winbindd/winbindd_util.c
>+++ b/source3/winbindd/winbindd_util.c
>@@ -1103,10 +1103,11 @@ bool parse_domain_user(const char *domuser, fstring domain, fstring user)
> 
> 	if ( !p ) {
> 		fstrcpy(user, domuser);
>+		p = strchr(domuser, '@');
> 
>-		if ( assume_domain(lp_workgroup())) {
>+		if ( assume_domain(lp_workgroup()) && p == NULL) {
> 			fstrcpy(domain, lp_workgroup());
>-		} else if ((p = strchr(domuser, '@')) != NULL) {
>+		} else if (p != NULL) {
> 			fstrcpy(domain, p + 1);
> 			user[PTR_DIFF(p, domuser)] = 0;
> 		} else {
>-- 
>2.1.4
>

Flags:

asn: review+

Actions: View

Attachments on bug 12298: 12519 | 12520 | 12521 | 12546 | 12547