Commit e6eb1950 authored by Ronnie Sahlberg's avatar Ronnie Sahlberg Committed by Steve French

cifs: pass a path to open_shroot and check if it is the root or not

Move the check for the directory path into the open_shroot() function
but still fail for any non-root directories.
This is preparation for later when we will start using the cache also
for other directories than the root.
Signed-off-by: default avatarRonnie Sahlberg <lsahlber@redhat.com>
Signed-off-by: default avatarSteve French <stfrench@microsoft.com>
parent 4df3d976
...@@ -524,22 +524,20 @@ smb2_query_path_info(const unsigned int xid, struct cifs_tcon *tcon, ...@@ -524,22 +524,20 @@ smb2_query_path_info(const unsigned int xid, struct cifs_tcon *tcon,
return -ENOMEM; return -ENOMEM;
/* If it is a root and its handle is cached then use it */ /* If it is a root and its handle is cached then use it */
if (!strlen(full_path)) { rc = open_shroot(xid, tcon, full_path, cifs_sb, &cfid);
rc = open_shroot(xid, tcon, cifs_sb, &cfid); if (!rc) {
if (!rc) { if (tcon->crfid.file_all_info_is_valid) {
if (tcon->crfid.file_all_info_is_valid) { move_smb2_info_to_cifs(data,
move_smb2_info_to_cifs(data,
&tcon->crfid.file_all_info); &tcon->crfid.file_all_info);
} else { } else {
rc = SMB2_query_info(xid, tcon, rc = SMB2_query_info(xid, tcon,
cfid->fid->persistent_fid, cfid->fid->persistent_fid,
cfid->fid->volatile_fid, smb2_data); cfid->fid->volatile_fid, smb2_data);
if (!rc) if (!rc)
move_smb2_info_to_cifs(data, smb2_data); move_smb2_info_to_cifs(data, smb2_data);
}
close_shroot(cfid);
goto out;
} }
close_shroot(cfid);
goto out;
} }
cifs_get_readable_path(tcon, full_path, &cfile); cifs_get_readable_path(tcon, full_path, &cfile);
......
...@@ -728,6 +728,7 @@ smb2_cached_lease_break(struct work_struct *work) ...@@ -728,6 +728,7 @@ smb2_cached_lease_break(struct work_struct *work)
* Open the directory at the root of a share * Open the directory at the root of a share
*/ */
int open_shroot(unsigned int xid, struct cifs_tcon *tcon, int open_shroot(unsigned int xid, struct cifs_tcon *tcon,
const char *path,
struct cifs_sb_info *cifs_sb, struct cifs_sb_info *cifs_sb,
struct cached_fid **cfid) struct cached_fid **cfid)
{ {
...@@ -749,6 +750,9 @@ int open_shroot(unsigned int xid, struct cifs_tcon *tcon, ...@@ -749,6 +750,9 @@ int open_shroot(unsigned int xid, struct cifs_tcon *tcon,
if (tcon->nohandlecache) if (tcon->nohandlecache)
return -ENOTSUPP; return -ENOTSUPP;
if (strlen(path))
return -ENOTSUPP;
mutex_lock(&tcon->crfid.fid_mutex); mutex_lock(&tcon->crfid.fid_mutex);
if (tcon->crfid.is_valid) { if (tcon->crfid.is_valid) {
cifs_dbg(FYI, "found a cached root file handle\n"); cifs_dbg(FYI, "found a cached root file handle\n");
...@@ -926,7 +930,7 @@ smb3_qfs_tcon(const unsigned int xid, struct cifs_tcon *tcon, ...@@ -926,7 +930,7 @@ smb3_qfs_tcon(const unsigned int xid, struct cifs_tcon *tcon,
oparms.fid = &fid; oparms.fid = &fid;
oparms.reconnect = false; oparms.reconnect = false;
rc = open_shroot(xid, tcon, cifs_sb, &cfid); rc = open_shroot(xid, tcon, "", cifs_sb, &cfid);
if (rc == 0) if (rc == 0)
memcpy(&fid, cfid->fid, sizeof(struct cifs_fid)); memcpy(&fid, cfid->fid, sizeof(struct cifs_fid));
else else
......
...@@ -70,6 +70,7 @@ extern int smb3_handle_read_data(struct TCP_Server_Info *server, ...@@ -70,6 +70,7 @@ extern int smb3_handle_read_data(struct TCP_Server_Info *server,
struct mid_q_entry *mid); struct mid_q_entry *mid);
extern int open_shroot(unsigned int xid, struct cifs_tcon *tcon, extern int open_shroot(unsigned int xid, struct cifs_tcon *tcon,
const char *path,
struct cifs_sb_info *cifs_sb, struct cifs_sb_info *cifs_sb,
struct cached_fid **cfid); struct cached_fid **cfid);
extern void close_shroot(struct cached_fid *cfid); extern void close_shroot(struct cached_fid *cfid);
......
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