[CIFS] add constants for string lengths of keynames in SPNEGO upcall string

Signed-off-by: Jeff Layton <jlayton@redhat.com>
Signed-off-by: Steve French <sfrench@us.ibm.com>

diff --git a/fs/cifs/cifs_spnego.c b/fs/cifs/cifs_spnego.c
index 117ef4b..fcee929 100644 (file)
--- a/fs/cifs/cifs_spnego.c
+++ b/fs/cifs/cifs_spnego.c
@@ -66,11 +66,28 @@ struct key_type cifs_spnego_key_type = {
        .describe       = user_describe,
 };
 
-#define MAX_VER_STR_LEN   8 /* length of longest version string e.g.
-                               strlen("ver=0xFF") */
-#define MAX_MECH_STR_LEN 13 /* length of longest security mechanism name, eg
-                              in future could have strlen(";sec=ntlmsspi") */
-#define MAX_IPV6_ADDR_LEN 42 /* eg FEDC:BA98:7654:3210:FEDC:BA98:7654:3210/60 */
+/* length of longest version string e.g.  strlen("ver=0xFF") */
+#define MAX_VER_STR_LEN                8
+
+/* length of longest security mechanism name, eg in future could have
+ * strlen(";sec=ntlmsspi") */
+#define MAX_MECH_STR_LEN       13
+
+/* max possible addr len eg FEDC:BA98:7654:3210:FEDC:BA98:7654:3210/60 */
+#define MAX_IPV6_ADDR_LEN      42
+
+/* strlen of "host=" */
+#define HOST_KEY_LEN           5
+
+/* strlen of ";ip4=" or ";ip6=" */
+#define IP_KEY_LEN             5
+
+/* strlen of ";uid=0x" */
+#define UID_KEY_LEN            7
+
+/* strlen of ";user=" */
+#define USER_KEY_LEN           6
+
 /* get a key struct with a SPNEGO security blob, suitable for session setup */
 struct key *
 cifs_get_spnego_key(struct cifsSesInfo *sesInfo)
@@ -84,11 +101,11 @@ cifs_get_spnego_key(struct cifsSesInfo *sesInfo)
        /* length of fields (with semicolons): ver=0xyz ip4=ipaddress
           host=hostname sec=mechanism uid=0xFF user=username */
        desc_len = MAX_VER_STR_LEN +
-                  6 /* len of "host=" */ + strlen(hostname) +
-                  5 /* len of ";ipv4=" */ + MAX_IPV6_ADDR_LEN +
+                  HOST_KEY_LEN + strlen(hostname) +
+                  IP_KEY_LEN + MAX_IPV6_ADDR_LEN +
                   MAX_MECH_STR_LEN +
-                  7 /* len of ";uid=0x" */ + (sizeof(uid_t) * 2) +
-                  6 /* len of ";user=" */ + strlen(sesInfo->userName) + 1;
+                  UID_KEY_LEN + (sizeof(uid_t) * 2) +
+                  USER_KEY_LEN + strlen(sesInfo->userName) + 1;
 
        spnego_key = ERR_PTR(-ENOMEM);
        description = kzalloc(desc_len, GFP_KERNEL);