You'd think there'd be a compiler warning for "pointless if", wouldn't you?

git-svn-id: svn://busybox.net/trunk/busybox@14746 69ca8d6d-28ef-0310-b511-8ec308f3f277

1 files changed, 3 insertions, 3 deletions

diff --git a/util-linux/mount.c b/util-linux/mount.c
index 42d0eb223..ba55d2480 100644
--- a/util-linux/mount.c
+++ b/util-linux/mount.c
@@ -291,11 +291,11 @@ static int singlemount(struct mntent *mp)
                 }
         }
 
-        // Look at the file.  (Not found isn't a failure for remount.)
+        // Look at the file.  (Not found isn't a failure for remount, or for
+        // a synthetic filesystem like proc or sysfs.)
 
         if (lstat(mp->mnt_fsname, &st));
-
-        if (!(vfsflags & (MS_REMOUNT | MS_BIND | MS_MOVE))) {
+        else if (!(vfsflags & (MS_REMOUNT | MS_BIND | MS_MOVE))) {
                 // Do we need to allocate a loopback device for it?
 
                 if (ENABLE_FEATURE_MOUNT_LOOP && S_ISREG(st.st_mode)) {