diff options
| author |   Paulo Alcantara <pc@cjr.nz> | 2022-06-24 15:01:43 -0300 |
|---|---|---|
| committer |   Steve French <stfrench@microsoft.com> | 2022-06-24 13:34:28 -0500 |
| commit | af3a6d1018f02c6dc8388f1f3785a559c7ab5961 (patch) | |
| tree | 1ea9d2bc699c8245c67fa976ab78573c72b98958 /fs/cifs/connect.c | |
| parent | cifs: avoid deadlocks while updating iface (diff) | |
| download | linux-dev-af3a6d1018f02c6dc8388f1f3785a559c7ab5961.tar.xz linux-dev-af3a6d1018f02c6dc8388f1f3785a559c7ab5961.zip | |
cifs: update cifs_ses::ip_addr after failover
cifs_ses::ip_addr wasn't being updated in cifs_session_setup() when
reconnecting SMB sessions thus returning wrong value in
/proc/fs/cifs/DebugData.
Signed-off-by: Paulo Alcantara (SUSE) <pc@cjr.nz>
Cc: stable@kernel.org
Signed-off-by: Steve French <stfrench@microsoft.com>
Diffstat (limited to 'fs/cifs/connect.c')
| -rw-r--r-- | fs/cifs/connect.c | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/fs/cifs/connect.c b/fs/cifs/connect.c index 8d56325915d0..fa29c9aae24b 100644 --- a/fs/cifs/connect.c +++ b/fs/cifs/connect.c @@ -4025,10 +4025,16 @@ cifs_setup_session(const unsigned int xid, struct cifs_ses *ses, struct nls_table *nls_info) { int rc = -ENOSYS; + struct sockaddr_in6 *addr6 = (struct sockaddr_in6 *)&server->dstaddr; + struct sockaddr_in *addr = (struct sockaddr_in *)&server->dstaddr; bool is_binding = false; - spin_lock(&cifs_tcp_ses_lock); + if (server->dstaddr.ss_family == AF_INET6) + scnprintf(ses->ip_addr, sizeof(ses->ip_addr), "%pI6", &addr6->sin6_addr); + else + scnprintf(ses->ip_addr, sizeof(ses->ip_addr), "%pI4", &addr->sin_addr); + if (ses->ses_status != SES_GOOD && ses->ses_status != SES_NEW && ses->ses_status != SES_NEED_RECON) { |