From 5b13dcc73a799c87aae8376f424c75b43db74988 Mon Sep 17 00:00:00 2001 From: Sebastiaan van Stijn Date: Mon, 28 Sep 2020 13:07:56 +0200 Subject: [PATCH] mount.isFUSE(): remove unused error return The error itself was unused, so may as well remove it. Signed-off-by: Sebastiaan van Stijn --- mount/mount_linux.go | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/mount/mount_linux.go b/mount/mount_linux.go index d12a9c5a2..3d5dceee6 100644 --- a/mount/mount_linux.go +++ b/mount/mount_linux.go @@ -106,20 +106,20 @@ func Unmount(target string, flags int) error { return nil } -func isFUSE(dir string) (bool, error) { +func isFUSE(dir string) bool { // fuseSuperMagic is defined in statfs(2) const fuseSuperMagic = 0x65735546 var st unix.Statfs_t if err := unix.Statfs(dir, &st); err != nil { - return false, err + return false } - return st.Type == fuseSuperMagic, nil + return st.Type == fuseSuperMagic } func unmount(target string, flags int) error { // For FUSE mounts, attempting to execute fusermount helper binary is preferred // https://github.com/containerd/containerd/pull/3765#discussion_r342083514 - if ok, err := isFUSE(target); err == nil && ok { + if isFUSE(target) { for _, helperBinary := range []string{"fusermount3", "fusermount"} { cmd := exec.Command(helperBinary, "-u", target) if err := cmd.Run(); err == nil {