cifs: all initializations for tcon should happen in tcon_info_alloc
[ Upstream commit 74ebd02163fde05baa23129e06dde4b8f0f2377a ] Today, a few work structs inside tcon are initialized inside cifs_get_tcon and not in tcon_info_alloc. As a result, if a tcon is obtained from tcon_info_alloc, but not called as a part of cifs_get_tcon, we may trip over. Cc: <stable@vger.kernel.org> Signed-off-by: Shyam Prasad N <sprasad@microsoft.com> Reviewed-by: Paulo Alcantara (Red Hat) <pc@manguebit.org> Signed-off-by: Steve French <stfrench@microsoft.com> Signed-off-by: Sasha Levin <sashal@kernel.org> (cherry picked from commit 739296467a59c2089477f0a709fa28bf496d4621)
This commit is contained in:
parent
ae5d6a1092
commit
418c35bd19
|
@ -136,6 +136,7 @@ extern int SendReceiveBlockingLock(const unsigned int xid,
|
|||
struct smb_hdr *out_buf,
|
||||
int *bytes_returned);
|
||||
|
||||
void smb2_query_server_interfaces(struct work_struct *work);
|
||||
void
|
||||
cifs_signal_cifsd_for_reconnect(struct TCP_Server_Info *server,
|
||||
bool all_channels);
|
||||
|
|
|
@ -113,7 +113,7 @@ static int reconn_set_ipaddr_from_hostname(struct TCP_Server_Info *server)
|
|||
return rc;
|
||||
}
|
||||
|
||||
static void smb2_query_server_interfaces(struct work_struct *work)
|
||||
void smb2_query_server_interfaces(struct work_struct *work)
|
||||
{
|
||||
int rc;
|
||||
int xid;
|
||||
|
@ -2818,20 +2818,14 @@ cifs_get_tcon(struct cifs_ses *ses, struct smb3_fs_context *ctx)
|
|||
tcon->max_cached_dirs = ctx->max_cached_dirs;
|
||||
tcon->nodelete = ctx->nodelete;
|
||||
tcon->local_lease = ctx->local_lease;
|
||||
INIT_LIST_HEAD(&tcon->pending_opens);
|
||||
tcon->status = TID_GOOD;
|
||||
|
||||
INIT_DELAYED_WORK(&tcon->query_interfaces,
|
||||
smb2_query_server_interfaces);
|
||||
if (ses->server->dialect >= SMB30_PROT_ID &&
|
||||
(ses->server->capabilities & SMB2_GLOBAL_CAP_MULTI_CHANNEL)) {
|
||||
/* schedule query interfaces poll */
|
||||
queue_delayed_work(cifsiod_wq, &tcon->query_interfaces,
|
||||
(SMB_INTERFACE_POLL_INTERVAL * HZ));
|
||||
}
|
||||
#ifdef CONFIG_CIFS_DFS_UPCALL
|
||||
INIT_DELAYED_WORK(&tcon->dfs_cache_work, dfs_cache_refresh);
|
||||
#endif
|
||||
spin_lock(&cifs_tcp_ses_lock);
|
||||
list_add(&tcon->tcon_list, &ses->tcon_list);
|
||||
spin_unlock(&cifs_tcp_ses_lock);
|
||||
|
|
|
@ -148,6 +148,12 @@ tcon_info_alloc(bool dir_leases_enabled, enum smb3_tcon_ref_trace trace)
|
|||
#ifdef CONFIG_CIFS_DFS_UPCALL
|
||||
INIT_LIST_HEAD(&ret_buf->dfs_ses_list);
|
||||
#endif
|
||||
INIT_LIST_HEAD(&ret_buf->pending_opens);
|
||||
INIT_DELAYED_WORK(&ret_buf->query_interfaces,
|
||||
smb2_query_server_interfaces);
|
||||
#ifdef CONFIG_CIFS_DFS_UPCALL
|
||||
INIT_DELAYED_WORK(&ret_buf->dfs_cache_work, dfs_cache_refresh);
|
||||
#endif
|
||||
|
||||
return ret_buf;
|
||||
}
|
||||
|
|
Loading…
Reference in New Issue