diff options
| author | Shyam Prasad N <sprasad@microsoft.com> | 2024-01-23 08:07:57 +0300 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2024-02-01 03:19:13 +0300 |
| commit | d69a84cbbc30c7c386ad620b7704ce423d7fca03 (patch) | |
| tree | 9e49f43e29500349f763a2c22a987190c2106ebd /fs | |
| parent | 1bd81374bc2fa7ae98824b4eb9ac3e97c29d6971 (diff) | |
| download | linux-d69a84cbbc30c7c386ad620b7704ce423d7fca03.tar.xz | |
cifs: fix stray unlock in cifs_chan_skip_or_disable
[ Upstream commit 993d1c346b1a51ac41b2193609a0d4e51e9748f4 ]
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>
Signed-off-by: Sasha Levin <sashal@kernel.org>
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 bfec2ca0f4e6..f5006aa97f5b 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; } |