CIFS: Move protocol specific negotiate code to ops struct

Reviewed-by: Jeff Layton <jlayton@redhat.com>
Signed-off-by: Pavel Shilovsky <pshilovsky@samba.org>
Signed-off-by: Steve French <smfrench@gmail.com>

diff --git a/fs/cifs/cifsglob.h b/fs/cifs/cifsglob.h
index 844b77c2bc9c7ed0b82b82fb185f6db36bb98d2b..8a4150573cf819bd2c4e153d25d3a11f04cb030a 100644 (file)
--- a/fs/cifs/cifsglob.h
+++ b/fs/cifs/cifsglob.h
@@ -192,6 +192,10 @@ struct smb_version_operations {
        /* process transaction2 response */
        bool (*check_trans2)(struct mid_q_entry *, struct TCP_Server_Info *,
                             char *, int);
+       /* check if we need to negotiate */
+       bool (*need_neg)(struct TCP_Server_Info *);
+       /* negotiate to the server */
+       int (*negotiate)(const unsigned int, struct cifs_ses *);
 };
 
 struct smb_version_values {
@@ -324,7 +328,7 @@ struct TCP_Server_Info {
        struct mutex srv_mutex;
        struct task_struct *tsk;
        char server_GUID[16];
-       char sec_mode;
+       __u16 sec_mode;
        bool session_estab; /* mark when very first sess is established */
        u16 dialect; /* dialect index that server chose */
        enum securityEnum secType;
@@ -459,7 +463,7 @@ struct cifs_ses {
        char *serverOS;         /* name of operating system underlying server */
        char *serverNOS;        /* name of network operating system of server */
        char *serverDomain;     /* security realm of server */
-       int Suid;               /* remote smb uid  */
+       __u64 Suid;             /* remote smb uid  */
        uid_t linux_uid;        /* overriding owner of files on the mount */
        uid_t cred_uid;         /* owner of credentials */
        int capabilities;

diff --git a/fs/cifs/cifsproto.h b/fs/cifs/cifsproto.h
index b37399491fa3f02e23e1a892fdfe1f21c98d7ab3..723a3273c6bb92875fbe04e421eaf8c539046b87 100644 (file)
--- a/fs/cifs/cifsproto.h
+++ b/fs/cifs/cifsproto.h
@@ -178,11 +178,11 @@ extern void cifs_dfs_release_automount_timer(void);
 void cifs_proc_init(void);
 void cifs_proc_clean(void);
 
-extern int cifs_negotiate_protocol(unsigned int xid,
-                                 struct cifs_ses *ses);
+extern int cifs_negotiate_protocol(const unsigned int xid,
+                                  struct cifs_ses *ses);
 extern int cifs_setup_session(unsigned int xid, struct cifs_ses *ses,
                        struct nls_table *nls_info);
-extern int CIFSSMBNegotiate(unsigned int xid, struct cifs_ses *ses);
+extern int CIFSSMBNegotiate(const unsigned int xid, struct cifs_ses *ses);
 
 extern int CIFSTCon(unsigned int xid, struct cifs_ses *ses,
                        const char *tree, struct cifs_tcon *tcon,

diff --git a/fs/cifs/cifssmb.c b/fs/cifs/cifssmb.c
index 92bbd8487ead272fe4184b6da5f82da8e791ffa0..ae59d6e4e4f5cb417d2b2f29a887f4962615aa8b 100644 (file)
--- a/fs/cifs/cifssmb.c
+++ b/fs/cifs/cifssmb.c
@@ -396,7 +396,7 @@ static inline void inc_rfc1001_len(void *pSMB, int count)
 }
 
 int
-CIFSSMBNegotiate(unsigned int xid, struct cifs_ses *ses)
+CIFSSMBNegotiate(const unsigned int xid, struct cifs_ses *ses)
 {
        NEGOTIATE_REQ *pSMB;
        NEGOTIATE_RSP *pSMBr;
@@ -480,7 +480,7 @@ CIFSSMBNegotiate(unsigned int xid, struct cifs_ses *ses)
                        rc = -EOPNOTSUPP;
                        goto neg_err_exit;
                }
-               server->sec_mode = (__u8)le16_to_cpu(rsp->SecurityMode);
+               server->sec_mode = le16_to_cpu(rsp->SecurityMode);
                server->maxReq = min_t(unsigned int,
                                       le16_to_cpu(rsp->MaxMpxCount),
                                       cifs_max_pending);

diff --git a/fs/cifs/connect.c b/fs/cifs/connect.c
index 6d846e7624d03166f1c77dff9cbc14e42b414322..03389f59390fbd815ece51bb3c399ae26d1f8829 100644 (file)
--- a/fs/cifs/connect.c
+++ b/fs/cifs/connect.c
@@ -407,7 +407,7 @@ cifs_echo_request(struct work_struct *work)
         * done, which is indicated by maxBuf != 0. Also, no need to ping if
         * we got a response recently
         */
-       if (server->maxBuf == 0 ||
+       if (!server->ops->need_neg || server->ops->need_neg(server) ||
            time_before(jiffies, server->lstrp + SMB_ECHO_INTERVAL - HZ))
                goto requeue_echo;
 
@@ -2406,7 +2406,8 @@ static bool warned_on_ntlm;  /* globals init to false automatically */
 static struct cifs_ses *
 cifs_get_smb_ses(struct TCP_Server_Info *server, struct smb_vol *volume_info)
 {
-       int rc = -ENOMEM, xid;
+       int rc = -ENOMEM;
+       unsigned int xid;
        struct cifs_ses *ses;
        struct sockaddr_in *addr = (struct sockaddr_in *)&server->dstaddr;
        struct sockaddr_in6 *addr6 = (struct sockaddr_in6 *)&server->dstaddr;
@@ -3960,24 +3961,22 @@ cifs_umount(struct cifs_sb_info *cifs_sb)
        kfree(cifs_sb);
 }
 
-int cifs_negotiate_protocol(unsigned int xid, struct cifs_ses *ses)
+int
+cifs_negotiate_protocol(const unsigned int xid, struct cifs_ses *ses)
 {
        int rc = 0;
        struct TCP_Server_Info *server = ses->server;
 
+       if (!server->ops->need_neg || !server->ops->negotiate)
+               return -ENOSYS;
+
        /* only send once per connect */
-       if (server->maxBuf != 0)
+       if (!server->ops->need_neg(server))
                return 0;
 
        set_credits(server, 1);
-       rc = CIFSSMBNegotiate(xid, ses);
-       if (rc == -EAGAIN) {
-               /* retry only once on 1st time connection */
-               set_credits(server, 1);
-               rc = CIFSSMBNegotiate(xid, ses);
-               if (rc == -EAGAIN)
-                       rc = -EHOSTDOWN;
-       }
+
+       rc = server->ops->negotiate(xid, ses);
        if (rc == 0) {
                spin_lock(&GlobalMid_Lock);
                if (server->tcpStatus == CifsNeedNegotiate)
@@ -3985,7 +3984,6 @@ int cifs_negotiate_protocol(unsigned int xid, struct cifs_ses *ses)
                else
                        rc = -EHOSTDOWN;
                spin_unlock(&GlobalMid_Lock);
-
        }
 
        return rc;

diff --git a/fs/cifs/sess.c b/fs/cifs/sess.c
index b4219789049a93c6f39c36418fca67e5dc9cdc56..3ba3f3cd2397f260c04459c88e1a507c3311121b 100644 (file)
--- a/fs/cifs/sess.c
+++ b/fs/cifs/sess.c
@@ -898,7 +898,7 @@ ssetup_ntlmssp_authenticate:
        if (action & GUEST_LOGIN)
                cFYI(1, "Guest login"); /* BB mark SesInfo struct? */
        ses->Suid = smb_buf->Uid;   /* UID left in wire format (le) */
-       cFYI(1, "UID = %d ", ses->Suid);
+       cFYI(1, "UID = %llu ", ses->Suid);
        /* response can have either 3 or 4 word count - Samba sends 3 */
        /* and lanman response is 3 */
        bytes_remaining = get_bcc(smb_buf);

diff --git a/fs/cifs/smb1ops.c b/fs/cifs/smb1ops.c
index f4f839459e90ae56fc74925c0a57e70a183ee908..ea4fb8aaaafb2d7da71746f49efd506fb5b65ae6 100644 (file)
--- a/fs/cifs/smb1ops.c
+++ b/fs/cifs/smb1ops.c
@@ -389,6 +389,27 @@ cifs_check_trans2(struct mid_q_entry *mid, struct TCP_Server_Info *server,
        return true;
 }
 
+static bool
+cifs_need_neg(struct TCP_Server_Info *server)
+{
+       return server->maxBuf == 0;
+}
+
+static int
+cifs_negotiate(const unsigned int xid, struct cifs_ses *ses)
+{
+       int rc;
+       rc = CIFSSMBNegotiate(xid, ses);
+       if (rc == -EAGAIN) {
+               /* retry only once on 1st time connection */
+               set_credits(ses->server, 1);
+               rc = CIFSSMBNegotiate(xid, ses);
+               if (rc == -EAGAIN)
+                       rc = -EHOSTDOWN;
+       }
+       return rc;
+}
+
 struct smb_version_operations smb1_operations = {
        .send_cancel = send_nt_cancel,
        .compare_fids = cifs_compare_fids,
@@ -407,6 +428,8 @@ struct smb_version_operations smb1_operations = {
        .dump_detail = cifs_dump_detail,
        .is_oplock_break = is_valid_oplock_break,
        .check_trans2 = cifs_check_trans2,
+       .need_neg = cifs_need_neg,
+       .negotiate = cifs_negotiate,
 };
 
 struct smb_version_values smb1_values = {