fix mountpoint test to not prevemt mkfs_xxx from making image in any file

Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>

diff --git a/coreutils/df.c b/coreutils/df.c
index d95aff39d52432b868f4f535cd85254991bcf334..4bbfe943b0fd9c432f83856b5d2f8ef202f14e4a 100644 (file)
--- a/coreutils/df.c
+++ b/coreutils/df.c
@@ -119,7 +119,7 @@ int df_main(int argc, char **argv)
                        mount_point = *argv++;
                        if (!mount_point)
                                break;
-                       mount_entry = find_mount_point(mount_point);
+                       mount_entry = find_mount_point(mount_point, 1);
                        if (!mount_entry) {
                                bb_error_msg("%s: can't find mount point", mount_point);
  set_error:

diff --git a/include/libbb.h b/include/libbb.h
index 936d98a23947cf5d940860accb8464d6de212e34..c7b4de13cf05f24f943b16d2ec2c77afd1ae5944 100644 (file)
--- a/include/libbb.h
+++ b/include/libbb.h
@@ -995,7 +995,7 @@ extern void run_applet_no_and_exit(int a, char **argv) NORETURN FAST_FUNC;
 
 #ifdef HAVE_MNTENT_H
 extern int match_fstype(const struct mntent *mt, const char *fstypes) FAST_FUNC;
-extern struct mntent *find_mount_point(const char *name) FAST_FUNC;
+extern struct mntent *find_mount_point(const char *name, int subdir_too) FAST_FUNC;
 #endif
 extern void erase_mtab(const char * name) FAST_FUNC;
 extern unsigned int tty_baud_to_value(speed_t speed) FAST_FUNC;

diff --git a/libbb/find_mount_point.c b/libbb/find_mount_point.c
index 12b2cfce4e6fe9b140a24196c48f151dbf141fc2..bcb18effe5650af4bfb30888589341e4feb74f6f 100644 (file)
--- a/libbb/find_mount_point.c
+++ b/libbb/find_mount_point.c
@@ -17,27 +17,29 @@
  * Given any other file (or directory), find the mount table entry for its
  * filesystem.
  */
-struct mntent* FAST_FUNC find_mount_point(const char *name)
+struct mntent* FAST_FUNC find_mount_point(const char *name, int subdir_too)
 {
        struct stat s;
-       dev_t mountDevice;
-       FILE *mountTable;
+       FILE *mtab_fp;
        struct mntent *mountEntry;
+       dev_t devno_of_name;
+       bool block_dev;
 
        if (stat(name, &s) != 0)
                return NULL;
 
-       if (S_ISBLK(s.st_mode))
-               mountDevice = s.st_rdev;
-       else
-               mountDevice = s.st_dev;
-
+       devno_of_name = s.st_dev;
+       block_dev = 0;
+       if (S_ISBLK(s.st_mode)) {
+               devno_of_name = s.st_rdev;
+               block_dev = 1;
+       }
 
-       mountTable = setmntent(bb_path_mtab_file, "r");
-       if (!mountTable)
-               return 0;
+       mtab_fp = setmntent(bb_path_mtab_file, "r");
+       if (!mtab_fp)
+               return NULL;
 
-       while ((mountEntry = getmntent(mountTable)) != NULL) {
+       while ((mountEntry = getmntent(mtab_fp)) != NULL) {
                /* rootfs mount in Linux 2.6 exists always,
                 * and it makes sense to always ignore it.
                 * Otherwise people can't reference their "real" root! */
@@ -49,13 +51,18 @@ struct mntent* FAST_FUNC find_mount_point(const char *name)
                ) { /* String match. */
                        break;
                }
-               /* Match the device. */
-               if (stat(mountEntry->mnt_fsname, &s) == 0 && s.st_rdev == mountDevice)
+
+               if (!(subdir_too || block_dev))
+                       continue;
+
+               /* Is device's dev_t == name's dev_t? */
+               if (stat(mountEntry->mnt_fsname, &s) == 0 && s.st_rdev == devno_of_name)
                        break;
                /* Match the directory's mount point. */
-               if (stat(mountEntry->mnt_dir, &s) == 0 && s.st_dev == mountDevice)
+               if (stat(mountEntry->mnt_dir, &s) == 0 && s.st_dev == devno_of_name)
                        break;
        }
-       endmntent(mountTable);
+       endmntent(mtab_fp);
+
        return mountEntry;
 }

diff --git a/testsuite/README b/testsuite/README
index 3c52254dd2efc233415f2594f7e7131d3730db21..03ac377baa031e59246b3b0b0d39a6c0c4cc4fac 100644 (file)
--- a/testsuite/README
+++ b/testsuite/README
@@ -27,6 +27,8 @@ For busybox built against uclibc, /etc/TZ does not exist or does not match
 host system timezone setting. For glibc based host systems, timezona settings
 are in /etc/localtime.
 
+LANG and LC_xxx environment variables set to non-C locale.
+
 
 For the entire testsuite, the copyright is as follows:
 

diff --git a/testsuite/cpio.tests b/testsuite/cpio.tests
index 576180ff1ac253cc86c8412737d90866c63ab0e1..fa0b65a08a445e4b153b3fdd3f1a974e57983aef 100755 (executable)
--- a/testsuite/cpio.tests
+++ b/testsuite/cpio.tests
@@ -41,6 +41,7 @@ ls -ln cpio.testdir | $FILTER_LS" \
 " \
        "" ""
 
+test x"$SKIP_KNOWN_BUGS" = x"" && {
 # Currently fails. Numerous buglets: "1 blocks" versus "1 block",
 # "1 block" must go to stderr, does not list cpio.testdir/x and cpio.testdir/y
 testing "cpio lists hardlinks" \
@@ -53,6 +54,7 @@ cpio.testdir/y
 0
 " \
        "" ""
+}
 
 # More complex case
 rm -rf cpio.testdir cpio.testdir2 2>/dev/null

diff --git a/testsuite/mount.tests b/testsuite/mount.tests
index 5374ecbd6429e8db10a69c11fce2e2c7dac71916..c8381a0de13f2f599675c576e17b5bd761cc16a0 100755 (executable)
--- a/testsuite/mount.tests
+++ b/testsuite/mount.tests
@@ -1,5 +1,4 @@
 #!/bin/sh
-
 # Copyright 2007 by Denys Vlasenko <vda.linux@googlemail.com>
 # Licensed under GPL v2, see file LICENSE for details.
 
@@ -10,8 +9,8 @@ test "`id -u`" = 0 || {
        exit 0
 }
 
-dd if=/dev/zero of=mount.image1m count=1 bs=1M 2>/dev/null || exit 1
-mkfs.minix -v mount.image1m >/dev/null 2>&1 || exit 1
+dd if=/dev/zero of=mount.image1m count=1 bs=1M 2>/dev/null || { echo "dd error"; exit 1; }
+mkfs.minix -v mount.image1m >/dev/null 2>&1 || { echo "mkfs.minix error"; exit 1; }
 testdir=$PWD/testdir
 mkdir $testdir 2>/dev/null
 umount -d $testdir 2>/dev/null

diff --git a/testsuite/od.tests b/testsuite/od.tests
index 69c29952a637435f30fec7f1b13a46201bbe3462..f136bb5fa686a8cc6e14e521cc5d279fcdd6650d 100755 (executable)
--- a/testsuite/od.tests
+++ b/testsuite/od.tests
@@ -10,7 +10,7 @@ testing "od -b" \
        "od -b" \
 "\
 0000000 110 105 114 114 117
-0000006
+0000005
 " \
        "" "HELLO"
 

diff --git a/testsuite/runtest b/testsuite/runtest
index 5560f7bc230ec3ce0d7ddf641b738f29d6ab1a85..2428f6c368c7fabc577b57ceab22f293d0e6188a 100755 (executable)
--- a/testsuite/runtest
+++ b/testsuite/runtest
@@ -1,20 +1,8 @@
 #!/bin/sh
-
 # Usage:
 # runtest [applet1] [applet2...]
 
-# Helper for helpers. Oh my...
-test x"$ECHO" != x"" || {
-       ECHO="echo"
-       test x"`echo -ne`" = x"" || {
-               # Compile and use a replacement 'echo' which understands -e -n
-               ECHO="$PWD/echo-ne"
-               test -x "$ECHO" || {
-                       gcc -Os -o "$ECHO" ../scripts/echo.c || exit 1
-               }
-       }
-       export ECHO
-}
+. testing.sh
 
 # Run one old-style test.
 # Tests are stored in applet/testcase shell scripts.

diff --git a/testsuite/sed.tests b/testsuite/sed.tests
index 677303be1fb6d94ec7bd3a660428895cb1dbd3c9..a9d9ada311e2e1b0f4d5d4d10feedcba4034ecbd 100755 (executable)
--- a/testsuite/sed.tests
+++ b/testsuite/sed.tests
@@ -51,8 +51,10 @@ testing "sed -n" "sed -n -e s/foo/bar/ -e s/bar/baz/" "" "" "foo\n"
 testing "sed s//p" "sed -e s/foo/bar/p -e s/bar/baz/p" "bar\nbaz\nbaz\n" \
        "" "foo\n"
 testing "sed -n s//p" "sed -ne s/abc/def/p" "def\n" "" "abc\n"
+test x"$SKIP_KNOWN_BUGS" = x"" && {
 testing "sed s//g (exhaustive)" "sed -e 's/[[:space:]]*/,/g'" ",1,2,3,4,5,\n" \
        "" "12345\n"
+}
 testing "sed s arbitrary delimiter" "sed -e 's woo boing '" "boing\n" "" "woo\n"
 testing "sed s chains" "sed -e s/foo/bar/ -e s/bar/baz/" "baz\n" "" "foo\n"
 testing "sed s chains2" "sed -e s/foo/bar/ -e s/baz/nee/" "bar\n" "" "foo\n"
@@ -72,6 +74,7 @@ testing "sed t (test/branch clears test bit)" "sed -e 's/a/b/;:loop;t loop'" \
 testing "sed T (!test/branch)" "sed -e 's/a/1/;T notone;p;: notone;p'" \
        "1\n1\n1\nb\nb\nc\nc\n" "" "a\nb\nc\n"
 
+test x"$SKIP_KNOWN_BUGS" = x"" && {
 # Normal sed end-of-script doesn't print "c" because n flushed the pattern
 # space.  If n hits EOF, pattern space is empty when script ends.
 # Query: how does this interact with no newline at EOF?
@@ -80,6 +83,7 @@ testing "sed n (flushes pattern space, terminates early)" "sed -e 'n;p'" \
 # N does _not_ flush pattern space, therefore c is still in there @ script end.
 testing "sed N (doesn't flush pattern space when terminating)" "sed -e 'N;p'" \
        "a\nb\na\nb\nc\n" "" "a\nb\nc\n"
+}
 testing "sed address match newline" 'sed "/b/N;/b\\nc/i woo"' \
        "a\nwoo\nb\nc\nd\n" "" "a\nb\nc\nd\n"
 
@@ -99,13 +103,17 @@ testing "sed d ends script iteration (2)" \
        "sed -e '/ook/d;a\' -e 'bang'" "woot\nbang\n" "" "ook\nwoot\n"
 
 # Multiple files, with varying newlines and NUL bytes
+test x"$SKIP_KNOWN_BUGS" = x"" && {
 testing "sed embedded NUL" "sed -e 's/woo/bang/'" "\0bang\0woo\0" "" \
        "\0woo\0woo\0"
+}
 testing "sed embedded NUL g" "sed -e 's/woo/bang/g'" "bang\0bang\0" "" \
        "woo\0woo\0"
+test x"$SKIP_KNOWN_BUGS" = x"" && {
 echo -e "/woo/a he\0llo" > sed.commands
 testing "sed NUL in command" "sed -f sed.commands" "woo\nhe\0llo\n" "" "woo"
 rm sed.commands
+}
 
 # sed has funky behavior with newlines at the end of file.  Test lots of
 # corner cases with the optional newline appending behavior.
@@ -120,8 +128,10 @@ testing "sed empty file plus cat" "sed -e 's/nohit//' input -" "one\ntwo" \
        "" "one\ntwo"
 testing "sed cat plus empty file" "sed -e 's/nohit//' input -" "one\ntwo" \
        "one\ntwo" ""
+test x"$SKIP_KNOWN_BUGS" = x"" && {
 testing "sed append autoinserts newline" "sed -e '/woot/a woo' -" \
        "woot\nwoo\n" "" "woot"
+}
 testing "sed insert doesn't autoinsert newline" "sed -e '/woot/i woo' -" \
        "woo\nwoot" "" "woot"
 testing "sed print autoinsert newlines" "sed -e 'p' -" "one\none" "" "one"
@@ -137,9 +147,11 @@ testing "sed selective matches insert newline" \
 testing "sed selective matches noinsert newline" \
        "sed -ne 's/woo/bang/p' input -" "a bang\nb bang" "a woo\nb woo" \
        "c no\nd no"
+test x"$SKIP_KNOWN_BUGS" = x"" && {
 testing "sed clusternewline" \
        "sed -e '/one/a 111' -e '/two/i 222' -e p input -" \
        "one\none\n111\n222\ntwo\ntwo" "one" "two"
+}
 testing "sed subst+write" \
        "sed -e 's/i/z/' -e 'woutputw' input -; echo -n X; cat outputw" \
        "thzngy\nagaznXthzngy\nagazn" "thingy" "again"
@@ -175,8 +187,12 @@ testing "sed lie-to-autoconf" "sed --version | grep -o 'GNU sed version '" \
        "GNU sed version \n" "" ""
 
 # Jump to nonexistent label
-testing "sed nonexistent label" "sed -e 'b walrus' 2> /dev/null || echo yes" \
+test x"$SKIP_KNOWN_BUGS" = x"" && {
+# Incompatibility: illegal jump is not detected if input is ""
+# (that is, no lines at all). GNU sed 4.1.5 complains even in this case
+testing "sed nonexistent label" "sed -e 'b walrus' 2>/dev/null || echo yes" \
        "yes\n" "" ""
+}
 
 testing "sed backref from empty s uses range regex" \
        "sed -e '/woot/s//eep \0 eep/'" "eep woot eep" "" "woot"

diff --git a/testsuite/sort.tests b/testsuite/sort.tests
index f700dc0c16ca1992e82be9e955b41d586fe45ef9..627ca51f858f278726405832079eda8b875b4f1e 100755 (executable)
--- a/testsuite/sort.tests
+++ b/testsuite/sort.tests
@@ -45,8 +45,8 @@ egg   1       2       papyrus
 999    3       0       algebra
 " "$data" ""
 
-# Busybox is definitely doing this one wrong just now.  FIXME
-
+test x"$SKIP_KNOWN_BUGS" = x"" && {
+# Busybox is definitely doing these wrong.  FIXME
 testing "sort key range with numeric option and global reverse" \
 "sort -k2,3n -r input" \
 "egg   1       2       papyrus
@@ -56,8 +56,6 @@ testing "sort key range with numeric option and global reverse" \
 7      3       42      soup
 " "$data" ""
 
-#
-
 testing "sort key range with multiple options" "sort -k2,3rn input" \
 "7     3       42      soup
 999    3       0       algebra
@@ -65,6 +63,7 @@ testing "sort key range with multiple options" "sort -k2,3rn input" \
 42     1       3       woot
 egg    1       2       papyrus
 " "$data" ""
+}
 
 testing "sort key range with two -k options" "sort -k 2,2n -k 1,1r input" "\
 d 2

diff --git a/testsuite/testing.sh b/testsuite/testing.sh
index a57c4d68d332cabd8f4817d033da851d95defd9b..22c640fafd1c2c6f74f1fb69368a171815125296 100755 (executable)
--- a/testsuite/testing.sh
+++ b/testsuite/testing.sh
@@ -36,6 +36,20 @@
 export FAILCOUNT=0
 export SKIP=
 
+# Helper for helpers. Oh my...
+
+test x"$ECHO" != x"" || {
+       ECHO="echo"
+       test x"`echo -ne`" = x"" || {
+               # Compile and use a replacement 'echo' which understands -e -n
+               ECHO="$PWD/echo-ne"
+               test -x "$ECHO" || {
+                       gcc -Os -o "$ECHO" ../scripts/echo.c || exit 1
+               }
+       }
+       export ECHO
+}
+
 # Helper functions
 
 optional()
@@ -73,7 +87,7 @@ testing()
 
   $ECHO -ne "$3" > expected
   $ECHO -ne "$4" > input
-  [ -z "$VERBOSE" ] || echo "echo '$5' | $2"
+  [ -z "$VERBOSE" ] || echo "echo -ne '$5' | $2"
   $ECHO -ne "$5" | eval "$2" > actual
   RETVAL=$?
 

diff --git a/util-linux/fsck_minix.c b/util-linux/fsck_minix.c
index ca0b17efedb8a94068027b82771c287fa253854f..5d1d2af31823d47a01b0d3f6e6bcba74ab18e90b 100644 (file)
--- a/util-linux/fsck_minix.c
+++ b/util-linux/fsck_minix.c
@@ -374,7 +374,7 @@ static int ask(const char *string, int def)
  */
 static void check_mount(void)
 {
-       if (find_mount_point(device_name)) {
+       if (find_mount_point(device_name, 0)) {
                int cont;
 #if ENABLE_FEATURE_MTAB_SUPPORT
                /*

diff --git a/util-linux/mkfs_minix.c b/util-linux/mkfs_minix.c
index 18512a395ed826bf566265947320f9473c22aa60..cf757610b19f6d6c3cd973563d5a3ee1a83f7bf3 100644 (file)
--- a/util-linux/mkfs_minix.c
+++ b/util-linux/mkfs_minix.c
@@ -682,7 +682,7 @@ int mkfs_minix_main(int argc UNUSED_PARAM, char **argv)
                G.total_blocks = 65535;
 
        /* Check if it is mounted */
-       if (find_mount_point(G.device_name))
+       if (find_mount_point(G.device_name, 0))
                bb_error_msg_and_die("can't format mounted filesystem");
 
        xmove_fd(xopen(G.device_name, O_RDWR), dev_fd);

diff --git a/util-linux/mkfs_vfat.c b/util-linux/mkfs_vfat.c
index dff10f9e1c6e40854ac0355789be16e7b113dd06..bdd4dd803d775d15826470139b4e35cc4fcde047 100644 (file)
--- a/util-linux/mkfs_vfat.c
+++ b/util-linux/mkfs_vfat.c
@@ -279,7 +279,7 @@ int mkfs_vfat_main(int argc UNUSED_PARAM, char **argv)
                )
                        bb_error_msg_and_die("will not try to make filesystem on full-disk device (use -I if wanted)");
                // can't work on mounted filesystems
-               if (find_mount_point(device_name))
+               if (find_mount_point(device_name, 0))
                        bb_error_msg_and_die("can't format mounted filesystem");
 #endif
                // get true sector size