diff options
| author |   Shyam Prasad N <sprasad@microsoft.com> | 2023-06-09 17:46:54 +0000 |
|---|---|---|
| committer |   Steve French <stfrench@microsoft.com> | 2023-06-11 20:52:48 -0500 |
| commit | 91f4480c41f56f7c723323cf7f581f1d95d9ffbc (patch) | |
| tree | d67702910335ed217ddcfa87c7162c78a4f4eb83 /fs/smb/client/dfs.c | |
| parent | smb: remove obsolete comment (diff) | |
| download | wireguard-linux-91f4480c41f56f7c723323cf7f581f1d95d9ffbc.tar.xz wireguard-linux-91f4480c41f56f7c723323cf7f581f1d95d9ffbc.zip | |
cifs: fix status checks in cifs_tree_connect
The ordering of status checks at the beginning of
cifs_tree_connect is wrong. As a result, a tcon
which is good may stay marked as needing reconnect
infinitely.
Fixes: 2f0e4f034220 ("cifs: check only tcon status on tcon related functions")
Cc: stable@vger.kernel.org # 6.3
Signed-off-by: Shyam Prasad N <sprasad@microsoft.com>
Signed-off-by: Steve French <stfrench@microsoft.com>
Diffstat (limited to '')
| -rw-r--r-- | fs/smb/client/dfs.c | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/fs/smb/client/dfs.c b/fs/smb/client/dfs.c index 2f93bf8c3325..2390b2fedd6a 100644 --- a/fs/smb/client/dfs.c +++ b/fs/smb/client/dfs.c @@ -575,16 +575,17 @@ int cifs_tree_connect(const unsigned int xid, struct cifs_tcon *tcon, const stru /* only send once per connect */ spin_lock(&tcon->tc_lock); + if (tcon->status == TID_GOOD) { + spin_unlock(&tcon->tc_lock); + return 0; + } + if (tcon->status != TID_NEW && tcon->status != TID_NEED_TCON) { spin_unlock(&tcon->tc_lock); return -EHOSTDOWN; } - if (tcon->status == TID_GOOD) { - spin_unlock(&tcon->tc_lock); - return 0; - } tcon->status = TID_IN_TCON; spin_unlock(&tcon->tc_lock); |