NFSv4: nfs_atomic_open() can race when looking up a non-regular file
commit 1751fc1db36f6f411709e143d5393f92d12137a9 upstream.
If the file type changes back to being a regular file on the server
between the failed OPEN and our LOOKUP, then we need to re-run the OPEN.
Fixes: 0dd2b474d0
("nfs: implement i_op->atomic_open()")
Signed-off-by: Trond Myklebust <trond.myklebust@hammerspace.com>
Signed-off-by: Anna Schumaker <Anna.Schumaker@Netapp.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:

committed by
Greg Kroah-Hartman

parent
ce8c552b88
commit
87880e3803
@@ -1782,12 +1782,17 @@ no_open:
|
|||||||
if ((lookup_flags & LOOKUP_DIRECTORY) && inode &&
|
if ((lookup_flags & LOOKUP_DIRECTORY) && inode &&
|
||||||
!S_ISDIR(inode->i_mode))
|
!S_ISDIR(inode->i_mode))
|
||||||
res = ERR_PTR(-ENOTDIR);
|
res = ERR_PTR(-ENOTDIR);
|
||||||
|
else if (inode && S_ISREG(inode->i_mode))
|
||||||
|
res = ERR_PTR(-EOPENSTALE);
|
||||||
} else if (!IS_ERR(res)) {
|
} else if (!IS_ERR(res)) {
|
||||||
inode = d_inode(res);
|
inode = d_inode(res);
|
||||||
if ((lookup_flags & LOOKUP_DIRECTORY) && inode &&
|
if ((lookup_flags & LOOKUP_DIRECTORY) && inode &&
|
||||||
!S_ISDIR(inode->i_mode)) {
|
!S_ISDIR(inode->i_mode)) {
|
||||||
dput(res);
|
dput(res);
|
||||||
res = ERR_PTR(-ENOTDIR);
|
res = ERR_PTR(-ENOTDIR);
|
||||||
|
} else if (inode && S_ISREG(inode->i_mode)) {
|
||||||
|
dput(res);
|
||||||
|
res = ERR_PTR(-EOPENSTALE);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
if (switched) {
|
if (switched) {
|
||||||
|
Reference in New Issue
Block a user