multiplier suffixes are short, store them directly in struct suffix_mult
[oweals/busybox.git] / util-linux / mount.c
index ab20ab9fc506b53f5d525fe5da808e573aebbb69..7ee24ca143241bd2a9b586918d5d5a18718420f0 100644 (file)
@@ -9,10 +9,6 @@
  * Licensed under GPLv2 or later, see file LICENSE in this tarball for details.
  */
 
-/* todo:
- * bb_getopt_ulflags();
- */
-
 /* Design notes: There is no spec for mount.  Remind me to write one.
 
    mount_main() calls singlemount() which calls mount_it_now().
@@ -22,7 +18,7 @@
    mount_it_now() does the actual mount.
 */
 
-#include "busybox.h"
+#include "libbb.h"
 #include <mntent.h>
 
 /* Needed for nfs support only... */
 #include <rpc/pmap_clnt.h>
 
 
+#if defined(__dietlibc__)
+/* 16.12.2006, Sampo Kellomaki (sampo@iki.fi)
+ * dietlibc-0.30 does not have implementation of getmntent_r() */
+/* OTOH: why we use getmntent_r instead of getmntent? TODO... */
+struct mntent *getmntent_r(FILE* stream, struct mntent* result, char* buffer, int bufsize)
+{
+       /* *** XXX FIXME WARNING: This hack is NOT thread safe. --Sampo */
+       struct mntent* ment = getmntent(stream);
+       memcpy(result, ment, sizeof(struct mntent));
+       return result;
+}
+#endif
+
+
 // Not real flags, but we want to be able to check for this.
-#define MOUNT_NOAUTO    (1<<29)
-#define MOUNT_SWAP      (1<<30)
+enum {
+       MOUNT_USERS  = (1<<28)*ENABLE_DESKTOP,
+       MOUNT_NOAUTO = (1<<29),
+       MOUNT_SWAP   = (1<<30),
+};
+// TODO: more "user" flag compatibility.
+// "user" option (from mount manpage):
+// Only the user that mounted a filesystem can unmount it again.
+// If any user should be able to unmount, then use users instead of user
+// in the fstab line.  The owner option is similar to the user option,
+// with the restriction that the user must be the owner of the special file.
+// This may be useful e.g. for /dev/fd if a login script makes
+// the console user owner of this device.
 
 /* Standard mount options (from -o options or --options), with corresponding
  * flags */
 
 struct {
-       char *name;
+       const char *name;
        long flags;
 } static mount_options[] = {
        // MS_FLAGS set a bit.  ~MS_FLAGS disable that bit.  0 flags are NOPs.
@@ -54,9 +75,12 @@ struct {
 
        USE_FEATURE_MOUNT_FSTAB(
                {"defaults", 0},
-               {"quiet", 0},
-               {"noauto",MOUNT_NOAUTO},
-               {"swap",MOUNT_SWAP},
+               /* {"quiet", 0}, - do not filter out, vfat wants to see it */
+               {"noauto", MOUNT_NOAUTO},
+               {"sw", MOUNT_SWAP},
+               {"swap", MOUNT_SWAP},
+               USE_DESKTOP({"user",  MOUNT_USERS},)
+               USE_DESKTOP({"users", MOUNT_USERS},)
        )
 
        USE_FEATURE_MOUNT_FLAGS(
@@ -97,9 +121,8 @@ struct {
 };
 
 
-
 /* Append mount options to string */
-static void append_mount_options(char **oldopts, char *newopts)
+static void append_mount_options(char **oldopts, const char *newopts)
 {
        if (*oldopts && **oldopts) {
                /* do not insert options which are already there */
@@ -110,10 +133,11 @@ static void append_mount_options(char **oldopts, char *newopts)
                        if (p) len = p - newopts;
                        p = *oldopts;
                        while (1) {
-                               if (!strncmp(p,newopts,len) && (p[len]==',' || p[len]==0))
+                               if (!strncmp(p, newopts, len)
+                                && (p[len]==',' || p[len]==0))
                                        goto skip;
                                p = strchr(p,',');
-                               if(!p) break;
+                               if (!p) break;
                                p++;
                        }
                        p = xasprintf("%s,%.*s", *oldopts, len, newopts);
@@ -143,7 +167,7 @@ static int parse_mount_options(char *options, char **unrecognized)
                if (comma) *comma = 0;
 
                // Find this option in mount_options
-               for (i = 0; i < (sizeof(mount_options) / sizeof(*mount_options)); i++) {
+               for (i = 0; i < ARRAY_SIZE(mount_options); i++) {
                        if (!strcasecmp(mount_options[i].name, options)) {
                                long fl = mount_options[i].flags;
                                if (fl < 0) flags &= fl;
@@ -152,9 +176,7 @@ static int parse_mount_options(char *options, char **unrecognized)
                        }
                }
                // If unrecognized not NULL, append unrecognized mount options */
-               if (unrecognized
-                               && i == (sizeof(mount_options) / sizeof(*mount_options)))
-               {
+               if (unrecognized && i == ARRAY_SIZE(mount_options)) {
                        // Add it to strflags, to pass on to kernel
                        i = *unrecognized ? strlen(*unrecognized) : 0;
                        *unrecognized = xrealloc(*unrecognized, i+strlen(options)+2);
@@ -178,26 +200,27 @@ static int parse_mount_options(char *options, char **unrecognized)
 
 static llist_t *get_block_backed_filesystems(void)
 {
-       char *fs, *buf,
-                *filesystems[] = {"/etc/filesystems", "/proc/filesystems", 0};
+       static const char filesystems[2][sizeof("/proc/filesystems")] = {
+               "/etc/filesystems",
+               "/proc/filesystems",
+       };
+       char *fs, *buf;
        llist_t *list = 0;
        int i;
        FILE *f;
 
-       for (i = 0; filesystems[i]; i++) {
+       for (i = 0; i < 2; i++) {
                f = fopen(filesystems[i], "r");
                if (!f) continue;
 
-               for (fs = buf = 0; (fs = buf = bb_get_chomped_line_from_file(f));
-                       free(buf))
-               {
-                       if (!strncmp(buf,"nodev",5) && isspace(buf[5])) continue;
-
-                       while (isspace(*fs)) fs++;
-                       if (*fs=='#' || *fs=='*') continue;
-                       if (!*fs) continue;
+               while ((buf = xmalloc_getline(f)) != 0) {
+                       if (!strncmp(buf, "nodev", 5) && isspace(buf[5]))
+                               continue;
+                       fs = skip_whitespace(buf);
+                       if (*fs=='#' || *fs=='*' || !*fs) continue;
 
-                       llist_add_to_end(&list,xstrdup(fs));
+                       llist_add_to_end(&list, xstrdup(fs));
+                       free(buf);
                }
                if (ENABLE_FEATURE_CLEAN_UP) fclose(f);
        }
@@ -220,19 +243,17 @@ void delete_block_backed_filesystems(void);
 static int useMtab = 1;
 static int fakeIt;
 #else
-enum {
-       useMtab = 0,
-       fakeIt = 0,
-};
+#define useMtab 0
+#define fakeIt 0
 #endif
 
 // Perform actual mount of specific filesystem at specific location.
 // NB: mp->xxx fields may be trashed on exit
 static int mount_it_now(struct mntent *mp, int vfsflags, char *filteropts)
 {
-       int rc;
+       int rc = 0;
 
-       if (fakeIt) return 0;
+       if (fakeIt) goto mtab;
 
        // Mount, with fallback to read-only if necessary.
 
@@ -253,14 +274,16 @@ static int mount_it_now(struct mntent *mp, int vfsflags, char *filteropts)
 
        /* If the mount was successful, and we're maintaining an old-style
         * mtab file by hand, add the new entry to it now. */
-
+ mtab:
        if (ENABLE_FEATURE_MTAB_SUPPORT && useMtab && !rc && !(vfsflags & MS_REMOUNT)) {
                char *fsname;
                FILE *mountTable = setmntent(bb_path_mtab_file, "a+");
                int i;
 
-               if (!mountTable)
+               if (!mountTable) {
                        bb_error_msg("no %s",bb_path_mtab_file);
+                       goto ret;
+               }
 
                // Add vfs string flags
 
@@ -279,7 +302,7 @@ static int mount_it_now(struct mntent *mp, int vfsflags, char *filteropts)
                fsname = 0;
                if (!mp->mnt_type || !*mp->mnt_type) { /* bind mount */
                        mp->mnt_fsname = fsname = bb_simplify_path(mp->mnt_fsname);
-                       mp->mnt_type = "bind";
+                       mp->mnt_type = (char*)"bind";
                }
                mp->mnt_freq = mp->mnt_passno = 0;
 
@@ -292,7 +315,7 @@ static int mount_it_now(struct mntent *mp, int vfsflags, char *filteropts)
                        free(fsname);
                }
        }
-
+ ret:
        return rc;
 }
 
@@ -415,7 +438,7 @@ struct ppathcnf {
        short pc_name_max;
        short pc_path_max;
        short pc_pipe_buf;
-       u_char pc_vdisable;
+       uint8_t pc_vdisable;
        char pc_xxx;
        short pc_mask[2];
 };
@@ -663,6 +686,8 @@ get_mountport(struct sockaddr_in *server_addr,
        static struct pmap p = {0, 0, 0, 0};
 
        server_addr->sin_port = PMAPPORT;
+/* glibc 2.4 (still) has pmap_getmaps(struct sockaddr_in *).
+ * I understand it like "IPv6 for this is not 100% ready" */
        pmap = pmap_getmaps(server_addr);
 
        if (version > MAX_NFSPROT)
@@ -713,9 +738,9 @@ static int daemonize(void)
        dup2(fd, 0);
        dup2(fd, 1);
        dup2(fd, 2);
-       if (fd > 2) close(fd);
+       while (fd > 2) close(fd--);
        setsid();
-       openlog(bb_applet_name, LOG_PID, LOG_DAEMON);
+       openlog(applet_name, LOG_PID, LOG_DAEMON);
        logmode = LOGMODE_SYSLOG;
        return 1;
 }
@@ -860,103 +885,167 @@ static int nfsmount(struct mntent *mp, int vfsflags, char *filteropts)
        nfsvers = 0;
 
        /* parse options */
-
-       for (opt = strtok(filteropts, ","); opt; opt = strtok(NULL, ",")) {
+       if (filteropts) for (opt = strtok(filteropts, ","); opt; opt = strtok(NULL, ",")) {
                char *opteq = strchr(opt, '=');
                if (opteq) {
-                       int val = atoi(opteq + 1);
+                       static const char options[] =
+                               /* 0 */ "rsize\0"
+                               /* 1 */ "wsize\0"
+                               /* 2 */ "timeo\0"
+                               /* 3 */ "retrans\0"
+                               /* 4 */ "acregmin\0"
+                               /* 5 */ "acregmax\0"
+                               /* 6 */ "acdirmin\0"
+                               /* 7 */ "acdirmax\0"
+                               /* 8 */ "actimeo\0"
+                               /* 9 */ "retry\0"
+                               /* 10 */ "port\0"
+                               /* 11 */ "mountport\0"
+                               /* 12 */ "mounthost\0"
+                               /* 13 */ "mountprog\0"
+                               /* 14 */ "mountvers\0"
+                               /* 15 */ "nfsprog\0"
+                               /* 16 */ "nfsvers\0"
+                               /* 17 */ "vers\0"
+                               /* 18 */ "proto\0"
+                               /* 19 */ "namlen\0"
+                               /* 20 */ "addr\0";
+                       int val = xatoi_u(opteq + 1);
                        *opteq = '\0';
-                       if (!strcmp(opt, "rsize"))
+                       switch (index_in_strings(options, opt)) {
+                       case 0: // "rsize"
                                data.rsize = val;
-                       else if (!strcmp(opt, "wsize"))
+                               break;
+                       case 1: // "wsize"
                                data.wsize = val;
-                       else if (!strcmp(opt, "timeo"))
+                               break;
+                       case 2: // "timeo"
                                data.timeo = val;
-                       else if (!strcmp(opt, "retrans"))
+                               break;
+                       case 3: // "retrans"
                                data.retrans = val;
-                       else if (!strcmp(opt, "acregmin"))
+                               break;
+                       case 4: // "acregmin"
                                data.acregmin = val;
-                       else if (!strcmp(opt, "acregmax"))
+                               break;
+                       case 5: // "acregmax"
                                data.acregmax = val;
-                       else if (!strcmp(opt, "acdirmin"))
+                               break;
+                       case 6: // "acdirmin"
                                data.acdirmin = val;
-                       else if (!strcmp(opt, "acdirmax"))
+                               break;
+                       case 7: // "acdirmax"
                                data.acdirmax = val;
-                       else if (!strcmp(opt, "actimeo")) {
+                               break;
+                       case 8: // "actimeo"
                                data.acregmin = val;
                                data.acregmax = val;
                                data.acdirmin = val;
                                data.acdirmax = val;
-                       }
-                       else if (!strcmp(opt, "retry"))
+                               break;
+                       case 9: // "retry"
                                retry = val;
-                       else if (!strcmp(opt, "port"))
+                               break;
+                       case 10: // "port"
                                port = val;
-                       else if (!strcmp(opt, "mountport"))
+                               break;
+                       case 11: // "mountport"
                                mountport = val;
-                       else if (!strcmp(opt, "mounthost"))
+                               break;
+                       case 12: // "mounthost"
                                mounthost = xstrndup(opteq+1,
-                                                 strcspn(opteq+1," \t\n\r,"));
-                       else if (!strcmp(opt, "mountprog"))
+                                               strcspn(opteq+1," \t\n\r,"));
+                               break;
+                       case 13: // "mountprog"
                                mountprog = val;
-                       else if (!strcmp(opt, "mountvers"))
+                               break;
+                       case 14: // "mountvers"
                                mountvers = val;
-                       else if (!strcmp(opt, "nfsprog"))
+                               break;
+                       case 15: // "nfsprog"
                                nfsprog = val;
-                       else if (!strcmp(opt, "nfsvers") ||
-                                !strcmp(opt, "vers"))
+                               break;
+                       case 16: // "nfsvers"
+                       case 17: // "vers"
                                nfsvers = val;
-                       else if (!strcmp(opt, "proto")) {
+                               break;
+                       case 18: // "proto"
                                if (!strncmp(opteq+1, "tcp", 3))
                                        tcp = 1;
                                else if (!strncmp(opteq+1, "udp", 3))
                                        tcp = 0;
                                else
                                        bb_error_msg("warning: unrecognized proto= option");
-                       } else if (!strcmp(opt, "namlen")) {
+                               break;
+                       case 19: // "namlen"
                                if (nfs_mount_version >= 2)
                                        data.namlen = val;
                                else
                                        bb_error_msg("warning: option namlen is not supported\n");
-                       } else if (!strcmp(opt, "addr"))
-                               /* ignore */;
-                       else {
+                               break;
+                       case 20: // "addr" - ignore
+                               break;
+                       default:
                                bb_error_msg("unknown nfs mount parameter: %s=%d", opt, val);
                                goto fail;
                        }
                }
                else {
+                       static const char options[] =
+                               "bg\0"
+                               "fg\0"
+                               "soft\0"
+                               "hard\0"
+                               "intr\0"
+                               "posix\0"
+                               "cto\0"
+                               "ac\0"
+                               "tcp\0"
+                               "udp\0"
+                               "lock\0";
                        int val = 1;
                        if (!strncmp(opt, "no", 2)) {
                                val = 0;
                                opt += 2;
                        }
-                       if (!strcmp(opt, "bg"))
+                       switch (index_in_strings(options, opt)) {
+                       case 0: // "bg"
                                bg = val;
-                       else if (!strcmp(opt, "fg"))
+                               break;
+                       case 1: // "fg"
                                bg = !val;
-                       else if (!strcmp(opt, "soft"))
+                               break;
+                       case 2: // "soft"
                                soft = val;
-                       else if (!strcmp(opt, "hard"))
+                               break;
+                       case 3: // "hard"
                                soft = !val;
-                       else if (!strcmp(opt, "intr"))
+                               break;
+                       case 4: // "intr"
                                intr = val;
-                       else if (!strcmp(opt, "posix"))
+                               break;
+                       case 5: // "posix"
                                posix = val;
-                       else if (!strcmp(opt, "cto"))
+                               break;
+                       case 6: // "cto"
                                nocto = !val;
-                       else if (!strcmp(opt, "ac"))
+                               break;
+                       case 7: // "ac"
                                noac = !val;
-                       else if (!strcmp(opt, "tcp"))
+                               break;
+                       case 8: // "tcp"
                                tcp = val;
-                       else if (!strcmp(opt, "udp"))
+                               break;
+                       case 9: // "udp"
                                tcp = !val;
-                       else if (!strcmp(opt, "lock")) {
+                               break;
+                       case 10: // "lock"
                                if (nfs_mount_version >= 3)
                                        nolock = !val;
                                else
                                        bb_error_msg("warning: option nolock is not supported");
-                       } else {
+                               break;
+                       default:
                                bb_error_msg("unknown nfs mount option: %s%s", val ? "" : "no", opt);
                                goto fail;
                        }
@@ -1249,7 +1338,7 @@ prepare_kernel_data:
 
 do_mount: /* perform actual mount */
 
-       mp->mnt_type = "nfs";
+       mp->mnt_type = (char*)"nfs";
        retval = mount_it_now(mp, vfsflags, (char*)&data);
        goto ret;
 
@@ -1293,16 +1382,19 @@ static int singlemount(struct mntent *mp, int ignore_busy)
 
        // Treat fstype "auto" as unspecified.
 
-       if (mp->mnt_type && !strcmp(mp->mnt_type,"auto")) mp->mnt_type = 0;
+       if (mp->mnt_type && strcmp(mp->mnt_type,"auto") == 0)
+               mp->mnt_type = 0;
 
        // Might this be an CIFS filesystem?
 
-       if (ENABLE_FEATURE_MOUNT_CIFS &&
-               (!mp->mnt_type || !strcmp(mp->mnt_type,"cifs")) &&
-               (mp->mnt_fsname[0]==mp->mnt_fsname[1] && (mp->mnt_fsname[0]=='/' || mp->mnt_fsname[0]=='\\')))
-       {
-               struct hostent *he;
-               char ip[32], *s;
+       if (ENABLE_FEATURE_MOUNT_CIFS
+        && (!mp->mnt_type || strcmp(mp->mnt_type,"cifs") == 0)
+        && (mp->mnt_fsname[0]=='/' || mp->mnt_fsname[0]=='\\')
+        && mp->mnt_fsname[0]==mp->mnt_fsname[1]
+       ) {
+               len_and_sockaddr *lsa;
+               char *ip, *dotted;
+               char *s;
 
                rc = 1;
                // Replace '/' with '\' and verify that unc points to "//server/share".
@@ -1313,47 +1405,55 @@ static int singlemount(struct mntent *mp, int ignore_busy)
                // get server IP
 
                s = strrchr(mp->mnt_fsname, '\\');
-               if (s == mp->mnt_fsname+1) goto report_error;
-               *s = 0;
-               he = gethostbyname(mp->mnt_fsname+2);
+               if (s <= mp->mnt_fsname+1) goto report_error;
+               *s = '\0';
+               lsa = host2sockaddr(mp->mnt_fsname+2, 0);
                *s = '\\';
-               if (!he) goto report_error;
+               if (!lsa) goto report_error;
 
-               // Insert ip=... option into string flags.  (NOTE: Add IPv6 support.)
+               // insert ip=... option into string flags.
 
-               sprintf(ip, "ip=%d.%d.%d.%d", he->h_addr[0], he->h_addr[1],
-                               he->h_addr[2], he->h_addr[3]);
+               dotted = xmalloc_sockaddr2dotted_noport(&lsa->sa, lsa->len);
+               ip = xasprintf("ip=%s", dotted);
                parse_mount_options(ip, &filteropts);
 
                // compose new unc '\\server-ip\share'
+               // (s => slash after hostname)
 
-               mp->mnt_fsname = xasprintf("\\\\%s%s", ip+3,
-                                       strchr(mp->mnt_fsname+2,'\\'));
+               mp->mnt_fsname = xasprintf("\\\\%s%s", dotted, s);
 
                // lock is required
                vfsflags |= MS_MANDLOCK;
 
-               mp->mnt_type = "cifs";
+               mp->mnt_type = (char*)"cifs";
                rc = mount_it_now(mp, vfsflags, filteropts);
-               if (ENABLE_FEATURE_CLEAN_UP) free(mp->mnt_fsname);
+               if (ENABLE_FEATURE_CLEAN_UP) {
+                       free(mp->mnt_fsname);
+                       free(ip);
+                       free(dotted);
+                       free(lsa);
+               }
                goto report_error;
        }
 
        // Might this be an NFS filesystem?
 
-       if (ENABLE_FEATURE_MOUNT_NFS &&
-               (!mp->mnt_type || !strcmp(mp->mnt_type,"nfs")) &&
-               strchr(mp->mnt_fsname, ':') != NULL)
-       {
+       if (ENABLE_FEATURE_MOUNT_NFS
+        && (!mp->mnt_type || !strcmp(mp->mnt_type,"nfs"))
+        && strchr(mp->mnt_fsname, ':') != NULL
+       {
                rc = nfsmount(mp, vfsflags, filteropts);
                goto report_error;
        }
 
        // Look at the file.  (Not found isn't a failure for remount, or for
        // a synthetic filesystem like proc or sysfs.)
+       // (We use stat, not lstat, in order to allow
+       // mount symlink_to_file_or_blkdev dir)
 
-       if (!lstat(mp->mnt_fsname, &st) && !(vfsflags & (MS_REMOUNT | MS_BIND | MS_MOVE)))
-       {
+       if (!stat(mp->mnt_fsname, &st)
+        && !(vfsflags & (MS_REMOUNT | MS_BIND | MS_MOVE))
+       ) {
                // Do we need to allocate a loopback device for it?
 
                if (ENABLE_FEATURE_MOUNT_LOOP && S_ISREG(st.st_mode)) {
@@ -1381,10 +1481,9 @@ static int singlemount(struct mntent *mp, int ignore_busy)
 
        if (mp->mnt_type || (vfsflags & (MS_REMOUNT | MS_BIND | MS_MOVE)))
                rc = mount_it_now(mp, vfsflags, filteropts);
-
-       // Loop through filesystem types until mount succeeds or we run out
-
        else {
+               // Loop through filesystem types until mount succeeds
+               // or we run out
 
                /* Initialize list of block backed filesystems.  This has to be
                 * done here so that during "mount -a", mounts after /proc shows up
@@ -1413,8 +1512,9 @@ static int singlemount(struct mntent *mp, int ignore_busy)
                }
        }
 
-report_error:
-       if (ENABLE_FEATURE_CLEAN_UP) free(filteropts);
+ report_error:
+       if (ENABLE_FEATURE_CLEAN_UP)
+               free(filteropts);
 
        if (rc && errno == EBUSY && ignore_busy) rc = 0;
        if (rc < 0)
@@ -1427,44 +1527,46 @@ report_error:
 // Parse options, if necessary parse fstab/mtab, and call singlemount for
 // each directory to be mounted.
 
+const char must_be_root[] = "you must be root";
+
+int mount_main(int argc, char **argv);
 int mount_main(int argc, char **argv)
 {
-       enum { OPT_ALL = 0x8 };
+       enum { OPT_ALL = 0x10 };
 
        char *cmdopts = xstrdup(""), *fstype=0, *storage_path=0;
        char *opt_o;
        const char *fstabname;
        FILE *fstab;
        int i, j, rc = 0;
-       unsigned long opt;
+       unsigned opt;
        struct mntent mtpair[2], *mtcur = mtpair;
+       SKIP_DESKTOP(const int nonroot = 0;)
+       USE_DESKTOP( int nonroot = (getuid() != 0);)
 
        /* parse long options, like --bind and --move.  Note that -o option
         * and --option are synonymous.  Yes, this means --remount,rw works. */
 
        for (i = j = 0; i < argc; i++) {
                if (argv[i][0] == '-' && argv[i][1] == '-') {
-                       append_mount_options(&cmdopts,argv[i]+2);
+                       append_mount_options(&cmdopts, argv[i]+2);
                } else argv[j++] = argv[i];
        }
+       argv[j] = 0;
        argc = j;
 
        // Parse remaining options
 
-       opt = bb_getopt_ulflags(argc, argv, "o:t:rwavnf", &opt_o, &fstype);
-       if (opt & 1) // -o
-               append_mount_options(&cmdopts, opt_o);
-       //if (opt & 1) // -t
-       if (opt & 2) // -r
-               append_mount_options(&cmdopts, "ro");
-       if (opt & 4) // -w
-               append_mount_options(&cmdopts, "rw");
-       //if (opt & 8) // -a
-       if (opt & 0x10) // -n
-               USE_FEATURE_MTAB_SUPPORT(useMtab = FALSE);
-       if (opt & 0x20) // -f
-               USE_FEATURE_MTAB_SUPPORT(fakeIt = FALSE);
-       //if (opt & 0x40) // ignore -v
+       opt = getopt32(argc, argv, "o:t:rwanfvs", &opt_o, &fstype);
+       if (opt & 0x1) append_mount_options(&cmdopts, opt_o); // -o
+       //if (opt & 0x2) // -t
+       if (opt & 0x4) append_mount_options(&cmdopts, "ro"); // -r
+       if (opt & 0x8) append_mount_options(&cmdopts, "rw"); // -w
+       //if (opt & 0x10) // -a
+       if (opt & 0x20) USE_FEATURE_MTAB_SUPPORT(useMtab = 0); // -n
+       if (opt & 0x40) USE_FEATURE_MTAB_SUPPORT(fakeIt = 1); // -f
+       //if (opt & 0x80) // -v: verbose (ignore)
+       //if (opt & 0x100) // -s: sloppy (ignore)
        argv += optind;
        argc -= optind;
 
@@ -1478,13 +1580,14 @@ int mount_main(int argc, char **argv)
                if (!(opt & OPT_ALL)) {
                        FILE *mountTable = setmntent(bb_path_mtab_file, "r");
 
-                       if (!mountTable) bb_error_msg_and_die("no %s",bb_path_mtab_file);
+                       if (!mountTable) bb_error_msg_and_die("no %s", bb_path_mtab_file);
 
-                       while (getmntent_r(mountTable,mtpair,bb_common_bufsiz1,
+                       while (getmntent_r(mountTable, mtpair, bb_common_bufsiz1,
                                                                sizeof(bb_common_bufsiz1)))
                        {
-                               // Don't show rootfs.
-                               if (!strcmp(mtpair->mnt_fsname, "rootfs")) continue;
+                               // Don't show rootfs. FIXME: why??
+                               // util-linux 2.12a happily shows rootfs...
+                               //if (!strcmp(mtpair->mnt_fsname, "rootfs")) continue;
 
                                if (!fstype || !strcmp(mtpair->mnt_type, fstype))
                                        printf("%s on %s type %s (%s)\n", mtpair->mnt_fsname,
@@ -1501,6 +1604,8 @@ int mount_main(int argc, char **argv)
        // argument when we get it.
 
        if (argc == 2) {
+               if (nonroot)
+                       bb_error_msg_and_die(must_be_root);
                mtpair->mnt_fsname = argv[0];
                mtpair->mnt_dir = argv[1];
                mtpair->mnt_type = fstype;
@@ -1509,11 +1614,15 @@ int mount_main(int argc, char **argv)
                goto clean_up;
        }
 
+       i = parse_mount_options(cmdopts, 0);
+       if (nonroot && (i & ~MS_SILENT)) // Non-root users cannot specify flags
+               bb_error_msg_and_die(must_be_root);
+
        // If we have a shared subtree flag, don't worry about fstab or mtab.
-       i = parse_mount_options(cmdopts,0);
-       if (ENABLE_FEATURE_MOUNT_FLAGS &&
-                       (i & (MS_SHARED | MS_PRIVATE | MS_SLAVE | MS_UNBINDABLE )))
-       {
+
+       if (ENABLE_FEATURE_MOUNT_FLAGS
+        && (i & (MS_SHARED | MS_PRIVATE | MS_SLAVE | MS_UNBINDABLE))
+       {
                rc = mount("", argv[0], "", i, "");
                if (rc) bb_perror_msg_and_die("%s", argv[0]);
                goto clean_up;
@@ -1521,17 +1630,17 @@ int mount_main(int argc, char **argv)
 
        // Open either fstab or mtab
 
-       if (parse_mount_options(cmdopts,0) & MS_REMOUNT)
+       fstabname = "/etc/fstab";
+       if (i & MS_REMOUNT) {
                fstabname = bb_path_mtab_file;
-       else fstabname="/etc/fstab";
-
-       fstab = setmntent(fstabname,"r");
+       }
+       fstab = setmntent(fstabname, "r");
        if (!fstab)
                bb_perror_msg_and_die("cannot read %s", fstabname);
 
        // Loop through entries until we find what we're looking for.
 
-       memset(mtpair,0,sizeof(mtpair));
+       memset(mtpair, 0, sizeof(mtpair));
        for (;;) {
                struct mntent *mtnext = (mtcur==mtpair ? mtpair+1 : mtpair);
 
@@ -1551,11 +1660,17 @@ int mount_main(int argc, char **argv)
                                        bb_error_msg_and_die("can't find %s in %s",
                                                argv[0], fstabname);
 
+                               mtcur = mtnext;
+                               if (nonroot) {
+                                       // fstab must have "users" or "user"
+                                       if (!(parse_mount_options(mtcur->mnt_opts, 0) & MOUNT_USERS))
+                                               bb_error_msg_and_die(must_be_root);
+                               }
+
                                // Mount the last thing we found.
 
-                               mtcur = mtnext;
                                mtcur->mnt_opts = xstrdup(mtcur->mnt_opts);
-                               append_mount_options(&(mtcur->mnt_opts),cmdopts);
+                               append_mount_options(&(mtcur->mnt_opts), cmdopts);
                                rc = singlemount(mtcur, 0);
                                free(mtcur->mnt_opts);
                        }
@@ -1570,10 +1685,10 @@ int mount_main(int argc, char **argv)
 
                        // Is this what we're looking for?
 
-                       if (strcmp(argv[0],mtcur->mnt_fsname) &&
-                          strcmp(storage_path,mtcur->mnt_fsname) &&
-                          strcmp(argv[0],mtcur->mnt_dir) &&
-                          strcmp(storage_path,mtcur->mnt_dir)) continue;
+                       if (strcmp(argv[0], mtcur->mnt_fsname) &&
+                          strcmp(storage_path, mtcur->mnt_fsname) &&
+                          strcmp(argv[0], mtcur->mnt_dir) &&
+                          strcmp(storage_path, mtcur->mnt_dir)) continue;
 
                        // Remember this entry.  Something later may have overmounted
                        // it, and we want the _last_ match.
@@ -1583,21 +1698,29 @@ int mount_main(int argc, char **argv)
                // If we're mounting all.
 
                } else {
-
                        // Do we need to match a filesystem type?
-                       if (fstype && strcmp(mtcur->mnt_type,fstype)) continue;
+                       if (fstype && match_fstype(mtcur, fstype)) continue;
 
                        // Skip noauto and swap anyway.
 
-                       if (parse_mount_options(mtcur->mnt_opts,0)
+                       if (parse_mount_options(mtcur->mnt_opts, 0)
                                & (MOUNT_NOAUTO | MOUNT_SWAP)) continue;
 
+                       // No, mount -a won't mount anything,
+                       // even user mounts, for mere humans.
+
+                       if (nonroot)
+                               bb_error_msg_and_die(must_be_root);
+
                        // Mount this thing.
 
+                       // NFS mounts want this to be xrealloc-able
+                       mtcur->mnt_opts = xstrdup(mtcur->mnt_opts);
                        if (singlemount(mtcur, 1)) {
                                /* Count number of failed mounts */
                                rc++;
                        }
+                       free(mtcur->mnt_opts);
                }
        }
        if (ENABLE_FEATURE_CLEAN_UP) endmntent(fstab);