Try to make a "type-punned pointer" warning go away for somebody on the

[oweals/busybox.git] / util-linux / mount.c

diff --git a/util-linux/mount.c b/util-linux/mount.c
index fecf81606d32795d647088ab43988c92b3fa1a26..f665a08758e2377809a04ba726448bba79c777f2 100644 (file)
--- a/util-linux/mount.c
+++ b/util-linux/mount.c
@@ -23,13 +23,7 @@
 */
 
 #include "busybox.h"
-#include <unistd.h>
-#include <errno.h>
-#include <string.h>
 #include <mntent.h>
-#include <ctype.h>
-#include <fcntl.h>             // for CONFIG_FEATURE_MOUNT_LOOP
-#include <sys/ioctl.h>  // for CONFIG_FEATURE_MOUNT_LOOP
 
 // These two aren't always defined in old headers
 #ifndef MS_BIND
@@ -89,12 +83,12 @@ struct {
 static void append_mount_options(char **oldopts, char *newopts)
 {
        if(*oldopts && **oldopts) {
-               char *temp=bb_xasprintf("%s,%s",*oldopts,newopts);
+               char *temp=xasprintf("%s,%s",*oldopts,newopts);
                free(*oldopts);
                *oldopts=temp;
        } else {
                if (ENABLE_FEATURE_CLEAN_UP) free(*oldopts);
-               *oldopts = bb_xstrdup(newopts);
+               *oldopts = xstrdup(newopts);
        }
 }
 
@@ -165,7 +159,7 @@ static llist_t *get_block_backed_filesystems(void)
                        if(*fs=='#' || *fs=='*') continue;
                        if(!*fs) continue;
 
-                       llist_add_to_end(&list,bb_xstrdup(fs));
+                       llist_add_to_end(&list,xstrdup(fs));
                }
                if (ENABLE_FEATURE_CLEAN_UP) fclose(f);
        }
@@ -256,9 +250,9 @@ static int mount_it_now(struct mntent *mp, int vfsflags, char *filteropts)
 // Mount one directory.  Handles NFS, loopback, autobind, and filesystem type
 // detection.  Returns 0 for success, nonzero for failure.
 
-static int singlemount(struct mntent *mp)
+static int singlemount(struct mntent *mp, int ignore_busy)
 {
-       int rc = 1, vfsflags;
+       int rc = -1, vfsflags;
        char *loopFile = 0, *filteropts = 0;
        llist_t *fl = 0;
        struct stat st;
@@ -277,14 +271,14 @@ static int singlemount(struct mntent *mp)
        {
                if (nfsmount(mp->mnt_fsname, mp->mnt_dir, &vfsflags, &filteropts, 1)) {
                        bb_perror_msg("nfsmount failed");
-                       return 1;
+                       goto report_error;
                } else {
                        // Strangely enough, nfsmount() doesn't actually mount() anything.
                        mp->mnt_type = "nfs";
                        rc = mount_it_now(mp, vfsflags, filteropts);
                        if (ENABLE_FEATURE_CLEAN_UP) free(filteropts);
                        
-                       return rc;
+                       goto report_error;
                }
        }
 
@@ -354,18 +348,22 @@ static int singlemount(struct mntent *mp)
                        free(mp->mnt_fsname);
                }
        }
+report_error:
+       if (rc && errno == EBUSY && ignore_busy) rc = 0;
+       if (rc < 0)
+               bb_perror_msg("Mounting %s on %s failed", mp->mnt_fsname, mp->mnt_dir);
+
        return rc;
 }
 
-
 // Parse options, if necessary parse fstab/mtab, and call singlemount for
 // each directory to be mounted.
 
 int mount_main(int argc, char **argv)
 {
-       char *cmdopts = bb_xstrdup(""), *fstabname, *fstype=0, *storage_path=0;
+       char *cmdopts = xstrdup(""), *fstabname, *fstype=0, *storage_path=0;
        FILE *fstab;
-       int i, opt, all = FALSE, rc = 1;
+       int i, opt, all = FALSE, rc = 0;
        struct mntent mtpair[2], *mtcur = mtpair;
 
        /* parse long options, like --bind and --move.  Note that -o option
@@ -447,7 +445,7 @@ int mount_main(int argc, char **argv)
                mtpair->mnt_dir = argv[optind+1];
                mtpair->mnt_type = fstype;
                mtpair->mnt_opts = cmdopts;
-               rc = singlemount(mtpair);
+               rc = singlemount(mtpair, 0);
                goto clean_up;
        }
 
@@ -489,12 +487,12 @@ int mount_main(int argc, char **argv)
                                // Mount the last thing we found.
 
                                mtcur = mtnext;
-                               mtcur->mnt_opts=bb_xstrdup(mtcur->mnt_opts);
+                               mtcur->mnt_opts = xstrdup(mtcur->mnt_opts);
                                append_mount_options(&(mtcur->mnt_opts),cmdopts);
-                               rc = singlemount(mtcur);
+                               rc = singlemount(mtcur, 0);
                                free(mtcur->mnt_opts);
                        }
-                       break;
+                       goto clean_up;
                }
 
                /* If we're trying to mount something specific and this isn't it,
@@ -529,14 +527,9 @@ int mount_main(int argc, char **argv)
 
                        // Mount this thing.
 
-                       rc = singlemount(mtcur);
-                       if(rc) {
-                               // Don't whine about already mounted fs when mounting all.
-                               // Note: we should probably change return value to indicate 
-                               // failure, without causing a duplicate error message.
-                               if (errno != EBUSY) bb_perror_msg("Mounting %s on %s failed",
-                                               mtcur->mnt_fsname, mtcur->mnt_dir);
-                               rc = 0;
+                       if (singlemount(mtcur, 1)) {
+                               /* Count number of failed mounts */
+                               rc++;
                        }
                }
        }
@@ -549,9 +542,5 @@ clean_up:
                free(cmdopts);
        }
 
-       if(rc)
-               bb_perror_msg("Mounting %s on %s failed",
-                               mtcur->mnt_fsname, mtcur->mnt_dir);
-
        return rc;
 }