diff options
author | Shyam Prasad N <sprasad@microsoft.com> | 2024-01-23 05:07:57 +0000 |
---|---|---|
committer | Steve French <stfrench@microsoft.com> | 2024-01-23 20:23:29 -0600 |
commit | 993d1c346b1a51ac41b2193609a0d4e51e9748f4 (patch) | |
tree | 5ab638098cc2c96eae712a25416a4b8b1d905923 /fs | |
parent | 4cdad80261862c8cdcbb5fd232aa713d0bdefe24 (diff) |
cifs: fix stray unlock in cifs_chan_skip_or_disable
A recent change moved the code that decides to skip
a channel or disable multichannel entirely, into a
helper function.
During this, a mutex_unlock of the session_mutex
should have been removed. Doing that here.
Fixes: f591062bdbf4 ("cifs: handle servers that still advertise multichannel after disabling")
Signed-off-by: Shyam Prasad N <sprasad@microsoft.com>
Signed-off-by: Steve French <stfrench@microsoft.com>
Diffstat (limited to 'fs')
-rw-r--r-- | fs/smb/client/smb2pdu.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/fs/smb/client/smb2pdu.c b/fs/smb/client/smb2pdu.c index 4f2cc8373b67..86f6f35b7f32 100644 --- a/fs/smb/client/smb2pdu.c +++ b/fs/smb/client/smb2pdu.c @@ -195,7 +195,6 @@ cifs_chan_skip_or_disable(struct cifs_ses *ses, pserver = server->primary_server; cifs_signal_cifsd_for_reconnect(pserver, false); skip_terminate: - mutex_unlock(&ses->session_mutex); return -EHOSTDOWN; } |