diff options
author | Al Viro <viro@zeniv.linux.org.uk> | 2020-04-05 21:59:55 -0400 |
---|---|---|
committer | Al Viro <viro@zeniv.linux.org.uk> | 2020-04-06 10:38:59 -0400 |
commit | 5bd73286d50fc242bbff1aaddb0e97c4527c9d78 (patch) | |
tree | 25668472f85965c97eb4839b14341ddd0d811817 /fs | |
parent | 9c577491b985e1b27995abe69b32b041893798cf (diff) | |
download | linux-5bd73286d50fc242bbff1aaddb0e97c4527c9d78.tar.gz linux-5bd73286d50fc242bbff1aaddb0e97c4527c9d78.tar.bz2 linux-5bd73286d50fc242bbff1aaddb0e97c4527c9d78.zip |
fix a braino in legitimize_path()
brown paperbag time... wrong order of arguments ended up confusing
the values to check dentry and mount_lock seqcounts against.
Reported-by: kernel test robot <rong.a.chen@intel.com>
Fixes: 2aa38470853a ("non-RCU analogue of the previous commit")
Tested-by: kernel test robot <rong.a.chen@intel.com>
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Diffstat (limited to 'fs')
-rw-r--r-- | fs/namei.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/namei.c b/fs/namei.c index 61fdb77a7d58..a320371899cf 100644 --- a/fs/namei.c +++ b/fs/namei.c @@ -610,7 +610,7 @@ static bool __legitimize_path(struct path *path, unsigned seq, unsigned mseq) static inline bool legitimize_path(struct nameidata *nd, struct path *path, unsigned seq) { - return __legitimize_path(path, nd->m_seq, seq); + return __legitimize_path(path, seq, nd->m_seq); } static bool legitimize_links(struct nameidata *nd) |