fix bogus read_seqretry() checks introduced in b37199e
authorAl Viro <viro@zeniv.linux.org.uk>
Sun, 14 Sep 2014 01:50:45 +0000 (21:50 -0400)
committerAl Viro <viro@zeniv.linux.org.uk>
Sun, 14 Sep 2014 02:14:16 +0000 (22:14 -0400)
read_seqretry() returns true on mismatch, not on match...

Cc: stable@vger.kernel.org # 3.15+
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
fs/namei.c

index a996bb48dfabf4f645dced56f527ed3d5568bb2c..3d1dc745f9d8c168b4b8500371482a2b04609139 100644 (file)
@@ -1137,7 +1137,7 @@ static bool __follow_mount_rcu(struct nameidata *nd, struct path *path,
                 */
                *inode = path->dentry->d_inode;
        }
-       return read_seqretry(&mount_lock, nd->m_seq) &&
+       return !read_seqretry(&mount_lock, nd->m_seq) &&
                !(path->dentry->d_flags & DCACHE_NEED_AUTOMOUNT);
 }
 
@@ -1174,7 +1174,7 @@ static int follow_dotdot_rcu(struct nameidata *nd)
                nd->path.mnt = &mounted->mnt;
                nd->path.dentry = mounted->mnt.mnt_root;
                nd->seq = read_seqcount_begin(&nd->path.dentry->d_seq);
-               if (!read_seqretry(&mount_lock, nd->m_seq))
+               if (read_seqretry(&mount_lock, nd->m_seq))
                        goto failed;
        }
        nd->inode = nd->path.dentry->d_inode;