| From 8ea21823aa584b55ba4b861307093b78054b0c1b Mon Sep 17 00:00:00 2001 |
| From: Shyam Prasad N <sprasad@microsoft.com> |
| Date: Tue, 31 May 2022 12:31:05 +0000 |
| Subject: cifs: return errors during session setup during reconnects |
| |
| From: Shyam Prasad N <sprasad@microsoft.com> |
| |
| commit 8ea21823aa584b55ba4b861307093b78054b0c1b upstream. |
| |
| During reconnects, we check the return value from |
| cifs_negotiate_protocol, and have handlers for both success |
| and failures. But if that passes, and cifs_setup_session |
| returns any errors other than -EACCES, we do not handle |
| that. This fix adds a handler for that, so that we don't |
| go ahead and try a tree_connect on a failed session. |
| |
| Signed-off-by: Shyam Prasad N <sprasad@microsoft.com> |
| Reviewed-by: Enzo Matsumiya <ematsumiya@suse.de> |
| Cc: stable@vger.kernel.org |
| Signed-off-by: Steve French <stfrench@microsoft.com> |
| Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> |
| --- |
| fs/cifs/smb2pdu.c | 3 +++ |
| 1 file changed, 3 insertions(+) |
| |
| --- a/fs/cifs/smb2pdu.c |
| +++ b/fs/cifs/smb2pdu.c |
| @@ -265,6 +265,9 @@ smb2_reconnect(__le16 smb2_command, stru |
| rc = -EHOSTDOWN; |
| mutex_unlock(&tcon->ses->session_mutex); |
| goto failed; |
| + } else if (rc) { |
| + mutex_unlock(&ses->session_mutex); |
| + goto out; |
| } |
| } |
| if (rc || !tcon->need_reconnect) { |
