From 5e6892dd8be6cd55e5f4615f1b1dbe9651535a1c Mon Sep 17 00:00:00 2001 From: Eryu Guan Date: Fri, 10 Mar 2017 12:26:34 +0800 Subject: [PATCH] common/rc: use findmnt to check mounted device Doing 'grep -F "$dev on "' to find the mounted device is not always accurate, e.g. SCRATCH_DEV=/vda6 is mounted but not on SCRATCH_MNT=/vda6/ovl-mnt - aborting Already mounted result: /dev/vda6 on /vda6 type xfs (rw,relatime,context=system_u:object_r:nfs_t:s0,attr2,inode64,noquota) Fix it by using findmnt command and specifying the $dev as mount source, print the result in "$dev $mnt" format. This works for local filesystems, network filesystems and overlayfs, avoids all kinds of tricky and error-prone grep pattern/regex. Also fixed the if-then-fi format in _check_mounted_on() while we're at it. Reviewed-by: Amir Goldstein Signed-off-by: Eryu Guan --- common/rc | 13 ++++--------- 1 file changed, 4 insertions(+), 9 deletions(-) diff --git a/common/rc b/common/rc index 756c4146..d53f6796 100644 --- a/common/rc +++ b/common/rc @@ -1462,24 +1462,19 @@ _check_mounted_on() local mnt=$4 local type=$5 - # Note that we use -F here so grep doesn't try to interpret an NFS over - # IPv6 server as a regular expression. Because of that, we cannot use - # ^$dev so we use "$dev on " to avoid matching $dev to mount point field - # for overlay case, where $dev is a directory. - local mount_rec=`_mount | grep -F "$dev on "` + # find $dev as the source, and print result in "$dev $mnt" format + local mount_rec=`findmnt -rnc -S $dev -o SOURCE,TARGET` [ -n "$mount_rec" ] || return 1 # 1 = not mounted # if it's mounted, make sure its on $mnt - if ! (echo $mount_rec | grep -q "$dev on $mnt") - then + if [ "$mount_rec" != "$dev $mnt" ]; then echo "$devname=$dev is mounted but not on $mntname=$mnt - aborting" echo "Already mounted result:" echo $mount_rec return 2 # 2 = mounted on wrong mnt fi - if [ -n "$type" -a "`_fs_type $dev`" != "$type" ] - then + if [ -n "$type" -a "`_fs_type $dev`" != "$type" ]; then echo "$devname=$dev is mounted but not a type $type filesystem" # raw $DF_PROG cannot handle NFS/CIFS/overlay correctly _df_device $dev -- 2.39.5