ksmbd: Remove unused parameter from smb2_get_name()
authorMarios Makassikis <mmakassikis@freebox.fr>
Wed, 1 Dec 2021 20:41:19 +0000 (21:41 +0100)
committerSteve French <stfrench@microsoft.com>
Wed, 29 Dec 2021 04:47:22 +0000 (22:47 -0600)
The 'share' parameter is no longer used by smb2_get_name() since
commit 265fd1991c1d ("ksmbd: use LOOKUP_BENEATH to prevent the out of
share access").

Acked-by: Namjae Jeon <linkinjeon@kernel.org>
Signed-off-by: Marios Makassikis <mmakassikis@freebox.fr>
Signed-off-by: Steve French <stfrench@microsoft.com>
fs/ksmbd/smb2pdu.c

index b8b3a4c28b749a9317d57511aae3a49d5739db8f..225f2bcb46b6c9528422a3b77b803b83aaf70eca 100644 (file)
@@ -610,7 +610,6 @@ static void destroy_previous_session(struct ksmbd_user *user, u64 id)
 
 /**
  * smb2_get_name() - get filename string from on the wire smb format
- * @share:     ksmbd_share_config pointer
  * @src:       source buffer
  * @maxlen:    maxlen of source string
  * @nls_table: nls_table pointer
@@ -618,8 +617,7 @@ static void destroy_previous_session(struct ksmbd_user *user, u64 id)
  * Return:      matching converted filename on success, otherwise error ptr
  */
 static char *
-smb2_get_name(struct ksmbd_share_config *share, const char *src,
-             const int maxlen, struct nls_table *local_nls)
+smb2_get_name(const char *src, const int maxlen, struct nls_table *local_nls)
 {
        char *name;
 
@@ -2530,8 +2528,7 @@ int smb2_open(struct ksmbd_work *work)
                        goto err_out1;
                }
 
-               name = smb2_get_name(share,
-                                    req->Buffer,
+               name = smb2_get_name(req->Buffer,
                                     le16_to_cpu(req->NameLength),
                                     work->conn->local_nls);
                if (IS_ERR(name)) {
@@ -5398,8 +5395,7 @@ static int smb2_rename(struct ksmbd_work *work,
                goto out;
        }
 
-       new_name = smb2_get_name(share,
-                                file_info->FileName,
+       new_name = smb2_get_name(file_info->FileName,
                                 le32_to_cpu(file_info->FileNameLength),
                                 local_nls);
        if (IS_ERR(new_name)) {
@@ -5510,8 +5506,7 @@ static int smb2_create_link(struct ksmbd_work *work,
        if (!pathname)
                return -ENOMEM;
 
-       link_name = smb2_get_name(share,
-                                 file_info->FileName,
+       link_name = smb2_get_name(file_info->FileName,
                                  le32_to_cpu(file_info->FileNameLength),
                                  local_nls);
        if (IS_ERR(link_name) || S_ISDIR(file_inode(filp)->i_mode)) {