aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJakob Unterwurzacher2018-11-04 22:05:38 +0100
committerJakob Unterwurzacher2019-01-01 16:24:25 +0100
commit2de3851abddd26949b72b9b255ce836fc93ed284 (patch)
treee7be53226bbe58fbb243617d5e46e6f7f83c4217
parent4fae240153a5300b7108bf5c2de2b6581c8ee7fd (diff)
nametransform: rename WriteLongName() -> WriteLongNameAt()
And also rename DeleteLongName() -> DeleteLongNameAt(). The naming follow the names open the openat() etc syscalls.
-rw-r--r--internal/fusefrontend/fs.go26
-rw-r--r--internal/fusefrontend/fs_dir.go6
-rw-r--r--internal/fusefrontend/xattr.go5
-rw-r--r--internal/nametransform/longnames.go4
4 files changed, 22 insertions, 19 deletions
diff --git a/internal/fusefrontend/fs.go b/internal/fusefrontend/fs.go
index 022fd5c..2108ea4 100644
--- a/internal/fusefrontend/fs.go
+++ b/internal/fusefrontend/fs.go
@@ -226,14 +226,14 @@ func (fs *FS) Create(path string, flags uint32, mode uint32, context *fuse.Conte
// Handle long file name
if !fs.args.PlaintextNames && nametransform.IsLongContent(cName) {
// Create ".name"
- err = fs.nameTransform.WriteLongName(dirfd, cName, path)
+ err = fs.nameTransform.WriteLongNameAt(dirfd, cName, path)
if err != nil {
return nil, fuse.ToStatus(err)
}
// Create content
fd, err = syscallcompat.Openat(dirfd, cName, newFlags|os.O_CREATE|os.O_EXCL, mode)
if err != nil {
- nametransform.DeleteLongName(dirfd, cName)
+ nametransform.DeleteLongNameAt(dirfd, cName)
return nil, fuse.ToStatus(err)
}
@@ -314,14 +314,14 @@ func (fs *FS) Mknod(path string, mode uint32, dev uint32, context *fuse.Context)
defer syscall.Close(dirfd)
// Create ".name" file to store long file name (except in PlaintextNames mode)
if !fs.args.PlaintextNames && nametransform.IsLongContent(cName) {
- err = fs.nameTransform.WriteLongName(dirfd, cName, path)
+ err = fs.nameTransform.WriteLongNameAt(dirfd, cName, path)
if err != nil {
return fuse.ToStatus(err)
}
// Create "gocryptfs.longfile." device node
err = syscallcompat.Mknodat(dirfd, cName, mode, int(dev))
if err != nil {
- nametransform.DeleteLongName(dirfd, cName)
+ nametransform.DeleteLongNameAt(dirfd, cName)
}
} else {
// Create regular device node
@@ -436,7 +436,7 @@ func (fs *FS) Unlink(path string, context *fuse.Context) (code fuse.Status) {
}
// Delete ".name" file
if !fs.args.PlaintextNames && nametransform.IsLongContent(cName) {
- err = nametransform.DeleteLongName(dirfd, cName)
+ err = nametransform.DeleteLongNameAt(dirfd, cName)
if err != nil {
tlog.Warn.Printf("Unlink: could not delete .name file: %v", err)
}
@@ -474,14 +474,14 @@ func (fs *FS) Symlink(target string, linkName string, context *fuse.Context) (co
}
// Create ".name" file to store long file name (except in PlaintextNames mode)
if !fs.args.PlaintextNames && nametransform.IsLongContent(cName) {
- err = fs.nameTransform.WriteLongName(dirfd, cName, linkName)
+ err = fs.nameTransform.WriteLongNameAt(dirfd, cName, linkName)
if err != nil {
return fuse.ToStatus(err)
}
// Create "gocryptfs.longfile." symlink
err = syscallcompat.Symlinkat(cTarget, dirfd, cName)
if err != nil {
- nametransform.DeleteLongName(dirfd, cName)
+ nametransform.DeleteLongNameAt(dirfd, cName)
}
} else {
// Create symlink
@@ -501,7 +501,7 @@ func (fs *FS) Symlink(target string, linkName string, context *fuse.Context) (co
return fuse.OK
}
-// Rename implements pathfs.Filesystem.
+// Rename - FUSE call.
func (fs *FS) Rename(oldPath string, newPath string, context *fuse.Context) (code fuse.Status) {
if fs.isFiltered(newPath) {
return fuse.EPERM
@@ -526,7 +526,7 @@ func (fs *FS) Rename(oldPath string, newPath string, context *fuse.Context) (cod
// Long destination file name: create .name file
nameFileAlreadyThere := false
if nametransform.IsLongContent(newCName) {
- err = fs.nameTransform.WriteLongName(newDirfd, newCName, newPath)
+ err = fs.nameTransform.WriteLongNameAt(newDirfd, newCName, newPath)
// Failure to write the .name file is expected when the target path already
// exists. Since hashes are pretty unique, there is no need to modify the
// .name file in this case, and we ignore the error.
@@ -553,12 +553,12 @@ func (fs *FS) Rename(oldPath string, newPath string, context *fuse.Context) (cod
if err != nil {
if nametransform.IsLongContent(newCName) && nameFileAlreadyThere == false {
// Roll back .name creation unless the .name file was already there
- nametransform.DeleteLongName(newDirfd, newCName)
+ nametransform.DeleteLongNameAt(newDirfd, newCName)
}
return fuse.ToStatus(err)
}
if nametransform.IsLongContent(oldCName) {
- nametransform.DeleteLongName(oldDirfd, oldCName)
+ nametransform.DeleteLongNameAt(oldDirfd, oldCName)
}
return fuse.OK
}
@@ -583,14 +583,14 @@ func (fs *FS) Link(oldPath string, newPath string, context *fuse.Context) (code
defer syscall.Close(newDirFd)
// Handle long file name (except in PlaintextNames mode)
if !fs.args.PlaintextNames && nametransform.IsLongContent(cNewName) {
- err = fs.nameTransform.WriteLongName(newDirFd, cNewName, newPath)
+ err = fs.nameTransform.WriteLongNameAt(newDirFd, cNewName, newPath)
if err != nil {
return fuse.ToStatus(err)
}
// Create "gocryptfs.longfile." link
err = syscallcompat.Linkat(oldDirFd, cOldName, newDirFd, cNewName, 0)
if err != nil {
- nametransform.DeleteLongName(newDirFd, cNewName)
+ nametransform.DeleteLongNameAt(newDirFd, cNewName)
}
} else {
// Create regular link
diff --git a/internal/fusefrontend/fs_dir.go b/internal/fusefrontend/fs_dir.go
index 9486802..5a6bc9d 100644
--- a/internal/fusefrontend/fs_dir.go
+++ b/internal/fusefrontend/fs_dir.go
@@ -84,7 +84,7 @@ func (fs *FS) Mkdir(newPath string, mode uint32, context *fuse.Context) (code fu
// Handle long file name
if nametransform.IsLongContent(cName) {
// Create ".name"
- err = fs.nameTransform.WriteLongName(dirfd, cName, newPath)
+ err = fs.nameTransform.WriteLongNameAt(dirfd, cName, newPath)
if err != nil {
return fuse.ToStatus(err)
}
@@ -92,7 +92,7 @@ func (fs *FS) Mkdir(newPath string, mode uint32, context *fuse.Context) (code fu
// Create directory
err = fs.mkdirWithIv(dirfd, cName, mode)
if err != nil {
- nametransform.DeleteLongName(dirfd, cName)
+ nametransform.DeleteLongNameAt(dirfd, cName)
return fuse.ToStatus(err)
}
} else {
@@ -253,7 +253,7 @@ retry:
}
// Delete .name file
if nametransform.IsLongContent(cName) {
- nametransform.DeleteLongName(parentDirFd, cName)
+ nametransform.DeleteLongNameAt(parentDirFd, cName)
}
// The now-deleted directory may have been in the DirIV cache. Clear it.
fs.nameTransform.DirIVCache.Clear()
diff --git a/internal/fusefrontend/xattr.go b/internal/fusefrontend/xattr.go
index 81cb207..a0f795a 100644
--- a/internal/fusefrontend/xattr.go
+++ b/internal/fusefrontend/xattr.go
@@ -70,7 +70,10 @@ func (fs *FS) SetXAttr(path string, attr string, data []byte, flags int, context
return unpackXattrErr(xattr.LSetWithFlags(cPath, cAttr, cData, flags))
}
-// RemoveXAttr implements pathfs.Filesystem.
+// RemoveXAttr - FUSE call.
+//
+// TODO: Make symlink-safe. Blocker: package xattr does not provide
+// fremovexattr(2).
func (fs *FS) RemoveXAttr(path string, attr string, context *fuse.Context) fuse.Status {
if fs.isFiltered(path) {
return fuse.EPERM
diff --git a/internal/nametransform/longnames.go b/internal/nametransform/longnames.go
index 6788ce6..f0f0490 100644
--- a/internal/nametransform/longnames.go
+++ b/internal/nametransform/longnames.go
@@ -99,7 +99,7 @@ func ReadLongNameAt(dirfd int, cName string) (string, error) {
// DeleteLongName deletes "hashName.name" in the directory openend at "dirfd".
//
// This function is symlink-safe through the use of Unlinkat().
-func DeleteLongName(dirfd int, hashName string) error {
+func DeleteLongNameAt(dirfd int, hashName string) error {
err := syscallcompat.Unlinkat(dirfd, hashName+LongNameSuffix, 0)
if err != nil {
tlog.Warn.Printf("DeleteLongName: %v", err)
@@ -112,7 +112,7 @@ func DeleteLongName(dirfd int, hashName string) error {
// Base()named internally.
//
// This function is symlink-safe through the use of Openat().
-func (n *NameTransform) WriteLongName(dirfd int, hashName string, plainName string) (err error) {
+func (n *NameTransform) WriteLongNameAt(dirfd int, hashName string, plainName string) (err error) {
plainName = filepath.Base(plainName)
// Encrypt the basename