diff options
| author | J. Bruce Fields <bfields@redhat.com> | 2010-08-18 00:46:33 +0200 |
|---|---|---|
| committer | J. Bruce Fields <bfields@redhat.com> | 2010-08-26 19:20:51 +0200 |
| commit | 30c0e1ef0a8a6cab4e0f9357698c81a2f7f73cc5 (patch) | |
| tree | 3bccc5ba7e8dff301f8dd4536158306d9261244d /fs | |
| parent | nfsd4: fix file open accounting for RDWR opens (diff) | |
| download | linux-30c0e1ef0a8a6cab4e0f9357698c81a2f7f73cc5.tar.xz 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; |