diff options
| author | Paulo Alcantara (SUSE) <pc@cjr.nz> | 2019-11-22 18:30:53 +0300 |
|---|---|---|
| committer | Steve French <stfrench@microsoft.com> | 2019-11-25 18:33:04 +0300 |
| commit | 84a1f5b1cc6fd7f6cd99fc5630c36f631b19fa60 (patch) | |
| tree | 58346cce7b4f68ef8dc41b39c09c9885436750c4 /fs/cifs/connect.c | |
| parent | df3df923b31d298c3d3653a0380202b9f2df9864 (diff) | |
| download | linux-84a1f5b1cc6fd7f6cd99fc5630c36f631b19fa60.tar.xz | |
cifs: Fix potential softlockups while refreshing DFS cache
We used to skip reconnects on all SMB2_IOCTL commands due to SMB3+
FSCTL_VALIDATE_NEGOTIATE_INFO - which made sense since we're still
establishing a SMB session.
However, when refresh_cache_worker() calls smb2_get_dfs_refer() and
we're under reconnect, SMB2_ioctl() will not be able to get a proper
status error (e.g. -EHOSTDOWN in case we failed to reconnect) but an
-EAGAIN from cifs_send_recv() thus looping forever in
refresh_cache_worker().
Fixes: e99c63e4d86d ("SMB3: Fix deadlock in validate negotiate hits reconnect")
Signed-off-by: Paulo Alcantara (SUSE) <pc@cjr.nz>
Suggested-by: Aurelien Aptel <aaptel@suse.com>
Reviewed-by: Aurelien Aptel <aaptel@suse.com>
Signed-off-by: Steve French <stfrench@microsoft.com>
Diffstat (limited to 'fs/cifs/connect.c')
0 files changed, 0 insertions, 0 deletions