mount: extract FUSE unmounting to a function

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>

mount/mount_linux.go

@@ -106,9 +106,10 @@ func Unmount(target string, flags int) error {
 	return nil
 }
 
+// fuseSuperMagic is defined in statfs(2)
+const fuseSuperMagic = 0x65735546
+
 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
@@ -116,16 +117,26 @@ func isFUSE(dir string) bool {
 	return st.Type == fuseSuperMagic
 }
 
+// unmountFUSE attempts to unmount using fusermount/fusermount3 helper binary.
+//
+// For FUSE mounts, using these helper binaries is preferred, see:
+// https://github.com/containerd/containerd/pull/3765#discussion_r342083514
+func unmountFUSE(target string) error {
+	var err error
+	for _, helperBinary := range []string{"fusermount3", "fusermount"} {
+		cmd := exec.Command(helperBinary, "-u", target)
+		err = cmd.Run()
+		if err == nil {
+			return nil
+		}
+	}
+	return err
+}
+
 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 isFUSE(target) {
-		for _, helperBinary := range []string{"fusermount3", "fusermount"} {
+		if err := unmountFUSE(target); err == nil {
-			cmd := exec.Command(helperBinary, "-u", target)
+			return nil
-			if err := cmd.Run(); err == nil {
-				return nil
-			}
-			// ignore error and try unix.Unmount
 		}
 	}
 	for i := 0; i < 50; i++ {