diff options
| author |   ChenXiaoSong <chenxiaosong2@huawei.com> | 2022-03-29 19:32:07 +0800 |
|---|---|---|
| committer |   Trond Myklebust <trond.myklebust@hammerspace.com> | 2022-03-29 22:14:00 -0400 |
| commit | ab0fc21bc7105b54bafd85bd8b82742f9e68898a (patch) | |
| tree | f8596336e2328573fa67d3c72329291fb72dd415 /fs/nfs/inode.c | |
| parent | NFSv4/pNFS: Fix another issue with a list iterator pointing to the head (diff) | |
| download | linux-dev-ab0fc21bc7105b54bafd85bd8b82742f9e68898a.tar.xz linux-dev-ab0fc21bc7105b54bafd85bd8b82742f9e68898a.zip | |
Revert "NFSv4: Handle the special Linux file open access mode"
This reverts commit 44942b4e457beda00981f616402a1a791e8c616e.
After secondly opening a file with O_ACCMODE|O_DIRECT flags,
nfs4_valid_open_stateid() will dereference NULL nfs4_state when lseek().
Reproducer:
1. mount -t nfs -o vers=4.2 $server_ip:/ /mnt/
2. fd = open("/mnt/file", O_ACCMODE|O_DIRECT|O_CREAT)
3. close(fd)
4. fd = open("/mnt/file", O_ACCMODE|O_DIRECT)
5. lseek(fd)
Reported-by: Lyu Tao <tao.lyu@epfl.ch>
Signed-off-by: ChenXiaoSong <chenxiaosong2@huawei.com>
Signed-off-by: Trond Myklebust <trond.myklebust@hammerspace.com>
Diffstat (limited to 'fs/nfs/inode.c')
| -rw-r--r-- | fs/nfs/inode.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/fs/nfs/inode.c b/fs/nfs/inode.c index e51d86707fca..e72900c059ee 100644 --- a/fs/nfs/inode.c +++ b/fs/nfs/inode.c @@ -1180,7 +1180,6 @@ int nfs_open(struct inode *inode, struct file *filp) nfs_fscache_open_file(inode, filp); return 0; } -EXPORT_SYMBOL_GPL(nfs_open); /* * This function is called whenever some part of NFS notices that |