nfs41: directly encode back channel error
authorAndy Adamson <andros@netapp.com>
Thu, 14 Jan 2010 22:45:05 +0000 (17:45 -0500)
committerTrond Myklebust <Trond.Myklebust@netapp.com>
Wed, 10 Feb 2010 13:30:56 +0000 (08:30 -0500)
Skip all other processing when error is encountered.

Signed-off-by: Andy Adamson <andros@netapp.com>
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
fs/nfs/callback_xdr.c

index 6ae327871b86e3262e0d7fdccbd48869c3ead88c..d3e07f4699496c8e0bbed7e2426973b9dfcb7a7a 100644 (file)
@@ -625,16 +625,19 @@ static __be32 process_op(uint32_t minorversion, int nop,
                                preprocess_nfs4_op(op_nr, &op);
        if (status == htonl(NFS4ERR_OP_ILLEGAL))
                op_nr = OP_CB_ILLEGAL;
+       if (status)
+               goto encode_hdr;
 
        maxlen = xdr_out->end - xdr_out->p;
        if (maxlen > 0 && maxlen < PAGE_SIZE) {
-               if (likely(status == 0 && op->decode_args != NULL))
+               if (likely(op->decode_args != NULL))
                        status = op->decode_args(rqstp, xdr_in, argp);
                if (likely(status == 0 && op->process_op != NULL))
                        status = op->process_op(argp, resp);
        } else
                status = htonl(NFS4ERR_RESOURCE);
 
+encode_hdr:
        res = encode_op_hdr(xdr_out, op_nr, status);
        if (unlikely(res))
                return res;