diff options
author | Bryan Schumaker <bjschuma@netapp.com> | 2011-04-27 15:47:15 -0400 |
---|---|---|
committer | J. Bruce Fields <bfields@redhat.com> | 2011-04-29 20:47:58 -0400 |
commit | 1db2b9dde3317e181f76860410cb0e7433896f28 (patch) | |
tree | e5175ed6fe697559debe07d44c52d1f6141fb61f /fs/nfsd/nfs4state.c | |
parent | fccb13c947de83a368e1f3c2216bbf4d8d41efa1 (diff) |
NFSD: Check status from nfsd4_map_bcts_dir()
Compiling gave me this warning:
fs/nfsd/nfs4state.c: In function ‘nfsd4_bind_conn_to_session’:
fs/nfsd/nfs4state.c:1623:9: warning: variable ‘status’ set but not used
[-Wunused-but-set-variable]
The local variable "status" was being set by nfsd4_map_bcts_dir() and
then ignored before calling nfsd4_new_conn().
Signed-off-by: Bryan Schumaker <bjschuma@netapp.com>
Signed-off-by: J. Bruce Fields <bfields@redhat.com>
Diffstat (limited to 'fs/nfsd/nfs4state.c')
-rw-r--r-- | fs/nfsd/nfs4state.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/fs/nfsd/nfs4state.c b/fs/nfsd/nfs4state.c index 2bb03f86a037..a2ea14f40b42 100644 --- a/fs/nfsd/nfs4state.c +++ b/fs/nfsd/nfs4state.c | |||
@@ -1636,8 +1636,9 @@ __be32 nfsd4_bind_conn_to_session(struct svc_rqst *rqstp, | |||
1636 | return nfserr_badsession; | 1636 | return nfserr_badsession; |
1637 | 1637 | ||
1638 | status = nfsd4_map_bcts_dir(&bcts->dir); | 1638 | status = nfsd4_map_bcts_dir(&bcts->dir); |
1639 | nfsd4_new_conn(rqstp, cstate->session, bcts->dir); | 1639 | if (!status) |
1640 | return nfs_ok; | 1640 | nfsd4_new_conn(rqstp, cstate->session, bcts->dir); |
1641 | return status; | ||
1641 | } | 1642 | } |
1642 | 1643 | ||
1643 | static bool nfsd4_compound_in_session(struct nfsd4_session *session, struct nfs4_sessionid *sid) | 1644 | static bool nfsd4_compound_in_session(struct nfsd4_session *session, struct nfs4_sessionid *sid) |