summaryrefslogtreecommitdiff
path: root/tests
diff options
context:
space:
mode:
authorSebastian Lackner2019-01-05 04:33:50 +0100
committerrfjakob2019-01-05 12:34:40 +0100
commit5055f39bd534b1f13257f95ffdc28575b9b2e3ed (patch)
treeeb3f1c960d9a51a6301dc6586bdab10d579b715b /tests
parentf17721c364a522f87b3351a73676405d44b955f6 (diff)
fusefrontend: Allow to set/remove xattr on directory without read permission.
Setting/removing extended attributes on directories was partially fixed with commit eff35e60b63331e3e10f921792baa10b236a721d. However, on most file systems it is also possible to do these operations without read access (see tests). Since we cannot open a write-access fd to a directory, we have to use the /proc/self/fd trick (already used for ListXAttr) for the other operations aswell. For simplicity, let's separate the Linux and Darwin code again (basically revert commit f320b76fd189a363a34bffe981aa67ab97df3362), and always use the /proc/self/fd trick on Linux. On Darwin we use the best-effort approach with openBackingFile() as a fallback. More discussion about the available options is available in https://github.com/rfjakob/gocryptfs/issues/308.
Diffstat (limited to 'tests')
-rw-r--r--tests/xattr/xattr_integration_test.go30
1 files changed, 29 insertions, 1 deletions
diff --git a/tests/xattr/xattr_integration_test.go b/tests/xattr/xattr_integration_test.go
index f182fd9..29c3e84 100644
--- a/tests/xattr/xattr_integration_test.go
+++ b/tests/xattr/xattr_integration_test.go
@@ -129,7 +129,7 @@ func TestSetGetRmDir(t *testing.T) {
fn := test_helpers.DefaultPlainDir + "/TestSetGetRmDir"
err := syscall.Mkdir(fn, 0700)
if err != nil {
- t.Fatalf("creating fifo failed: %v", err)
+ t.Fatalf("creating directory failed: %v", err)
}
setGetRmList(fn)
}
@@ -316,3 +316,31 @@ func TestSet0200File(t *testing.T) {
t.Error(err)
}
}
+
+// Listing xattrs should work even when we don't have read access
+func TestList0000Dir(t *testing.T) {
+ fn := test_helpers.DefaultPlainDir + "/TestList0000Dir"
+ err := syscall.Mkdir(fn, 0000)
+ if err != nil {
+ t.Fatalf("creating directory failed: %v", err)
+ }
+ _, err = xattr.LList(fn)
+ os.Chmod(fn, 0700)
+ if err != nil {
+ t.Error(err)
+ }
+}
+
+// Setting xattrs should work even when we don't have read access
+func TestSet0200Dir(t *testing.T) {
+ fn := test_helpers.DefaultPlainDir + "/TestSet0200Dir"
+ err := syscall.Mkdir(fn, 0200)
+ if err != nil {
+ t.Fatalf("creating directory failed: %v", err)
+ }
+ err = xattr.LSet(fn, "user.foo", []byte("bar"))
+ os.Chmod(fn, 0700)
+ if err != nil {
+ t.Error(err)
+ }
+}