[CIFS] Fix warnings from checkpatch
Signed-off-by: Steve French <sfrench@us.ibm.com>
This commit is contained in:
parent
b1910ad622
commit
ef571cadd5
|
@ -208,9 +208,9 @@ static int cifs_debug_data_proc_show(struct seq_file *m, void *v)
|
||||||
seq_puts(m, " type: CDROM ");
|
seq_puts(m, " type: CDROM ");
|
||||||
else
|
else
|
||||||
seq_printf(m, " type: %d ", dev_type);
|
seq_printf(m, " type: %d ", dev_type);
|
||||||
if (tcon->tidStatus == CifsNeedReconnect) {
|
|
||||||
|
if (tcon->tidStatus == CifsNeedReconnect)
|
||||||
seq_puts(m, "\tDISCONNECTED ");
|
seq_puts(m, "\tDISCONNECTED ");
|
||||||
}
|
|
||||||
}
|
}
|
||||||
read_unlock(&GlobalSMBSeslock);
|
read_unlock(&GlobalSMBSeslock);
|
||||||
|
|
||||||
|
|
|
@ -56,7 +56,7 @@ int match_sid(struct cifs_sid *ctsid)
|
||||||
struct cifs_sid *cwsid;
|
struct cifs_sid *cwsid;
|
||||||
|
|
||||||
if (!ctsid)
|
if (!ctsid)
|
||||||
return (-1);
|
return -1;
|
||||||
|
|
||||||
for (i = 0; i < NUM_WK_SIDS; ++i) {
|
for (i = 0; i < NUM_WK_SIDS; ++i) {
|
||||||
cwsid = &(wksidarr[i].cifssid);
|
cwsid = &(wksidarr[i].cifssid);
|
||||||
|
@ -87,11 +87,11 @@ int match_sid(struct cifs_sid *ctsid)
|
||||||
}
|
}
|
||||||
|
|
||||||
cFYI(1, ("matching sid: %s\n", wksidarr[i].sidname));
|
cFYI(1, ("matching sid: %s\n", wksidarr[i].sidname));
|
||||||
return (0); /* sids compare/match */
|
return 0; /* sids compare/match */
|
||||||
}
|
}
|
||||||
|
|
||||||
cFYI(1, ("No matching sid"));
|
cFYI(1, ("No matching sid"));
|
||||||
return (-1);
|
return -1;
|
||||||
}
|
}
|
||||||
|
|
||||||
/* if the two SIDs (roughly equivalent to a UUID for a user or group) are
|
/* if the two SIDs (roughly equivalent to a UUID for a user or group) are
|
||||||
|
@ -102,16 +102,16 @@ int compare_sids(const struct cifs_sid *ctsid, const struct cifs_sid *cwsid)
|
||||||
int num_subauth, num_sat, num_saw;
|
int num_subauth, num_sat, num_saw;
|
||||||
|
|
||||||
if ((!ctsid) || (!cwsid))
|
if ((!ctsid) || (!cwsid))
|
||||||
return (0);
|
return 0;
|
||||||
|
|
||||||
/* compare the revision */
|
/* compare the revision */
|
||||||
if (ctsid->revision != cwsid->revision)
|
if (ctsid->revision != cwsid->revision)
|
||||||
return (0);
|
return 0;
|
||||||
|
|
||||||
/* compare all of the six auth values */
|
/* compare all of the six auth values */
|
||||||
for (i = 0; i < 6; ++i) {
|
for (i = 0; i < 6; ++i) {
|
||||||
if (ctsid->authority[i] != cwsid->authority[i])
|
if (ctsid->authority[i] != cwsid->authority[i])
|
||||||
return (0);
|
return 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
/* compare all of the subauth values if any */
|
/* compare all of the subauth values if any */
|
||||||
|
@ -121,11 +121,11 @@ int compare_sids(const struct cifs_sid *ctsid, const struct cifs_sid *cwsid)
|
||||||
if (num_subauth) {
|
if (num_subauth) {
|
||||||
for (i = 0; i < num_subauth; ++i) {
|
for (i = 0; i < num_subauth; ++i) {
|
||||||
if (ctsid->sub_auth[i] != cwsid->sub_auth[i])
|
if (ctsid->sub_auth[i] != cwsid->sub_auth[i])
|
||||||
return (0);
|
return 0;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
return (1); /* sids compare/match */
|
return 1; /* sids compare/match */
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
|
@ -284,7 +284,7 @@ static __u16 fill_ace_for_sid(struct cifs_ace *pntace,
|
||||||
size = 1 + 1 + 2 + 4 + 1 + 1 + 6 + (psid->num_subauth * 4);
|
size = 1 + 1 + 2 + 4 + 1 + 1 + 6 + (psid->num_subauth * 4);
|
||||||
pntace->size = cpu_to_le16(size);
|
pntace->size = cpu_to_le16(size);
|
||||||
|
|
||||||
return (size);
|
return size;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
|
@ -425,7 +425,7 @@ static int set_chmod_dacl(struct cifs_acl *pndacl, struct cifs_sid *pownersid,
|
||||||
pndacl->size = cpu_to_le16(size + sizeof(struct cifs_acl));
|
pndacl->size = cpu_to_le16(size + sizeof(struct cifs_acl));
|
||||||
pndacl->num_aces = cpu_to_le32(3);
|
pndacl->num_aces = cpu_to_le32(3);
|
||||||
|
|
||||||
return (0);
|
return 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
|
@ -509,7 +509,7 @@ static int parse_sec_desc(struct cifs_ntsd *pntsd, int acl_len,
|
||||||
sizeof(struct cifs_sid)); */
|
sizeof(struct cifs_sid)); */
|
||||||
|
|
||||||
|
|
||||||
return (0);
|
return 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
|
@ -526,7 +526,7 @@ static int build_sec_desc(struct cifs_ntsd *pntsd, struct cifs_ntsd *pnntsd,
|
||||||
struct cifs_acl *ndacl_ptr = NULL; /* no need for SACL ptr */
|
struct cifs_acl *ndacl_ptr = NULL; /* no need for SACL ptr */
|
||||||
|
|
||||||
if ((inode == NULL) || (pntsd == NULL) || (pnntsd == NULL))
|
if ((inode == NULL) || (pntsd == NULL) || (pnntsd == NULL))
|
||||||
return (-EIO);
|
return -EIO;
|
||||||
|
|
||||||
owner_sid_ptr = (struct cifs_sid *)((char *)pntsd +
|
owner_sid_ptr = (struct cifs_sid *)((char *)pntsd +
|
||||||
le32_to_cpu(pntsd->osidoffset));
|
le32_to_cpu(pntsd->osidoffset));
|
||||||
|
@ -549,7 +549,7 @@ static int build_sec_desc(struct cifs_ntsd *pntsd, struct cifs_ntsd *pnntsd,
|
||||||
/* copy security descriptor control portion and owner and group sid */
|
/* copy security descriptor control portion and owner and group sid */
|
||||||
copy_sec_desc(pntsd, pnntsd, sidsoffset);
|
copy_sec_desc(pntsd, pnntsd, sidsoffset);
|
||||||
|
|
||||||
return (rc);
|
return rc;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
|
@ -628,11 +628,11 @@ static int set_cifs_acl(struct cifs_ntsd *pnntsd, __u32 acllen,
|
||||||
cFYI(DBG2, ("set ACL for %s from mode 0x%x", path, inode->i_mode));
|
cFYI(DBG2, ("set ACL for %s from mode 0x%x", path, inode->i_mode));
|
||||||
|
|
||||||
if (!inode)
|
if (!inode)
|
||||||
return (rc);
|
return rc;
|
||||||
|
|
||||||
sb = inode->i_sb;
|
sb = inode->i_sb;
|
||||||
if (sb == NULL)
|
if (sb == NULL)
|
||||||
return (rc);
|
return rc;
|
||||||
|
|
||||||
cifs_sb = CIFS_SB(sb);
|
cifs_sb = CIFS_SB(sb);
|
||||||
xid = GetXid();
|
xid = GetXid();
|
||||||
|
@ -651,7 +651,7 @@ static int set_cifs_acl(struct cifs_ntsd *pnntsd, __u32 acllen,
|
||||||
if (rc != 0) {
|
if (rc != 0) {
|
||||||
cERROR(1, ("Unable to open file to set ACL"));
|
cERROR(1, ("Unable to open file to set ACL"));
|
||||||
FreeXid(xid);
|
FreeXid(xid);
|
||||||
return (rc);
|
return rc;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -664,7 +664,7 @@ static int set_cifs_acl(struct cifs_ntsd *pnntsd, __u32 acllen,
|
||||||
|
|
||||||
FreeXid(xid);
|
FreeXid(xid);
|
||||||
|
|
||||||
return (rc);
|
return rc;
|
||||||
}
|
}
|
||||||
|
|
||||||
/* Translate the CIFS ACL (simlar to NTFS ACL) for a file into mode bits */
|
/* Translate the CIFS ACL (simlar to NTFS ACL) for a file into mode bits */
|
||||||
|
@ -714,7 +714,7 @@ int mode_to_acl(struct inode *inode, const char *path, __u64 nmode)
|
||||||
if (!pnntsd) {
|
if (!pnntsd) {
|
||||||
cERROR(1, ("Unable to allocate security descriptor"));
|
cERROR(1, ("Unable to allocate security descriptor"));
|
||||||
kfree(pntsd);
|
kfree(pntsd);
|
||||||
return (-ENOMEM);
|
return -ENOMEM;
|
||||||
}
|
}
|
||||||
|
|
||||||
rc = build_sec_desc(pntsd, pnntsd, inode, nmode);
|
rc = build_sec_desc(pntsd, pnntsd, inode, nmode);
|
||||||
|
@ -731,6 +731,6 @@ int mode_to_acl(struct inode *inode, const char *path, __u64 nmode)
|
||||||
kfree(pntsd);
|
kfree(pntsd);
|
||||||
}
|
}
|
||||||
|
|
||||||
return (rc);
|
return rc;
|
||||||
}
|
}
|
||||||
#endif /* CONFIG_CIFS_EXPERIMENTAL */
|
#endif /* CONFIG_CIFS_EXPERIMENTAL */
|
||||||
|
|
|
@ -310,9 +310,8 @@ void calc_lanman_hash(struct cifsSesInfo *ses, char *lnm_session_key)
|
||||||
utf8 and other multibyte codepages each need their own strupper
|
utf8 and other multibyte codepages each need their own strupper
|
||||||
function since a byte at a time will ont work. */
|
function since a byte at a time will ont work. */
|
||||||
|
|
||||||
for (i = 0; i < CIFS_ENCPWD_SIZE; i++) {
|
for (i = 0; i < CIFS_ENCPWD_SIZE; i++)
|
||||||
password_with_pad[i] = toupper(password_with_pad[i]);
|
password_with_pad[i] = toupper(password_with_pad[i]);
|
||||||
}
|
|
||||||
|
|
||||||
SMBencrypt(password_with_pad, ses->server->cryptKey, lnm_session_key);
|
SMBencrypt(password_with_pad, ses->server->cryptKey, lnm_session_key);
|
||||||
/* clear password before we return/free memory */
|
/* clear password before we return/free memory */
|
||||||
|
|
|
@ -27,7 +27,7 @@
|
||||||
#define MAX_SES_INFO 2
|
#define MAX_SES_INFO 2
|
||||||
#define MAX_TCON_INFO 4
|
#define MAX_TCON_INFO 4
|
||||||
|
|
||||||
#define MAX_TREE_SIZE 2 + MAX_SERVER_SIZE + 1 + MAX_SHARE_SIZE + 1
|
#define MAX_TREE_SIZE (2 + MAX_SERVER_SIZE + 1 + MAX_SHARE_SIZE + 1)
|
||||||
#define MAX_SERVER_SIZE 15
|
#define MAX_SERVER_SIZE 15
|
||||||
#define MAX_SHARE_SIZE 64 /* used to be 20, this should still be enough */
|
#define MAX_SHARE_SIZE 64 /* used to be 20, this should still be enough */
|
||||||
#define MAX_USERNAME_SIZE 32 /* 32 is to allow for 15 char names + null
|
#define MAX_USERNAME_SIZE 32 /* 32 is to allow for 15 char names + null
|
||||||
|
@ -537,8 +537,8 @@ require use of the stronger protocol */
|
||||||
#endif /* WEAK_PW_HASH */
|
#endif /* WEAK_PW_HASH */
|
||||||
#define CIFSSEC_MUST_SEAL 0x40040 /* not supported yet */
|
#define CIFSSEC_MUST_SEAL 0x40040 /* not supported yet */
|
||||||
|
|
||||||
#define CIFSSEC_DEF CIFSSEC_MAY_SIGN | CIFSSEC_MAY_NTLM | CIFSSEC_MAY_NTLMV2
|
#define CIFSSEC_DEF (CIFSSEC_MAY_SIGN | CIFSSEC_MAY_NTLM | CIFSSEC_MAY_NTLMV2)
|
||||||
#define CIFSSEC_MAX CIFSSEC_MUST_SIGN | CIFSSEC_MUST_NTLMV2
|
#define CIFSSEC_MAX (CIFSSEC_MUST_SIGN | CIFSSEC_MUST_NTLMV2)
|
||||||
#define CIFSSEC_AUTH_MASK (CIFSSEC_MAY_NTLM | CIFSSEC_MAY_NTLMV2 | CIFSSEC_MAY_LANMAN | CIFSSEC_MAY_PLNTXT | CIFSSEC_MAY_KRB5)
|
#define CIFSSEC_AUTH_MASK (CIFSSEC_MAY_NTLM | CIFSSEC_MAY_NTLMV2 | CIFSSEC_MAY_LANMAN | CIFSSEC_MAY_PLNTXT | CIFSSEC_MAY_KRB5)
|
||||||
/*
|
/*
|
||||||
*****************************************************************
|
*****************************************************************
|
||||||
|
|
|
@ -31,7 +31,7 @@
|
||||||
#else
|
#else
|
||||||
#define CIFS_PROT 0
|
#define CIFS_PROT 0
|
||||||
#endif
|
#endif
|
||||||
#define POSIX_PROT CIFS_PROT+1
|
#define POSIX_PROT (CIFS_PROT+1)
|
||||||
#define BAD_PROT 0xFFFF
|
#define BAD_PROT 0xFFFF
|
||||||
|
|
||||||
/* SMB command codes */
|
/* SMB command codes */
|
||||||
|
@ -341,7 +341,7 @@
|
||||||
#define CREATE_COMPLETE_IF_OPLK 0x00000100 /* should be zero */
|
#define CREATE_COMPLETE_IF_OPLK 0x00000100 /* should be zero */
|
||||||
#define CREATE_NO_EA_KNOWLEDGE 0x00000200
|
#define CREATE_NO_EA_KNOWLEDGE 0x00000200
|
||||||
#define CREATE_EIGHT_DOT_THREE 0x00000400 /* doc says this is obsolete
|
#define CREATE_EIGHT_DOT_THREE 0x00000400 /* doc says this is obsolete
|
||||||
"open for recovery" flag - should
|
"open for recovery" flag should
|
||||||
be zero in any case */
|
be zero in any case */
|
||||||
#define CREATE_OPEN_FOR_RECOVERY 0x00000400
|
#define CREATE_OPEN_FOR_RECOVERY 0x00000400
|
||||||
#define CREATE_RANDOM_ACCESS 0x00000800
|
#define CREATE_RANDOM_ACCESS 0x00000800
|
||||||
|
@ -414,8 +414,8 @@ struct smb_hdr {
|
||||||
__u8 WordCount;
|
__u8 WordCount;
|
||||||
} __attribute__((packed));
|
} __attribute__((packed));
|
||||||
/* given a pointer to an smb_hdr retrieve the value of byte count */
|
/* given a pointer to an smb_hdr retrieve the value of byte count */
|
||||||
#define BCC(smb_var) ( *(__u16 *)((char *)smb_var + sizeof(struct smb_hdr) + (2 * smb_var->WordCount)))
|
#define BCC(smb_var) (*(__u16 *)((char *)smb_var + sizeof(struct smb_hdr) + (2 * smb_var->WordCount)))
|
||||||
#define BCC_LE(smb_var) ( *(__le16 *)((char *)smb_var + sizeof(struct smb_hdr) + (2 * smb_var->WordCount)))
|
#define BCC_LE(smb_var) (*(__le16 *)((char *)smb_var + sizeof(struct smb_hdr) + (2 * smb_var->WordCount)))
|
||||||
/* given a pointer to an smb_hdr retrieve the pointer to the byte area */
|
/* given a pointer to an smb_hdr retrieve the pointer to the byte area */
|
||||||
#define pByteArea(smb_var) ((unsigned char *)smb_var + sizeof(struct smb_hdr) + (2 * smb_var->WordCount) + 2)
|
#define pByteArea(smb_var) ((unsigned char *)smb_var + sizeof(struct smb_hdr) + (2 * smb_var->WordCount) + 2)
|
||||||
|
|
||||||
|
|
|
@ -686,11 +686,10 @@ CIFSSMBNegotiate(unsigned int xid, struct cifsSesInfo *ses)
|
||||||
SecurityBlob,
|
SecurityBlob,
|
||||||
count - 16,
|
count - 16,
|
||||||
&server->secType);
|
&server->secType);
|
||||||
if (rc == 1) {
|
if (rc == 1)
|
||||||
rc = 0;
|
rc = 0;
|
||||||
} else {
|
else
|
||||||
rc = -EINVAL;
|
rc = -EINVAL;
|
||||||
}
|
|
||||||
}
|
}
|
||||||
} else
|
} else
|
||||||
server->capabilities &= ~CAP_EXTENDED_SECURITY;
|
server->capabilities &= ~CAP_EXTENDED_SECURITY;
|
||||||
|
|
Loading…
Reference in New Issue