diff options
| author | Namjae Jeon <linkinjeon@kernel.org> | 2022-07-22 10:15:57 +0900 |
|---|---|---|
| committer | Steve French <stfrench@microsoft.com> | 2022-07-26 23:38:05 -0500 |
| commit | 8e06b31e348107c5d78e2c90bb7e69388cb97fb6 (patch) | |
| tree | 374e7f18ad234a149295de41681ba0f28d07034b /fs/ksmbd/mgmt | |
| parent | e4d3e6b524c0c928f7fc9e03e047885c4951ae60 (diff) | |
| download | linux-8e06b31e348107c5d78e2c90bb7e69388cb97fb6.tar.gz | |
ksmbd: add channel rwlock
Add missing rwlock for channel list in session.
Signed-off-by: Namjae Jeon <linkinjeon@kernel.org>
Reviewed-by: Hyunchul Lee <hyc.lee@gmail.com>
Signed-off-by: Steve French <stfrench@microsoft.com>
Diffstat (limited to 'fs/ksmbd/mgmt')
| -rw-r--r-- | fs/ksmbd/mgmt/user_session.c | 3 | ||||
| -rw-r--r-- | fs/ksmbd/mgmt/user_session.h | 1 |
2 files changed, 4 insertions, 0 deletions
diff --git a/fs/ksmbd/mgmt/user_session.c b/fs/ksmbd/mgmt/user_session.c index 3a44e66456fc..25e9ba3b7550 100644 --- a/fs/ksmbd/mgmt/user_session.c +++ b/fs/ksmbd/mgmt/user_session.c @@ -32,11 +32,13 @@ static void free_channel_list(struct ksmbd_session *sess) { struct channel *chann, *tmp; + write_lock(&sess->chann_lock); list_for_each_entry_safe(chann, tmp, &sess->ksmbd_chann_list, chann_list) { list_del(&chann->chann_list); kfree(chann); } + write_unlock(&sess->chann_lock); } static void __session_rpc_close(struct ksmbd_session *sess, @@ -303,6 +305,7 @@ static struct ksmbd_session *__session_create(int protocol) INIT_LIST_HEAD(&sess->rpc_handle_list); sess->sequence_number = 1; atomic_set(&sess->refcnt, 1); + rwlock_init(&sess->chann_lock); switch (protocol) { case CIFDS_SESSION_FLAG_SMB2: diff --git a/fs/ksmbd/mgmt/user_session.h b/fs/ksmbd/mgmt/user_session.h index 8b08189be3fc..1ec659f0151b 100644 --- a/fs/ksmbd/mgmt/user_session.h +++ b/fs/ksmbd/mgmt/user_session.h @@ -48,6 +48,7 @@ struct ksmbd_session { char sess_key[CIFS_KEY_SIZE]; struct hlist_node hlist; + rwlock_t chann_lock; struct list_head ksmbd_chann_list; struct xarray tree_conns; struct ida tree_conn_ida; |