NFS: Return meaningful status from decode_secinfo()
authorBryan Schumaker <bjschuma@netapp.com>
Wed, 27 Apr 2011 19:28:44 +0000 (15:28 -0400)
committerTrond Myklebust <Trond.Myklebust@netapp.com>
Wed, 27 Apr 2011 20:17:29 +0000 (16:17 -0400)
When compiling, I was getting this warning:
fs/nfs/nfs4xdr.c: In function ‘decode_secinfo’:
fs/nfs/nfs4xdr.c:4839:6: warning: variable ‘status’ set but not used
[-Wunused-but-set-variable]

We were unconditionally returning 0 as long as there wasn't an error
coming out of xdr_inline_decode().  We probably want to check the error
status coming out of decode_op_hdr() and decode_secinfo_gss(), rather
than assuming that everything is OK all the time.

Signed-off-by: Bryan Schumaker <bjschuma@netapp.com>
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
fs/nfs/nfs4xdr.c

Simple merge