Attachment 18337 Details for Bug 15435 – backport for 4.18

[patch] backport for 4.18

bug-15435-v4-18.patch (text/plain), 7.54 KB, created by Noel Power on 2024-06-12 10:12:36 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Noel Power

Created: 2024-06-12 10:12:36 UTC

Size: 7.54 KB

patch

obsolete

>From 4da6b04ba62489d9ec503fda25ca6103d2edadc0 Mon Sep 17 00:00:00 2001
>From: Noel Power <noel.power@suse.com>
>Date: Tue, 11 Jun 2024 11:19:50 +0100
>Subject: [PATCH 1/2] selftest: Add a python blackbox test for some misc
> (widelink) DFS tests
>
>On master attempting to chdir into a nested dfs link
>
>e.g. cd dfslink (works)
>     cd dfslink/another_dfslink (fails)
>
>[1] Add a test for this scenario (nested chdir)
>[2] Add test for enumerating a dfs link in root of dfs share
>[3] Add a test to check case insensitive chdir into dfs link on widelink
>  enabled share
>
>Add knownfails for tests 1 and 3
>
>Signed-off-by: Noel Power <noel.power@suse.com>
>Reviewed-by: Jeremy Allison <jra@samba.org>
>
>BUG: https://bugzilla.samba.org/show_bug.cgi?id=15435
>(cherry picked from commit 7f1de90f72d6e8287aec6ab1d9f7776b7df624e5)
>
>[noel.power@suse.com backported to Samba 4.19 changed knownfails because
>  test_ci_chdir and test_enumerate_dfs_link don't fail]
>---
> .../samba/tests/blackbox/misc_dfs_widelink.py | 86 +++++++++++++++++++
> selftest/knownfail.d/dfs_widelink             |  1 +
> source4/selftest/tests.py                     |  3 +
> 3 files changed, 90 insertions(+)
> create mode 100644 python/samba/tests/blackbox/misc_dfs_widelink.py
> create mode 100644 selftest/knownfail.d/dfs_widelink
>
>diff --git a/python/samba/tests/blackbox/misc_dfs_widelink.py b/python/samba/tests/blackbox/misc_dfs_widelink.py
>new file mode 100644
>index 00000000000..7948590d710
>--- /dev/null
>+++ b/python/samba/tests/blackbox/misc_dfs_widelink.py
>@@ -0,0 +1,86 @@
>+# Blackbox tests for DFS (widelink)
>+#
>+# Copyright (C) Noel Power noel.power@suse.com
>+#
>+# This program is free software; you can redistribute it and/or modify
>+# it under the terms of the GNU General Public License as published by
>+# the Free Software Foundation; either version 3 of the License, or
>+# (at your option) any later version.
>+#
>+# This program is distributed in the hope that it will be useful,
>+# but WITHOUT ANY WARRANTY; without even the implied warranty of
>+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
>+# GNU General Public License for more details.
>+#
>+# You should have received a copy of the GNU General Public License
>+# along with this program.  If not, see <http://www.gnu.org/licenses/>.
>+#
>+from samba.tests import BlackboxTestCase, BlackboxProcessError
>+from samba.samba3 import param as s3param
>+
>+from samba.credentials import Credentials
>+
>+import os
>+
>+class DfsWidelinkBlockboxTestBase(BlackboxTestCase):
>+
>+    def setUp(self):
>+        super().setUp()
>+        self.lp = s3param.get_context()
>+        self.server = os.environ["SERVER"]
>+        self.user = os.environ["USER"]
>+        self.passwd = os.environ["PASSWORD"]
>+        self.creds = Credentials()
>+        self.creds.guess(self.lp)
>+        self.creds.set_username(self.user)
>+        self.creds.set_password(self.passwd)
>+        self.testdir = os.getenv("TESTDIR", "msdfs-share-wl")
>+        self.share = os.getenv("SHARE", "msdfs-share-wl")
>+        self.dirpath = os.path.join(os.environ["LOCAL_PATH"],self.testdir)
>+        # allow a custom teardown function to be defined
>+        self.cleanup = None
>+        self.cleanup_args = []
>+
>+    def tearDown(self):
>+        try:
>+            if (self.cleanup):
>+                self.cleanup(self.cleanup_args)
>+        except Exception as e:
>+            print("remote remove failed: %s" % str(e))
>+
>+    def build_test_cmd(self, cmd, args):
>+        cmd = [cmd, "-U%s%%%s" % (self.user, self.passwd)]
>+        cmd.extend(args)
>+        return cmd
>+
>+    def test_ci_chdir(self):
>+        parent_dir = "msdfs-src1"
>+        dirs = [parent_dir, parent_dir.upper()]
>+        # try as named dir first then try upper-cased version
>+        for adir in dirs:
>+            smbclient_args = self.build_test_cmd("smbclient", ["//%s/%s" % (self.server, self.share), "-c", "cd %s" % (adir)])
>+            try:
>+                out_str = self.check_output(smbclient_args)
>+            except BlackboxProcessError as e:
>+                print(str(e))
>+                self.fail(str(e))
>+
>+    def test_nested_chdir(self):
>+        parent_dir = "dfshop1"
>+        child_dir = "dfshop2"
>+        smbclient_args = self.build_test_cmd("smbclient", ["//%s/%s" % (self.server, self.share), "-c", "cd %s/%s" % (parent_dir,child_dir)])
>+        try:
>+            out_str = self.check_output(smbclient_args)
>+        except BlackboxProcessError as e:
>+            print(str(e))
>+            self.fail(str(e))
>+
>+    def test_enumerate_dfs_link(self):
>+        smbclient_args = self.build_test_cmd("smbclient", ["//%s/%s" % (self.server, self.share), "-c", "dir"])
>+        try:
>+            out_str = self.check_output(smbclient_args)
>+        except BlackboxProcessError as e:
>+            print(str(e))
>+            self.fail(str(e))
>+        out_str = out_str.decode()
>+        self.assertIn("msdfs-src1", out_str)
>diff --git a/selftest/knownfail.d/dfs_widelink b/selftest/knownfail.d/dfs_widelink
>new file mode 100644
>index 00000000000..cd022842d5a
>--- /dev/null
>+++ b/selftest/knownfail.d/dfs_widelink
>@@ -0,0 +1 @@
>+^samba.tests.blackbox.misc_dfs_widelink.samba.tests.blackbox.misc_dfs_widelink.DfsWidelinkBlockboxTestBase.test_nested_chdir\(fileserver\)
>diff --git a/source4/selftest/tests.py b/source4/selftest/tests.py
>index f8f7aae700d..0bc8229c3b6 100755
>--- a/source4/selftest/tests.py
>+++ b/source4/selftest/tests.py
>@@ -1257,6 +1257,9 @@ planoldpythontestsuite("fileserver",
>                        "samba.tests.blackbox.smbcacls_dfs_propagate_inherit",
>                        "samba.tests.blackbox.smbcacls_dfs_propagate_inherit(DFS-msdfs-root)",
>                        environ={'SHARE': 'smbcacls_share'})
>+
>+planoldpythontestsuite("fileserver",
>+                       "samba.tests.blackbox.misc_dfs_widelink")
> #
> # Want a selection of environments across the process models
> #
>-- 
>2.35.3
>
>
>From ca18627a4ae3c41d7f79b91d20416d88f2f8b1b7 Mon Sep 17 00:00:00 2001
>From: Noel Power <noel.power@suse.com>
>Date: Fri, 7 Jun 2024 19:35:47 +0100
>Subject: [PATCH 2/2] s3/smbd: fix nested chdir into msdfs links on (widelinks
> = yes) share
>
>This patch also removes known fail for existing test
>
>BUG: https://bugzilla.samba.org/show_bug.cgi?id=15435
>
>Signed-off-by: Noel Power <noel.power@suse.com>
>Reviewed-by: Jeremy Allison <jra@samba.org>
>
>Autobuild-User(master): Jeremy Allison <jra@samba.org>
>Autobuild-Date(master): Tue Jun 11 19:31:40 UTC 2024 on atb-devel-224
>
>(cherry picked from commit 788ef8f07c75d5e6eca5b8f18d93d96f31574267)
>[noel.power@suse.com backported to Samba 4.18 changed existing handling
>     where MSDFS links are detected to also handle the ENOENT set from
>     vfs_widelink_openat]
>---
> selftest/knownfail.d/dfs_widelink | 1 -
> source3/smbd/files.c              | 2 +-
> 2 files changed, 1 insertion(+), 2 deletions(-)
> delete mode 100644 selftest/knownfail.d/dfs_widelink
>
>diff --git a/selftest/knownfail.d/dfs_widelink b/selftest/knownfail.d/dfs_widelink
>deleted file mode 100644
>index cd022842d5a..00000000000
>--- a/selftest/knownfail.d/dfs_widelink
>+++ /dev/null
>@@ -1 +0,0 @@
>-^samba.tests.blackbox.misc_dfs_widelink.samba.tests.blackbox.misc_dfs_widelink.DfsWidelinkBlockboxTestBase.test_nested_chdir\(fileserver\)
>diff --git a/source3/smbd/files.c b/source3/smbd/files.c
>index 32388f133f3..77b17beb62c 100644
>--- a/source3/smbd/files.c
>+++ b/source3/smbd/files.c
>@@ -966,7 +966,7 @@ next:
> 	 *
> 	 * But we should be prepared to handle ELOOP too.
> 	 */
>-	if ((fd == -1) && (errno == ENOTDIR || errno == ELOOP)) {
>+	if ((fd == -1) && (errno == ENOTDIR || errno == ELOOP || errno == ENOENT)) {
> 		NTSTATUS orig_status = map_nt_error_from_unix(errno);
> 
> 		status = readlink_talloc(
>-- 
>2.35.3
>

Flags:

jra: review+

Actions: View

Attachments on bug 15435: 18013 | 18022 | 18331 | 18332 | 18335 | 18336 | 18337