losetup: implement -r option. Closes 4033.
authorDenys Vlasenko <vda.linux@googlemail.com>
Mon, 12 Sep 2011 00:13:47 +0000 (02:13 +0200)
committerDenys Vlasenko <vda.linux@googlemail.com>
Mon, 12 Sep 2011 06:30:16 +0000 (08:30 +0200)
function                                             old     new   delta
packed_usage                                       28595   28633     +38
losetup_main                                         285     290      +5
singlemount                                          906     908      +2
set_loop                                             674     672      -2

Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
include/libbb.h
libbb/loop.c
util-linux/losetup.c
util-linux/mount.c

index 21cbe1cac077500e71012afade852379d4130b34..feae85259b65ef60cd0bdac6e014456a677966fa 100644 (file)
@@ -1164,7 +1164,7 @@ extern int del_loop(const char *device) FAST_FUNC;
 /* If *devname is not NULL, use that name, otherwise try to find free one,
  * malloc and return it in *devname.
  * return value: 1: read-only loopdev was setup, 0: rw, < 0: error */
-extern int set_loop(char **devname, const char *file, unsigned long long offset) FAST_FUNC;
+extern int set_loop(char **devname, const char *file, unsigned long long offset, int ro) FAST_FUNC;
 
 /* Like bb_ask below, but asks on stdin with no timeout.  */
 char *bb_ask_stdin(const char * prompt) FAST_FUNC;
index b798932faf66aa0e68a8bb4095f9daf74dc26b2b..b3a520848181ab7abfe4bf97979c81bceffa8198 100644 (file)
@@ -84,7 +84,7 @@ int FAST_FUNC del_loop(const char *device)
    search will re-use an existing loop device already bound to that
    file/offset if it finds one.
  */
-int FAST_FUNC set_loop(char **device, const char *file, unsigned long long offset)
+int FAST_FUNC set_loop(char **device, const char *file, unsigned long long offset, int ro)
 {
        char dev[LOOP_NAMESIZE];
        char *try;
@@ -93,11 +93,13 @@ int FAST_FUNC set_loop(char **device, const char *file, unsigned long long offse
        int i, dfd, ffd, mode, rc = -1;
 
        /* Open the file.  Barf if this doesn't work.  */
-       mode = O_RDWR;
+       mode = ro ? O_RDONLY : O_RDWR;
        ffd = open(file, mode);
        if (ffd < 0) {
-               mode = O_RDONLY;
-               ffd = open(file, mode);
+               if (mode != O_RDONLY) {
+                       mode = O_RDONLY;
+                       ffd = open(file, mode);
+               }
                if (ffd < 0)
                        return -errno;
        }
index 9b7c49f503599ba07dace4fe9f1a21e595ddb9ab..21108d0bfb372fced9e0925ebd1a0037b12c21d4 100644 (file)
@@ -8,11 +8,12 @@
  */
 
 //usage:#define losetup_trivial_usage
-//usage:       "[-o OFS] LOOPDEV FILE - associate loop devices\n"
+//usage:       "[-r] [-o OFS] LOOPDEV FILE - associate loop devices\n"
 //usage:       "       losetup -d LOOPDEV - disassociate\n"
 //usage:       "       losetup [-f] - show"
 //usage:#define losetup_full_usage "\n\n"
 //usage:       "       -o OFS  Start OFS bytes into FILE"
+//usage:     "\n       -r      Read-only"
 //usage:     "\n       -f      Show first free loop device"
 //usage:
 //usage:#define losetup_notes_usage
@@ -37,11 +38,12 @@ int losetup_main(int argc UNUSED_PARAM, char **argv)
                OPT_d = (1 << 0),
                OPT_o = (1 << 1),
                OPT_f = (1 << 2),
+               OPT_r = (1 << 3), /* must be last */
        };
 
-       /* max 2 args, all opts are mutually exclusive */
+       /* max 2 args, -d,-o,-f opts are mutually exclusive */
        opt_complementary = "?2:d--of:o--df:f--do";
-       opt = getopt32(argv, "do:f", &opt_o);
+       opt = getopt32(argv, "do:fr", &opt_o);
        argv += optind;
 
        if (opt == OPT_o)
@@ -63,12 +65,12 @@ int losetup_main(int argc UNUSED_PARAM, char **argv)
                        bb_show_usage();
 
                if (argv[1]) {
-                       /* [-o OFS] BLOCKDEV FILE */
-                       if (set_loop(&argv[0], argv[1], offset) < 0)
+                       /* [-r] [-o OFS] BLOCKDEV FILE */
+                       if (set_loop(&argv[0], argv[1], offset, (opt / OPT_r)) < 0)
                                bb_simple_perror_msg_and_die(argv[0]);
                        return EXIT_SUCCESS;
                }
-               /* [-o OFS] BLOCKDEV */
+               /* [-r] [-o OFS] BLOCKDEV */
                s = query_loop(argv[0]);
                if (!s)
                        bb_simple_perror_msg_and_die(argv[0]);
@@ -78,7 +80,7 @@ int losetup_main(int argc UNUSED_PARAM, char **argv)
                return EXIT_SUCCESS;
        }
 
-       /* [-o OFS|-f] with no params */
+       /* [-r] [-o OFS|-f] with no params */
        n = 0;
        while (1) {
                char *s;
index 05e532cda69e326cd41d0db7857b947f1407e2f3..b51ab1782e44387003659ba19950be208c996f18 100644 (file)
@@ -1809,7 +1809,7 @@ static int singlemount(struct mntent *mp, int ignore_busy)
                if (ENABLE_FEATURE_MOUNT_LOOP && S_ISREG(st.st_mode)) {
                        loopFile = bb_simplify_path(mp->mnt_fsname);
                        mp->mnt_fsname = NULL; // will receive malloced loop dev name
-                       if (set_loop(&mp->mnt_fsname, loopFile, 0) < 0) {
+                       if (set_loop(&mp->mnt_fsname, loopFile, 0, /*ro:*/ 0) < 0) {
                                if (errno == EPERM || errno == EACCES)
                                        bb_error_msg(bb_msg_perm_denied_are_you_root);
                                else