Commit 50f500b7 authored by Namjae Jeon's avatar Namjae Jeon Committed by Steve French
Browse files

ksmbd: remove filename in ksmbd_file 



If the filename is change by underlying rename the server, fp->filename
and real filename can be different. This patch remove the uses of
fp->filename in ksmbd and replace it with d_path().

Signed-off-by: default avatarNamjae Jeon <linkinjeon@kernel.org>
Reviewed-by: default avatarHyunchul Lee <hyc.lee@gmail.com>
Signed-off-by: default avatarSteve French <stfrench@microsoft.com>
parent ce522ba9

fs/ksmbd/misc.c

+31 −9
Original line number Diff line number Diff line
@@ -158,19 +158,41 @@ int parse_stream_name(char *filename, char **stream_name, int *s_type) 
 * Return : windows path string or error

 */



char *convert_to_nt_pathname(char *filename)

char *convert_to_nt_pathname(struct ksmbd_share_config *share,

			     struct path *path)

{

	char *ab_pathname;

	char *pathname, *ab_pathname, *nt_pathname;

	int share_path_len = share->path_sz;



	if (strlen(filename) == 0)

		filename = "\\";

	pathname = kmalloc(PATH_MAX, GFP_KERNEL);

	if (!pathname)

		return ERR_PTR(-EACCES);



	ab_pathname = kstrdup(filename, GFP_KERNEL);

	if (!ab_pathname)

		return NULL;

	ab_pathname = d_path(path, pathname, PATH_MAX);

	if (IS_ERR(ab_pathname)) {

		nt_pathname = ERR_PTR(-EACCES);

		goto free_pathname;

	}



	if (strncmp(ab_pathname, share->path, share_path_len)) {

		nt_pathname = ERR_PTR(-EACCES);

		goto free_pathname;

	}



	nt_pathname = kzalloc(strlen(&ab_pathname[share_path_len]) + 2, GFP_KERNEL);

	if (!nt_pathname) {

		nt_pathname = ERR_PTR(-ENOMEM);

		goto free_pathname;

	}

	if (ab_pathname[share_path_len] == '\0')

		strcpy(nt_pathname, "/");

	strcat(nt_pathname, &ab_pathname[share_path_len]);



	ksmbd_conv_path_to_windows(nt_pathname);



	ksmbd_conv_path_to_windows(ab_pathname);

	return ab_pathname;

free_pathname:

	kfree(pathname);

	return nt_pathname;

}



int get_nlink(struct kstat *st)

fs/ksmbd/misc.h

+2 −1
Original line number Diff line number Diff line
@@ -14,7 +14,8 @@ struct ksmbd_file; 
int match_pattern(const char *str, size_t len, const char *pattern);

int ksmbd_validate_filename(char *filename);

int parse_stream_name(char *filename, char **stream_name, int *s_type);

char *convert_to_nt_pathname(char *filename);

char *convert_to_nt_pathname(struct ksmbd_share_config *share,

			     struct path *path);

int get_nlink(struct kstat *st);

void ksmbd_conv_path_to_unix(char *path);

void ksmbd_strip_last_slash(char *path);

fs/ksmbd/oplock.c

+0 −30
Original line number Diff line number Diff line
@@ -1694,33 +1694,3 @@ struct oplock_info *lookup_lease_in_table(struct ksmbd_conn *conn, 
	read_unlock(&lease_list_lock);

	return ret_op;

}
 


int smb2_check_durable_oplock(struct ksmbd_file *fp,

			      struct lease_ctx_info *lctx, char *name)

{

	struct oplock_info *opinfo = opinfo_get(fp);

	int ret = 0;



	if (opinfo && opinfo->is_lease) {

		if (!lctx) {

			pr_err("open does not include lease\n");

			ret = -EBADF;

			goto out;

		}

		if (memcmp(opinfo->o_lease->lease_key, lctx->lease_key,

			   SMB2_LEASE_KEY_SIZE)) {

			pr_err("invalid lease key\n");

			ret = -EBADF;

			goto out;

		}

		if (name && strcmp(fp->filename, name)) {

			pr_err("invalid name reconnect %s\n", name);

			ret = -EINVAL;

			goto out;

		}

	}

out:

	if (opinfo)

		opinfo_put(opinfo);

	return ret;

}

fs/ksmbd/oplock.h

+0 −2
Original line number Diff line number Diff line
@@ -124,6 +124,4 @@ struct oplock_info *lookup_lease_in_table(struct ksmbd_conn *conn, 
int find_same_lease_key(struct ksmbd_session *sess, struct ksmbd_inode *ci,

			struct lease_ctx_info *lctx);

void destroy_lease_table(struct ksmbd_conn *conn);

int smb2_check_durable_oplock(struct ksmbd_file *fp,

			      struct lease_ctx_info *lctx, char *name);

#endif /* __KSMBD_OPLOCK_H */

fs/ksmbd/smb2pdu.c

+7 −14
Original line number Diff line number Diff line
@@ -2918,7 +2918,6 @@ int smb2_open(struct ksmbd_work *work) 
		goto err_out;

	}



	fp->filename = name;

	fp->cdoption = req->CreateDisposition;

	fp->daccess = daccess;

	fp->saccess = req->ShareAccess;
@@ -3270,14 +3269,13 @@ int smb2_open(struct ksmbd_work *work) 
		if (!rsp->hdr.Status)

			rsp->hdr.Status = STATUS_UNEXPECTED_IO_ERROR;



		if (!fp || !fp->filename)

			kfree(name);

		if (fp)

			ksmbd_fd_put(work, fp);

		smb2_set_err_rsp(work);

		ksmbd_debug(SMB, "Error response: %x\n", rsp->hdr.Status);

	}



	kfree(name);

	kfree(lc);



	return 0;
@@ -3895,8 +3893,6 @@ int smb2_query_dir(struct ksmbd_work *work) 
		ksmbd_debug(SMB, "Search pattern is %s\n", srch_ptr);

	}



	ksmbd_debug(SMB, "Directory name is %s\n", dir_fp->filename);



	if (srch_flag & SMB2_REOPEN || srch_flag & SMB2_RESTART_SCANS) {

		ksmbd_debug(SMB, "Restart directory scan\n");

		generic_file_llseek(dir_fp->filp, 0, SEEK_SET);
@@ -4390,9 +4386,9 @@ static int get_file_all_info(struct ksmbd_work *work, 
		return -EACCES;

	}



	filename = convert_to_nt_pathname(fp->filename);

	if (!filename)

		return -ENOMEM;

	filename = convert_to_nt_pathname(work->tcon->share_conf, &fp->filp->f_path);

	if (IS_ERR(filename))

		return PTR_ERR(filename);



	inode = file_inode(fp->filp);

	generic_fillattr(file_mnt_user_ns(fp->filp), inode, &stat);
@@ -5683,8 +5679,7 @@ static int set_file_allocation_info(struct ksmbd_work *work, 
		size = i_size_read(inode);

		rc = ksmbd_vfs_truncate(work, fp, alloc_blks * 512);

		if (rc) {

			pr_err("truncate failed! filename : %s, err %d\n",

			       fp->filename, rc);

			pr_err("truncate failed!, err %d\n", rc);

			return rc;

		}

		if (size < alloc_blks * 512)
@@ -5714,12 +5709,10 @@ static int set_end_of_file_info(struct ksmbd_work *work, struct ksmbd_file *fp, 
	 * truncated range.

	 */

	if (inode->i_sb->s_magic != MSDOS_SUPER_MAGIC) {

		ksmbd_debug(SMB, "filename : %s truncated to newsize %lld\n",

			    fp->filename, newsize);

		ksmbd_debug(SMB, "truncated to newsize %lld\n", newsize);

		rc = ksmbd_vfs_truncate(work, fp, newsize);

		if (rc) {

			ksmbd_debug(SMB, "truncate failed! filename : %s err %d\n",

				    fp->filename, rc);

			ksmbd_debug(SMB, "truncate failed!, err %d\n", rc);

			if (rc != -EAGAIN)

				rc = -EBADF;

			return rc;