diff options
author | Paulo Alcantara (SUSE) <pc@cjr.nz> | 2019-11-22 18:30:53 +0300 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2020-01-09 12:20:03 +0300 |
commit | 3d29dc60aac16831b1822c831a7f40b45a9686d9 (patch) | |
tree | f0eafaed5b9951cd5758d3f0c10cd2673ac99267 /drivers | |
parent | 6eaf5653ddab9ce8572f50d2e477e8cddba4994d (diff) | |
download | linux-3d29dc60aac16831b1822c831a7f40b45a9686d9.tar.xz |
cifs: Fix potential softlockups while refreshing DFS cache
commit 84a1f5b1cc6fd7f6cd99fc5630c36f631b19fa60 upstream.
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>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers')
0 files changed, 0 insertions, 0 deletions