From 5f955423b736d56b5b741fbd1b853c83044aa0fe Mon Sep 17 00:00:00 2001 From: Jakob Unterwurzacher Date: Mon, 10 Jan 2022 20:05:36 +0100 Subject: fusefrontend: fix -force_owner not affecting MKNOD Fixes https://github.com/rfjakob/gocryptfs/issues/629 --- tests/defaults/main_test.go | 15 ++++++++++++++- 1 file changed, 14 insertions(+), 1 deletion(-) (limited to 'tests') diff --git a/tests/defaults/main_test.go b/tests/defaults/main_test.go index 0f31a72..7633e8b 100644 --- a/tests/defaults/main_test.go +++ b/tests/defaults/main_test.go @@ -427,10 +427,11 @@ func TestFsync(t *testing.T) { } // force_owner was broken by the v2.0 rewrite: -// The owner was only forced for GETATTR, but not for CREATE or LOOKUP. +// The owner was only forced for GETATTR, but not for CREATE, LOOKUP, MKNOD. // // https://github.com/rfjakob/gocryptfs/issues/609 // https://github.com/rfjakob/gocryptfs/pull/610 +// https://github.com/rfjakob/gocryptfs/issues/629 func TestForceOwner(t *testing.T) { cDir := test_helpers.InitFS(t) os.Chmod(cDir, 0777) // Mount needs to be accessible for us @@ -479,6 +480,18 @@ func TestForceOwner(t *testing.T) { t.Errorf("GETATTR returned uid or gid != 1234: %#v", st) } + // Test MKNOD + sock := pDir + "/sock" + if err := syscall.Mknod(sock, syscall.S_IFSOCK|0600, 0); err != nil { + t.Fatal(err) + } + if err := syscall.Stat(sock, &st); err != nil { + t.Fatal(err) + } + if st.Uid != 1234 || st.Gid != 1234 { + t.Errorf("MKNOD returned uid or gid != 1234: %#v", st) + } + // Remount to clear cache test_helpers.UnmountPanic(pDir) test_helpers.MountOrFatal(t, cDir, pDir, "-force_owner=1234:1234", "-extpass=echo test") -- cgit v1.2.3