CIFS: Move getting dfs referalls to ops struct
authorPavel Shilovsky <pshilovsky@samba.org>
Sun, 27 May 2012 16:21:53 +0000 (20:21 +0400)
committerPavel Shilovsky <pshilovsky@samba.org>
Tue, 24 Jul 2012 17:55:01 +0000 (21:55 +0400)
Signed-off-by: Pavel Shilovsky <pshilovsky@samba.org>
Signed-off-by: Steve French <smfrench@gmail.com>
fs/cifs/cifsglob.h
fs/cifs/cifsproto.h
fs/cifs/cifssmb.c
fs/cifs/connect.c
fs/cifs/smb1ops.c

index ef4e0a0bc826f1663116873559d5e46554d87b46..2d80d82f41d043a613cc0842dcf3059098d46379 100644 (file)
@@ -162,6 +162,7 @@ struct TCP_Server_Info;
 struct cifsFileInfo;
 struct cifs_ses;
 struct cifs_tcon;
+struct dfs_info3_param;
 
 struct smb_version_operations {
        int (*send_cancel)(struct TCP_Server_Info *, void *,
@@ -208,6 +209,10 @@ struct smb_version_operations {
                            struct cifs_tcon *, const struct nls_table *);
        /* close tree connecion */
        int (*tree_disconnect)(const unsigned int, struct cifs_tcon *);
+       /* get DFS referrals */
+       int (*get_dfs_refer)(const unsigned int, struct cifs_ses *,
+                            const char *, struct dfs_info3_param **,
+                            unsigned int *, const struct nls_table *, int);
 };
 
 struct smb_version_values {
index 61baaa3330fbd9309a5b11cbcf9a35dfca8c6b7f..4857965b22dbb3bcf64217c4e32e39ee1ab96f66 100644 (file)
@@ -226,17 +226,16 @@ extern int CIFSSMBUnixQPathInfo(const unsigned int xid,
                        const struct nls_table *nls_codepage, int remap);
 
 extern int CIFSGetDFSRefer(const unsigned int xid, struct cifs_ses *ses,
-                       const unsigned char *searchName,
-                       struct dfs_info3_param **target_nodes,
-                       unsigned int *number_of_nodes_in_array,
-                       const struct nls_table *nls_codepage, int remap);
+                          const char *search_name,
+                          struct dfs_info3_param **target_nodes,
+                          unsigned int *num_of_nodes,
+                          const struct nls_table *nls_codepage, int remap);
 
-extern int get_dfs_path(unsigned int xid, struct cifs_ses *pSesInfo,
+extern int get_dfs_path(const unsigned int xid, struct cifs_ses *ses,
                        const char *old_path,
                        const struct nls_table *nls_codepage,
-                       unsigned int *pnum_referrals,
-                       struct dfs_info3_param **preferrals,
-                       int remap);
+                       unsigned int *num_referrals,
+                       struct dfs_info3_param **referrals, int remap);
 extern void reset_cifs_unix_caps(unsigned int xid, struct cifs_tcon *tcon,
                                 struct cifs_sb_info *cifs_sb,
                                 struct smb_vol *vol);
index f1dfc7844f1be755784a2ed48ae342a09c7943cb..af859c325db1ee0632b9856289dfd4b1f0eb3d79 100644 (file)
@@ -4766,8 +4766,7 @@ parse_DFS_referrals_exit:
 
 int
 CIFSGetDFSRefer(const unsigned int xid, struct cifs_ses *ses,
-               const unsigned char *searchName,
-               struct dfs_info3_param **target_nodes,
+               const char *search_name, struct dfs_info3_param **target_nodes,
                unsigned int *num_of_nodes,
                const struct nls_table *nls_codepage, int remap)
 {
@@ -4781,7 +4780,7 @@ CIFSGetDFSRefer(const unsigned int xid, struct cifs_ses *ses,
        *num_of_nodes = 0;
        *target_nodes = NULL;
 
-       cFYI(1, "In GetDFSRefer the path %s", searchName);
+       cFYI(1, "In GetDFSRefer the path %s", search_name);
        if (ses == NULL)
                return -ENODEV;
 getDFSRetry:
@@ -4804,14 +4803,14 @@ getDFSRetry:
                pSMB->hdr.Flags2 |= SMBFLG2_UNICODE;
                name_len =
                    cifsConvertToUTF16((__le16 *) pSMB->RequestFileName,
-                                      searchName, PATH_MAX, nls_codepage,
+                                      search_name, PATH_MAX, nls_codepage,
                                       remap);
                name_len++;     /* trailing null */
                name_len *= 2;
        } else {        /* BB improve the check for buffer overruns BB */
-               name_len = strnlen(searchName, PATH_MAX);
+               name_len = strnlen(search_name, PATH_MAX);
                name_len++;     /* trailing null */
-               strncpy(pSMB->RequestFileName, searchName, name_len);
+               strncpy(pSMB->RequestFileName, search_name, name_len);
        }
 
        if (ses->server) {
@@ -4867,7 +4866,7 @@ getDFSRetry:
        /* parse returned result into more usable form */
        rc = parse_DFS_referrals(pSMBr, num_of_nodes,
                                 target_nodes, nls_codepage, remap,
-                                searchName);
+                                search_name);
 
 GetDFSRefExit:
        cifs_buf_release(pSMB);
index 7cf8b163224237da705997a544abd6e87b63e7e8..70a34d126001ab779411bbbf6def1609e9975e2d 100644 (file)
@@ -2767,14 +2767,14 @@ out:
 }
 
 int
-get_dfs_path(unsigned int xid, struct cifs_ses *ses, const char *old_path,
+get_dfs_path(const unsigned int xid, struct cifs_ses *ses, const char *old_path,
             const struct nls_table *nls_codepage, unsigned int *num_referrals,
             struct dfs_info3_param **referrals, int remap)
 {
        char *temp_unc;
        int rc = 0;
 
-       if (!ses->server->ops->tree_connect)
+       if (!ses->server->ops->tree_connect || !ses->server->ops->get_dfs_refer)
                return -ENOSYS;
 
        *num_referrals = 0;
@@ -2796,11 +2796,12 @@ get_dfs_path(unsigned int xid, struct cifs_ses *ses, const char *old_path,
                kfree(temp_unc);
        }
        if (rc == 0)
-               rc = CIFSGetDFSRefer(xid, ses, old_path, referrals,
-                                    num_referrals, nls_codepage, remap);
+               rc = ses->server->ops->get_dfs_refer(xid, ses, old_path,
+                                                    referrals, num_referrals,
+                                                    nls_codepage, remap);
        /*
         * BB - map targetUNCs to dfs_info3 structures, here or in
-        * CIFSGetDFSRefer.
+        * ses->server->ops->get_dfs_refer.
         */
 
        return rc;
@@ -3488,7 +3489,7 @@ build_unc_path_to_root(const struct smb_vol *vol,
  * determine whether there were referrals.
  */
 static int
-expand_dfs_referral(unsigned int xid, struct cifs_ses *pSesInfo,
+expand_dfs_referral(const unsigned int xid, struct cifs_ses *ses,
                    struct smb_vol *volume_info, struct cifs_sb_info *cifs_sb,
                    int check_prefix)
 {
@@ -3504,7 +3505,7 @@ expand_dfs_referral(unsigned int xid, struct cifs_ses *pSesInfo,
        /* For DFS paths, skip the first '\' of the UNC */
        ref_path = check_prefix ? full_path + 1 : volume_info->UNC + 1;
 
-       rc = get_dfs_path(xid, pSesInfo , ref_path, cifs_sb->local_nls,
+       rc = get_dfs_path(xid, ses, ref_path, cifs_sb->local_nls,
                          &num_referrals, &referrals,
                          cifs_sb->mnt_cifs_flags & CIFS_MOUNT_MAP_SPECIAL_CHR);
 
index 8f873863142a07a77662639d03e2954fc034e5d4..6d9025b29e54a78f485b18ae9b0784b76e277a08 100644 (file)
@@ -434,6 +434,7 @@ struct smb_version_operations smb1_operations = {
        .logoff = CIFSSMBLogoff,
        .tree_connect = CIFSTCon,
        .tree_disconnect = CIFSSMBTDis,
+       .get_dfs_refer = CIFSGetDFSRefer,
 };
 
 struct smb_version_values smb1_values = {