Commit a421e3fe authored by wangrong's avatar wangrong Committed by Steve French

smb: client: use actual path when queryfs

Due to server permission control, the client does not have access to
the shared root directory, but can access subdirectories normally, so
users usually mount the shared subdirectories directly. In this case,
queryfs should use the actual path instead of the root directory to
avoid the call returning an error (EACCES).
Signed-off-by: default avatarwangrong <wangrong@uniontech.com>
Reviewed-by: default avatarPaulo Alcantara (Red Hat) <pc@manguebit.com>
Cc: stable@vger.kernel.org
Signed-off-by: default avatarSteve French <stfrench@microsoft.com>
parent c9432ad5
...@@ -313,8 +313,17 @@ cifs_statfs(struct dentry *dentry, struct kstatfs *buf) ...@@ -313,8 +313,17 @@ cifs_statfs(struct dentry *dentry, struct kstatfs *buf)
struct TCP_Server_Info *server = tcon->ses->server; struct TCP_Server_Info *server = tcon->ses->server;
unsigned int xid; unsigned int xid;
int rc = 0; int rc = 0;
const char *full_path;
void *page;
xid = get_xid(); xid = get_xid();
page = alloc_dentry_path();
full_path = build_path_from_dentry(dentry, page);
if (IS_ERR(full_path)) {
rc = PTR_ERR(full_path);
goto statfs_out;
}
if (le32_to_cpu(tcon->fsAttrInfo.MaxPathNameComponentLength) > 0) if (le32_to_cpu(tcon->fsAttrInfo.MaxPathNameComponentLength) > 0)
buf->f_namelen = buf->f_namelen =
...@@ -330,8 +339,10 @@ cifs_statfs(struct dentry *dentry, struct kstatfs *buf) ...@@ -330,8 +339,10 @@ cifs_statfs(struct dentry *dentry, struct kstatfs *buf)
buf->f_ffree = 0; /* unlimited */ buf->f_ffree = 0; /* unlimited */
if (server->ops->queryfs) if (server->ops->queryfs)
rc = server->ops->queryfs(xid, tcon, cifs_sb, buf); rc = server->ops->queryfs(xid, tcon, full_path, cifs_sb, buf);
statfs_out:
free_dentry_path(page);
free_xid(xid); free_xid(xid);
return rc; return rc;
} }
......
...@@ -481,7 +481,7 @@ struct smb_version_operations { ...@@ -481,7 +481,7 @@ struct smb_version_operations {
__u16 net_fid, struct cifsInodeInfo *cifs_inode); __u16 net_fid, struct cifsInodeInfo *cifs_inode);
/* query remote filesystem */ /* query remote filesystem */
int (*queryfs)(const unsigned int, struct cifs_tcon *, int (*queryfs)(const unsigned int, struct cifs_tcon *,
struct cifs_sb_info *, struct kstatfs *); const char *, struct cifs_sb_info *, struct kstatfs *);
/* send mandatory brlock to the server */ /* send mandatory brlock to the server */
int (*mand_lock)(const unsigned int, struct cifsFileInfo *, __u64, int (*mand_lock)(const unsigned int, struct cifsFileInfo *, __u64,
__u64, __u32, int, int, bool); __u64, __u32, int, int, bool);
......
...@@ -909,7 +909,7 @@ cifs_oplock_response(struct cifs_tcon *tcon, __u64 persistent_fid, ...@@ -909,7 +909,7 @@ cifs_oplock_response(struct cifs_tcon *tcon, __u64 persistent_fid,
static int static int
cifs_queryfs(const unsigned int xid, struct cifs_tcon *tcon, cifs_queryfs(const unsigned int xid, struct cifs_tcon *tcon,
struct cifs_sb_info *cifs_sb, struct kstatfs *buf) const char *path, struct cifs_sb_info *cifs_sb, struct kstatfs *buf)
{ {
int rc = -EOPNOTSUPP; int rc = -EOPNOTSUPP;
......
...@@ -2838,7 +2838,7 @@ smb2_query_info_compound(const unsigned int xid, struct cifs_tcon *tcon, ...@@ -2838,7 +2838,7 @@ smb2_query_info_compound(const unsigned int xid, struct cifs_tcon *tcon,
static int static int
smb2_queryfs(const unsigned int xid, struct cifs_tcon *tcon, smb2_queryfs(const unsigned int xid, struct cifs_tcon *tcon,
struct cifs_sb_info *cifs_sb, struct kstatfs *buf) const char *path, struct cifs_sb_info *cifs_sb, struct kstatfs *buf)
{ {
struct smb2_query_info_rsp *rsp; struct smb2_query_info_rsp *rsp;
struct smb2_fs_full_size_info *info = NULL; struct smb2_fs_full_size_info *info = NULL;
...@@ -2847,7 +2847,7 @@ smb2_queryfs(const unsigned int xid, struct cifs_tcon *tcon, ...@@ -2847,7 +2847,7 @@ smb2_queryfs(const unsigned int xid, struct cifs_tcon *tcon,
int rc; int rc;
rc = smb2_query_info_compound(xid, tcon, "", rc = smb2_query_info_compound(xid, tcon, path,
FILE_READ_ATTRIBUTES, FILE_READ_ATTRIBUTES,
FS_FULL_SIZE_INFORMATION, FS_FULL_SIZE_INFORMATION,
SMB2_O_INFO_FILESYSTEM, SMB2_O_INFO_FILESYSTEM,
...@@ -2875,28 +2875,33 @@ smb2_queryfs(const unsigned int xid, struct cifs_tcon *tcon, ...@@ -2875,28 +2875,33 @@ smb2_queryfs(const unsigned int xid, struct cifs_tcon *tcon,
static int static int
smb311_queryfs(const unsigned int xid, struct cifs_tcon *tcon, smb311_queryfs(const unsigned int xid, struct cifs_tcon *tcon,
struct cifs_sb_info *cifs_sb, struct kstatfs *buf) const char *path, struct cifs_sb_info *cifs_sb, struct kstatfs *buf)
{ {
int rc; int rc;
__le16 srch_path = 0; /* Null - open root of share */ __le16 *utf16_path = NULL;
u8 oplock = SMB2_OPLOCK_LEVEL_NONE; u8 oplock = SMB2_OPLOCK_LEVEL_NONE;
struct cifs_open_parms oparms; struct cifs_open_parms oparms;
struct cifs_fid fid; struct cifs_fid fid;
if (!tcon->posix_extensions) if (!tcon->posix_extensions)
return smb2_queryfs(xid, tcon, cifs_sb, buf); return smb2_queryfs(xid, tcon, path, cifs_sb, buf);
oparms = (struct cifs_open_parms) { oparms = (struct cifs_open_parms) {
.tcon = tcon, .tcon = tcon,
.path = "", .path = path,
.desired_access = FILE_READ_ATTRIBUTES, .desired_access = FILE_READ_ATTRIBUTES,
.disposition = FILE_OPEN, .disposition = FILE_OPEN,
.create_options = cifs_create_options(cifs_sb, 0), .create_options = cifs_create_options(cifs_sb, 0),
.fid = &fid, .fid = &fid,
}; };
rc = SMB2_open(xid, &oparms, &srch_path, &oplock, NULL, NULL, utf16_path = cifs_convert_path_to_utf16(path, cifs_sb);
if (utf16_path == NULL)
return -ENOMEM;
rc = SMB2_open(xid, &oparms, utf16_path, &oplock, NULL, NULL,
NULL, NULL); NULL, NULL);
kfree(utf16_path);
if (rc) if (rc)
return rc; return rc;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment