Revert "Fix cleanup in case of failed mount"

This reverts commit bf5ffb5fd8558bd799791834def431c0cee5a11f.

Cleanup of mount doesn't work the way it was envisioned, because the
kernel doesn't follow mounts on the umount() call, hence it will find
a non-mounted directory.

2 files changed, 6 insertions, 33 deletions

diff --git a/ChangeLog b/ChangeLog
index 6171675..1e9173f 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -8,9 +8,6 @@
 
 2011-01-31  Miklos Szeredi <miklos@szeredi.hu>
 
-	* In case of failure to add to /etc/mtab use same mountpoint for
-	cleanup as for mounting.  Reported by Marc Deslauriers
-
 	* fusermount: chdir to / before performing mount/umount
 
 	* fusermount: only allow mount and umount if util-linux supports
diff --git a/util/fusermount.c b/util/fusermount.c
index 94a5dbc..48f23fa 100644
--- a/util/fusermount.c
+++ b/util/fusermount.c
@@ -896,8 +896,7 @@ static int check_version(const char *dev)
 	return 0;
 }
 
-static int check_perm(const char **mntp, struct stat *stbuf,
-		      int *mountpoint_fd, int *isdir)
+static int check_perm(const char **mntp, struct stat *stbuf, int *mountpoint_fd)
 {
 	int res;
 	const char *mnt = *mntp;
@@ -915,17 +914,10 @@ static int check_perm(const char **mntp, struct stat *stbuf,
 		return 0;
 
 	if (S_ISDIR(stbuf->st_mode)) {
-		*isdir = 1;
-		*mountpoint_fd = open(mnt, O_RDONLY);
-		if (*mountpoint_fd == -1) {
-			fprintf(stderr, "%s: failed to open %s: %s\n",
-				progname, mnt, strerror(errno));
-			return -1;
-		}
-		res = fchdir(*mountpoint_fd);
+		res = chdir(mnt);
 		if (res == -1) {
 			fprintf(stderr,
-				"%s: failed to fchdir to mountpoint: %s\n",
+				"%s: failed to chdir to mountpoint: %s\n",
 				progname, strerror(errno));
 			return -1;
 		}
@@ -1050,7 +1042,6 @@ static int mount_fuse(const char *mnt, const char *opts)
 	char *mnt_opts = NULL;
 	const char *real_mnt = mnt;
 	int mountpoint_fd = -1;
-	int isdir = 0;
 
 	fd = open_fuse_device(&dev);
 	if (fd == -1)
@@ -1070,7 +1061,7 @@ static int mount_fuse(const char *mnt, const char *opts)
 
 	res = check_version(dev);
 	if (res != -1) {
-		res = check_perm(&real_mnt, &stbuf, &mountpoint_fd, &isdir);
+		res = check_perm(&real_mnt, &stbuf, &mountpoint_fd);
 		restore_privs();
 		if (res != -1)
 			res = do_mount(real_mnt, &type, stbuf.st_mode & S_IFMT,
@@ -1081,37 +1072,22 @@ static int mount_fuse(const char *mnt, const char *opts)
 
 	chdir("/");
 	if (mountpoint_fd != -1)
-		fcntl(mountpoint_fd, F_SETFD, FD_CLOEXEC);
+		close(mountpoint_fd);
 
 	if (res == -1) {
 		close(fd);
-		if (mountpoint_fd != -1)
-			close(mountpoint_fd);
 		return -1;
 	}
 
 	if (geteuid() == 0) {
 		res = add_mount(source, mnt, type, mnt_opts);
 		if (res == -1) {
-			if (isdir && mountpoint_fd != -1) {
-				res = fchdir(mountpoint_fd);
-				if (res == -1) {
-					close(mountpoint_fd);
-					close(fd);
-					return -1;
-				}
-			}
-			umount2(real_mnt, UMOUNT_DETACH); /* lazy umount */
-			if (mountpoint_fd != -1)
-				close(mountpoint_fd);
+			umount2(mnt, UMOUNT_DETACH); /* lazy umount */
 			close(fd);
 			return -1;
 		}
 	}
 
-	if (mountpoint_fd != -1)
-		close(mountpoint_fd);
-
 	free(source);
 	free(type);
 	free(mnt_opts);