]> git.decadent.org.uk Git - nfs-utils.git/blobdiff - utils/mount/mount.c
mount.nfs: /bin/mount already handles --bind & friends
[nfs-utils.git] / utils / mount / mount.c
index c163922b7d7159a59a4bbde0d4ba45fe9c3dea95..24ba394913a5f126eb56a54af780376cc69eecd8 100644 (file)
@@ -43,7 +43,6 @@
 char *progname;
 int nomtab;
 int verbose;
-int mounttype;
 int sloppy;
 
 static struct option longopts[] = {
@@ -57,14 +56,6 @@ static struct option longopts[] = {
   { "read-write", 0, 0, 'w' },
   { "rw", 0, 0, 'w' },
   { "options", 1, 0, 'o' },
-  { "nfsvers", 1, 0, 't' },
-  { "bind", 0, 0, 128 },
-  { "replace", 0, 0, 129 },
-  { "after", 0, 0, 130 },
-  { "before", 0, 0, 131 },
-  { "over", 0, 0, 132 },
-  { "move", 0, 0, 133 },
-  { "rbind", 0, 0, 135 },
   { NULL, 0, 0, 0 }
 };
 
@@ -185,6 +176,7 @@ int add_mtab(char *fsname, char *mount_point, char *fstype, int flags, char *opt
        lock_mtab();
 
         if ((mtab = setmntent(MOUNTED, "a+")) == NULL) {
+               unlock_mtab();
                fprintf(stderr, "Can't open " MOUNTED);
                return 1;
        }
@@ -217,17 +209,18 @@ int do_mount_syscall(char *spec, char *node, char *type, int flags, void *data)
 
 void mount_usage()
 {
-       printf("usage: %s remotetarget dir [-rvVwfnh] [-t version] [-o nfsoptions]\n", progname);
-       printf("options:\n\t-r\t\tMount file system readonly\n");
+       printf("usage: %s remotetarget dir [-rvVwfnh] [-o nfsoptions]\n",
+               progname);
+       printf("options:\n");
+       printf("\t-r\t\tMount file system readonly\n");
        printf("\t-v\t\tVerbose\n");
        printf("\t-V\t\tPrint version\n");
        printf("\t-w\t\tMount file system read-write\n");
-       printf("\t-f\t\tFake mount, don't actually mount\n");
+       printf("\t-f\t\tFake mount, do not actually mount\n");
        printf("\t-n\t\tDo not update /etc/mtab\n");
        printf("\t-s\t\tTolerate sloppy mount options rather than failing.\n");
        printf("\t-h\t\tPrint this help\n");
-       printf("\tversion\t\tnfs4 - NFS version 4, nfs - older NFS version supported\n");
-       printf("\tnfsoptions\tRefer mount.nfs(8) or nfs(5)\n\n");
+       printf("\tnfsoptions\tRefer to mount.nfs(8) or nfs(5)\n\n");
 }
 
 static inline void
@@ -381,24 +374,19 @@ static int start_statd()
 
 int main(int argc, char *argv[])
 {
-       int c, flags = 0, nfs_mount_vers = 0, mnt_err = 1, fake = 0;
-       char *spec, *mount_point, *extra_opts = NULL;
-       char *mount_opts = NULL, *p;
+       int c, flags = 0, mnt_err = 1, fake = 0;
+       char *spec, *mount_point, *fs_type = "nfs";
+       char *extra_opts = NULL, *mount_opts = NULL;
        uid_t uid = getuid();
 
-       progname = argv[0];
-       if (!progname)
-               exit(2);
-
-       if ((p = strrchr(progname, '/')) != NULL)
-               progname = p+1;
+       progname = basename(argv[0]);
 
        if(!strncmp(progname, "umount", strlen("umount"))) {
                if(argc < 2) {
                        umount_usage();
                        exit(1);
                }
-               exit(nfsumount(argc, argv) ? 0 : 1);
+               exit(nfsumount(argc, argv));
        }
 
        if(argv[1] && argv[1][0] == '-') {
@@ -418,23 +406,12 @@ int main(int argc, char *argv[])
        mount_point = argv[2];
 
        argv[2] = argv[0]; /* so that getopt error messages are correct */
-       while ((c = getopt_long (argc - 2, argv + 2, "rt:vVwfno:hs",
+       while ((c = getopt_long(argc - 2, argv + 2, "rvVwfno:hs",
                                longopts, NULL)) != -1) {
                switch (c) {
                case 'r':
                        flags |= MS_RDONLY;
                        break;
-               case 't':
-                       if (strcmp(optarg, "nfs4") == 0)
-                               nfs_mount_vers = 4;
-                       else if (strcmp(optarg, "nfs") == 0)
-                               nfs_mount_vers = 0;
-                       else {
-                               fprintf(stderr, "%s: unknown filesystem type: %s\n",
-                                       progname, optarg);
-                               exit(1);
-                       }
-                       break;
                case 'v':
                        ++verbose;
                        break;
@@ -459,27 +436,6 @@ int main(int argc, char *argv[])
                case 's':
                        ++sloppy;
                        break;
-               case 128: /* bind */
-                       mounttype = MS_BIND;
-                       break;
-               case 129: /* replace */
-                       mounttype = MS_REPLACE;
-                       break;
-               case 130: /* after */
-                       mounttype = MS_AFTER;
-                       break;
-               case 131: /* before */
-                       mounttype = MS_BEFORE;
-                       break;
-               case 132: /* over */
-                       mounttype = MS_OVER;
-                       break;
-               case 133: /* move */
-                       mounttype = MS_MOVE;
-                       break;
-               case 135: /* rbind */
-                       mounttype = MS_BIND | MS_REC;
-                       break;
                case 'h':
                default:
                        mount_usage();
@@ -493,29 +449,31 @@ int main(int argc, char *argv[])
        }
 
        if (strcmp(progname, "mount.nfs4") == 0)
-               nfs_mount_vers = 4;
+               fs_type = "nfs4";
 
+       /*
+        * If a non-root user is attempting to mount, make sure the
+        * user's requested options match the options specified in
+        * /etc/fstab; otherwise, don't allow the mount.
+        */
        if (uid != 0) {
-               /* don't even think about it unless options exactly
-                * match fstab
-                */
                struct mntentchn *mc;
 
                if ((mc = getfsfile(mount_point)) == NULL ||
                    strcmp(mc->m.mnt_fsname, spec) != 0 ||
-                   strcmp(mc->m.mnt_type, (nfs_mount_vers == 4 ? "nfs4":"nfs")) != 0
-                   ) {
-                       fprintf(stderr, "%s: permission died - no match for fstab\n",
-                               progname);
+                   strcmp(mc->m.mnt_type, fs_type) != 0) {
+                       fprintf(stderr, "%s: permission denied: no match for %s "
+                               "found in /etc/fstab\n", progname, mount_point);
                        exit(1);
                }
-               /* 'mount' munges the options from fstab before passing them
+
+               /*
+                * 'mount' munges the options from fstab before passing them
                 * to us, so it is non-trivial to test that we have the correct
                 * set of options and we don't want to trust what the user
-                * gave us, so just take whatever is in fstab
+                * gave us, so just take whatever is in /etc/fstab.
                 */
                mount_opts = strdup(mc->m.mnt_opts);
-               mounttype = 0;
        }
 
        mount_point = canonicalize(mount_point);
@@ -538,7 +496,7 @@ int main(int argc, char *argv[])
        if (chk_mountpoint(mount_point))
                exit(EX_FAIL);
 
-       if (nfs_mount_vers == 4)
+       if (strcmp(fs_type, "nfs4") == 0)
                mnt_err = nfs4mount(spec, mount_point, &flags, &extra_opts, &mount_opts, 0);
        else {
                int need_statd = 0;
@@ -562,8 +520,7 @@ int main(int argc, char *argv[])
                exit(EX_FAIL);
 
        if (!fake) {
-               mnt_err = do_mount_syscall(spec, mount_point,
-                                          nfs_mount_vers == 4 ? "nfs4" : "nfs",
+               mnt_err = do_mount_syscall(spec, mount_point, fs_type,
                                           flags & ~(MS_USER|MS_USERS) ,
                                           mount_opts);
 
@@ -574,8 +531,7 @@ int main(int argc, char *argv[])
        }
 
        if (!nomtab)
-               add_mtab(spec, mount_point,
-                        nfs_mount_vers == 4 ? "nfs4" : "nfs",
+               add_mtab(spec, mount_point, fs_type,
                         flags, extra_opts, 0, 0);
 
        return 0;