diff options
| author | Andy Adamson <andros@netapp.com> | 2010-01-14 23:45:06 +0100 |
|---|---|---|
| committer | Trond Myklebust <Trond.Myklebust@netapp.com> | 2010-02-10 14:30:57 +0100 |
| commit | e95e60daee44fade63f32429ddcf1c2012a95632 (patch) | |
| tree | bc9a627467386a3a7629b81a4d25b8c1d191d701 /fs/nfs/callback_xdr.c | |
| parent | nfs41: directly encode back channel error (diff) | |
| download | linux-e95e60daee44fade63f32429ddcf1c2012a95632.tar.xz linux-e95e60daee44fade63f32429ddcf1c2012a95632.zip | |
nfs41: remove uneeded checks in callback processing
All callback operations have arguments to decode and require processing.
The preprocess_nfs4X_op functions catch unsupported or illegal ops so
decode_args and process_op pointers are always non NULL.
Signed-off-by: Andy Adamson <andros@netapp.com>
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
Diffstat (limited to 'fs/nfs/callback_xdr.c')
| -rw-r--r-- | fs/nfs/callback_xdr.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/fs/nfs/callback_xdr.c b/fs/nfs/callback_xdr.c index d3e07f469949..a6f2ded72b17 100644 --- a/fs/nfs/callback_xdr.c +++ b/fs/nfs/callback_xdr.c @@ -630,9 +630,8 @@ static __be32 process_op(uint32_t minorversion, int nop, maxlen = xdr_out->end - xdr_out->p; if (maxlen > 0 && maxlen < PAGE_SIZE) { - if (likely(op->decode_args != NULL)) - status = op->decode_args(rqstp, xdr_in, argp); - if (likely(status == 0 && op->process_op != NULL)) + status = op->decode_args(rqstp, xdr_in, argp); + if (likely(status == 0)) status = op->process_op(argp, resp); } else status = htonl(NFS4ERR_RESOURCE); |