From 5306fc345be41371820ea50a3f64efab08f2e74c Mon Sep 17 00:00:00 2001 From: Jakob Unterwurzacher Date: Sat, 26 Jun 2021 18:44:52 +0200 Subject: fusefrontend: convert last callers from openBackingDir to prepareAtSyscall --- internal/fusefrontend/node_dir_ops.go | 12 +++++------- internal/fusefrontend/node_prepare_syscall.go | 7 ++++--- 2 files changed, 9 insertions(+), 10 deletions(-) (limited to 'internal/fusefrontend') diff --git a/internal/fusefrontend/node_dir_ops.go b/internal/fusefrontend/node_dir_ops.go index 0f48876..6d03544 100644 --- a/internal/fusefrontend/node_dir_ops.go +++ b/internal/fusefrontend/node_dir_ops.go @@ -4,7 +4,6 @@ import ( "context" "fmt" "io" - "path/filepath" "runtime" "syscall" @@ -239,15 +238,14 @@ func (n *Node) Readdir(ctx context.Context) (fs.DirStream, syscall.Errno) { // Symlink-safe through Unlinkat() + AT_REMOVEDIR. func (n *Node) Rmdir(ctx context.Context, name string) (code syscall.Errno) { rn := n.rootNode() - p := filepath.Join(n.Path(), name) - parentDirFd, cName, err := rn.openBackingDir(p) - if err != nil { - return fs.ToErrno(err) + parentDirFd, cName, errno := n.prepareAtSyscall(name) + if errno != 0 { + return errno } defer syscall.Close(parentDirFd) if rn.args.PlaintextNames { // Unlinkat with AT_REMOVEDIR is equivalent to Rmdir - err = unix.Unlinkat(parentDirFd, cName, unix.AT_REMOVEDIR) + err := unix.Unlinkat(parentDirFd, cName, unix.AT_REMOVEDIR) return fs.ToErrno(err) } // Unless we are running as root, we need read, write and execute permissions @@ -256,7 +254,7 @@ func (n *Node) Rmdir(ctx context.Context, name string) (code syscall.Errno) { var origMode uint32 if !rn.args.PreserveOwner { var st unix.Stat_t - err = syscallcompat.Fstatat(parentDirFd, cName, &st, unix.AT_SYMLINK_NOFOLLOW) + err := syscallcompat.Fstatat(parentDirFd, cName, &st, unix.AT_SYMLINK_NOFOLLOW) if err != nil { return fs.ToErrno(err) } diff --git a/internal/fusefrontend/node_prepare_syscall.go b/internal/fusefrontend/node_prepare_syscall.go index fa558e8..283096f 100644 --- a/internal/fusefrontend/node_prepare_syscall.go +++ b/internal/fusefrontend/node_prepare_syscall.go @@ -98,11 +98,12 @@ func (n *Node) prepareAtSyscallMyself() (dirfd int, cName string, errno syscall. if n.IsRoot() { var err error rn := n.rootNode() - dirfd, cName, err = rn.openBackingDir("") + // Open cipherdir (following symlinks) + dirfd, err = syscallcompat.Open(rn.args.Cipherdir, syscall.O_DIRECTORY|syscallcompat.O_PATH, 0) if err != nil { - errno = fs.ToErrno(err) + return -1, "", fs.ToErrno(err) } - return + return dirfd, ".", 0 } // Otherwise convert to prepareAtSyscall of parent node -- cgit v1.2.3