diff options
| author | Bryan Schumaker <bjschuma@netapp.com> | 2011-04-27 19:28:44 (GMT) |
|---|---|---|
| committer | Trond Myklebust <Trond.Myklebust@netapp.com> | 2011-04-27 20:17:29 (GMT) |
| commit | 613e901e1ee0e1096663b649eee8e5d6697919f3 (patch) | |
| tree | 1a7c6c79585da11445de9c3b3c6c701d5c6c0195 /Documentation/kref.txt | |
| parent | 28331a46d88459788c8fca72dbb0415cd7f514c9 (diff) | |
| download | linux-fsl-qoriq-613e901e1ee0e1096663b649eee8e5d6697919f3.tar.xz | |
NFS: Return meaningful status from decode_secinfo()
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>
Diffstat (limited to 'Documentation/kref.txt')
0 files changed, 0 insertions, 0 deletions