smb: client: fix potential race in cifs_put_tcon()
commitc32b624fa4
upstream. dfs_cache_refresh() delayed worker could race with cifs_put_tcon(), so make sure to call list_replace_init() on @tcon->dfs_ses_list after kworker is cancelled or finished. Fixes:4f42a8b54b
("smb: client: fix DFS interlink failover") Signed-off-by: Paulo Alcantara (Red Hat) <pc@manguebit.com> Signed-off-by: Steve French <stfrench@microsoft.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> (cherry picked from commit d3927e55c95976733d5986e185c1e3a39cdadd0c)
This commit is contained in:
parent
740b5f2300
commit
2179676f1c
|
@ -2557,9 +2557,6 @@ cifs_put_tcon(struct cifs_tcon *tcon, enum smb3_tcon_ref_trace trace)
|
|||
|
||||
list_del_init(&tcon->tcon_list);
|
||||
tcon->status = TID_EXITING;
|
||||
#ifdef CONFIG_CIFS_DFS_UPCALL
|
||||
list_replace_init(&tcon->dfs_ses_list, &ses_list);
|
||||
#endif
|
||||
spin_unlock(&tcon->tc_lock);
|
||||
spin_unlock(&cifs_tcp_ses_lock);
|
||||
|
||||
|
@ -2567,6 +2564,7 @@ cifs_put_tcon(struct cifs_tcon *tcon, enum smb3_tcon_ref_trace trace)
|
|||
cancel_delayed_work_sync(&tcon->query_interfaces);
|
||||
#ifdef CONFIG_CIFS_DFS_UPCALL
|
||||
cancel_delayed_work_sync(&tcon->dfs_cache_work);
|
||||
list_replace_init(&tcon->dfs_ses_list, &ses_list);
|
||||
#endif
|
||||
|
||||
if (tcon->use_witness) {
|
||||
|
|
Loading…
Reference in New Issue