diff options
| author |   Chuck Lever <chuck.lever@oracle.com> | 2006-10-19 23:28:42 -0700 |
|---|---|---|
| committer |   Linus Torvalds <torvalds@g5.osdl.org> | 2006-10-20 10:26:39 -0700 |
| commit | b87c0adfeaaf8d8310c4f790d76072a5961b3518 (patch) | |
| tree | 1c3f2395436f6ec3c8abc94af8266709860b0839 /fs/nfs/nfs4proc.c | |
| parent | [PATCH] NFS: __nfs_revalidate_inode() can use "inode" before checking it is non-NULL (diff) | |
| download | linux-dev-b87c0adfeaaf8d8310c4f790d76072a5961b3518.tar.xz linux-dev-b87c0adfeaaf8d8310c4f790d76072a5961b3518.zip | |
[PATCH] NFS: remove unused check in nfs4_open_revalidate
Coverity spotted a superfluous error check in nfs4_open_revalidate(). Remove
it.
Coverity: #cid 847
Test plan:
Code inspection; another pass through Coverity.
Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to '')
| -rw-r--r-- | fs/nfs/nfs4proc.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/fs/nfs/nfs4proc.c b/fs/nfs/nfs4proc.c index 47c7e6e3910d..7421bcb3b728 100644 --- a/fs/nfs/nfs4proc.c +++ b/fs/nfs/nfs4proc.c @@ -1314,11 +1314,9 @@ nfs4_open_revalidate(struct inode *dir, struct dentry *dentry, int openflags, st case -EROFS: lookup_instantiate_filp(nd, (struct dentry *)state, NULL); return 1; - case -ENOENT: - if (dentry->d_inode == NULL) - return 1; + default: + goto out_drop; } - goto out_drop; } if (state->inode == dentry->d_inode) { nfs4_intent_set_file(nd, dentry, state); |