diff options
author | J. Bruce Fields <bfields@redhat.com> | 2010-08-17 18:46:33 -0400 |
---|---|---|
committer | J. Bruce Fields <bfields@redhat.com> | 2010-08-26 13:20:51 -0400 |
commit | 30c0e1ef0a8a6cab4e0f9357698c81a2f7f73cc5 (patch) | |
tree | 3bccc5ba7e8dff301f8dd4536158306d9261244d /fs/nfsd | |
parent | 998db52c03cd293d16a457f1b396cea932244147 (diff) |
nfsd4: bad BUG() in preprocess_stateid_op
It's OK for this function to return without setting filp--we do it in
the special-stateid case.
And there's a legitimate case where we can hit this, since we do permit
reads on write-only stateid's.
Signed-off-by: J. Bruce Fields <bfields@redhat.com>
Diffstat (limited to 'fs/nfsd')
-rw-r--r-- | fs/nfsd/nfs4state.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/fs/nfsd/nfs4state.c b/fs/nfsd/nfs4state.c index 0a024917f052..b990eadb799c 100644 --- a/fs/nfsd/nfs4state.c +++ b/fs/nfsd/nfs4state.c | |||
@@ -2983,7 +2983,6 @@ nfs4_preprocess_stateid_op(struct nfsd4_compound_state *cstate, | |||
2983 | *filpp = find_readable_file(stp->st_file); | 2983 | *filpp = find_readable_file(stp->st_file); |
2984 | else | 2984 | else |
2985 | *filpp = find_writeable_file(stp->st_file); | 2985 | *filpp = find_writeable_file(stp->st_file); |
2986 | BUG_ON(!*filpp); /* assured by check_openmode */ | ||
2987 | } | 2986 | } |
2988 | } | 2987 | } |
2989 | status = nfs_ok; | 2988 | status = nfs_ok; |