diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2013-04-10 15:43:00 +0400 |
|---|---|---|
| committer | Steve French <smfrench@gmail.com> | 2013-05-05 07:18:10 +0400 |
| commit | b13ce4bbcae6181eb9cdda5a5bf88a28b0acd7aa (patch) | |
| tree | 6c1115ee3e1a5eb4f5575e8e73c705fb2a301711 /fs/cifs/connect.c | |
| parent | efb79f289f0a7e7fbfc82f78959093bf7d1558b9 (diff) | |
| download | linux-b13ce4bbcae6181eb9cdda5a5bf88a28b0acd7aa.tar.xz | |
cifs: small variable name cleanup
server and ses->server are the same, but it's a little bit ugly that we
lock &ses->server->srv_mutex and unlock &server->srv_mutex. It causes
a false positive in Smatch about inconsistent locking.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Steve French <sfrench@us.ibm.com>
Signed-off-by: Steve French <smfrench@gmail.com>
Diffstat (limited to 'fs/cifs/connect.c')
| -rw-r--r-- | fs/cifs/connect.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/cifs/connect.c b/fs/cifs/connect.c index 9535abe3d24c..99eeaa17ee00 100644 --- a/fs/cifs/connect.c +++ b/fs/cifs/connect.c @@ -3836,7 +3836,7 @@ cifs_setup_session(const unsigned int xid, struct cifs_ses *ses, if (rc) { cifs_dbg(VFS, "Send error in SessSetup = %d\n", rc); } else { - mutex_lock(&ses->server->srv_mutex); + mutex_lock(&server->srv_mutex); if (!server->session_estab) { server->session_key.response = ses->auth_key.response; server->session_key.len = ses->auth_key.len; |