summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJakob Unterwurzacher2021-10-15 17:29:03 +0200
committerJakob Unterwurzacher2021-10-15 17:35:12 +0200
commit3b881b01744992da138f7f72390776c65782bcc2 (patch)
tree7ccd554785378a5fb792107c3741ecfb10cb13b8
parent75cace05684df3afa1f2519ec59f73fcdfac25f5 (diff)
tests: add TestForceOwner
https://github.com/rfjakob/gocryptfs/issues/609 https://github.com/rfjakob/gocryptfs/pull/610
-rw-r--r--tests/defaults/main_test.go66
1 files changed, 66 insertions, 0 deletions
diff --git a/tests/defaults/main_test.go b/tests/defaults/main_test.go
index 0045981..5310003 100644
--- a/tests/defaults/main_test.go
+++ b/tests/defaults/main_test.go
@@ -425,3 +425,69 @@ func TestFsync(t *testing.T) {
t.Fatal(err)
}
}
+
+// force_owner was broken by the v2.0 rewrite:
+// The owner was only forced for GETATTR, but not for CREATE or LOOKUP.
+//
+// https://github.com/rfjakob/gocryptfs/issues/609
+// https://github.com/rfjakob/gocryptfs/pull/610
+func TestForceOwner(t *testing.T) {
+ cDir := test_helpers.InitFS(t)
+ os.Chmod(cDir, 0777) // Mount needs to be accessible for us
+ pDir := cDir + ".mnt"
+ test_helpers.MountOrFatal(t, cDir, pDir, "-force_owner=1234:1234", "-extpass=echo test")
+ defer test_helpers.UnmountPanic(pDir)
+
+ // We need an unrestricted umask
+ oldmask := syscall.Umask(0)
+ defer syscall.Umask(oldmask)
+
+ foo := pDir + "/foo"
+
+ // In the answer to a FUSE CREATE, gocryptfs sends file information including
+ // the owner. This is cached by the kernel and will be used for the next
+ // stat() call.
+ fd, err := syscall.Creat(foo, 0666)
+ if err != nil {
+ t.Fatal(err)
+ }
+ syscall.Close(fd)
+
+ var st syscall.Stat_t
+ if err := syscall.Stat(foo, &st); err != nil {
+ t.Fatal(err)
+ }
+ if st.Uid != 1234 || st.Gid != 1234 {
+ t.Errorf("CREATE returned uid or gid != 1234: %#v", st)
+ }
+
+ // We can clear the kernel stat() cache by writing to the file
+ fd, err = syscall.Open(foo, syscall.O_WRONLY, 0)
+ if err != nil {
+ t.Fatal(err)
+ }
+ if _, err := syscall.Write(fd, []byte("hello world")); err != nil {
+ t.Fatal(err)
+ }
+ syscall.Close(fd)
+
+ // This stat() triggers a new GETATTR
+ if err := syscall.Stat(foo, &st); err != nil {
+ t.Fatal(err)
+ }
+ if st.Uid != 1234 || st.Gid != 1234 {
+ t.Errorf("GETATTR 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")
+
+ // This stat() triggers a new LOOKUP
+ if err := syscall.Stat(foo, &st); err != nil {
+ t.Fatal(err)
+ }
+ if st.Uid != 1234 || st.Gid != 1234 {
+ t.Errorf("LOOKUP returned uid or gid != 1234: %#v", st)
+ }
+}