diff options
| author | Bryan Schumaker <bjschuma@netapp.com> | 2011-04-27 21:28:44 +0200 |
|---|---|---|
| committer | Trond Myklebust <Trond.Myklebust@netapp.com> | 2011-04-27 22:17:29 +0200 |
| commit | 613e901e1ee0e1096663b649eee8e5d6697919f3 (patch) | |
| tree | 1a7c6c79585da11445de9c3b3c6c701d5c6c0195 /fs/nfs/super.c | |
| parent | NFSv4: Ensure we request the ordinary fileid when doing readdirplus (diff) | |
| download | linux-613e901e1ee0e1096663b649eee8e5d6697919f3.tar.xz linux-613e901e1ee0e1096663b649eee8e5d6697919f3.zip | |
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 '')
0 files changed, 0 insertions, 0 deletions