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 | |
| parent | 998db52c03cd293d16a457f1b396cea932244147 (diff) | |
| download | blackbird-op-linux-30c0e1ef0a8a6cab4e0f9357698c81a2f7f73cc5.tar.gz blackbird-op-linux-30c0e1ef0a8a6cab4e0f9357698c81a2f7f73cc5.zip | |
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')
| -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, *filpp = find_readable_file(stp->st_file); else *filpp = find_writeable_file(stp->st_file); - BUG_ON(!*filpp); /* assured by check_openmode */ } } status = nfs_ok; |
