[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#21372: [PATCH] df: fix prioritize real mounts over bind mounts
From: |
Dave Chiluk |
Subject: |
bug#21372: [PATCH] df: fix prioritize real mounts over bind mounts |
Date: |
Fri, 28 Aug 2015 15:42:45 -0500 |
Fixes an issue where bind mounts with shorter mount directories than the
original mount are prioritized when running df. The root cause of this
is that /proc/self/mountinfo now lists the filesystem device with bind
mounts rather than the source directory. With /etc/mtab the source
device was listed as the originating directory so this was not an issue.
More information is available here.
https://bugs.launchpad.net/ubuntu/+source/coreutils/+bug/1432871
---
src/df.c | 7 +++++--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/src/df.c b/src/df.c
index 2e541b9..13e2661 100644
--- a/src/df.c
+++ b/src/df.c
@@ -652,9 +652,12 @@ filter_mount_list (bool devices_only)
else if ((strchr (me->me_devname, '/')
/* let "real" devices with '/' in the name win. */
&& ! strchr (devlist->me->me_devname, '/'))
- /* let a shorter mountdir win. */
- || (strlen (devlist->me->me_mountdir)
+ /* let a shorter mountdir win. */
+ /* Only if it's not a bind mount.*/
+ || ((strlen (devlist->me->me_mountdir)
> strlen (me->me_mountdir))
+ && (strlen (devlist->me->me_mountroot)
+ > strlen(me->me_mountroot)))
/* let an entry overmounted on a new device win... */
|| (! STREQ (devlist->me->me_devname, me->me_devname)
/* ... but only when matching an existing mnt point,
--
1.9.1