diff options
author | Steve French <stfrench@microsoft.com> | 2023-11-06 22:40:38 -0600 |
---|---|---|
committer | Steve French <stfrench@microsoft.com> | 2023-11-09 10:24:20 -0600 |
commit | 1bc081b67a79b6e75fae686e98048cea1038ae31 (patch) | |
tree | 8356a2131c2ba527ffe93beb510f7131dcf3646e /fs | |
parent | 43960dc2328e554c4c61b22c47e77e8b1c48d854 (diff) | |
download | linux-stable-1bc081b67a79b6e75fae686e98048cea1038ae31.tar.gz linux-stable-1bc081b67a79b6e75fae686e98048cea1038ae31.tar.bz2 linux-stable-1bc081b67a79b6e75fae686e98048cea1038ae31.zip |
smb3: more minor cleanups for session handling routines
Some trivial cleanup pointed out by checkpatch
Reviewed-by: Bharath SM <bharathsm@microsoft.com>
Signed-off-by: Steve French <stfrench@microsoft.com>
Diffstat (limited to 'fs')
-rw-r--r-- | fs/smb/client/sess.c | 25 |
1 files changed, 15 insertions, 10 deletions
diff --git a/fs/smb/client/sess.c b/fs/smb/client/sess.c index cd474cf98f30..829c7688a7b7 100644 --- a/fs/smb/client/sess.c +++ b/fs/smb/client/sess.c @@ -659,8 +659,7 @@ static void ascii_ssetup_strings(char **pbcc_area, struct cifs_ses *ses, if (WARN_ON_ONCE(len < 0)) len = CIFS_MAX_DOMAINNAME_LEN - 1; bcc_ptr += len; - } /* else we will send a null domain name - so the server will default to its own domain */ + } /* else we send a null domain name so server will default to its own domain */ *bcc_ptr = 0; bcc_ptr++; @@ -756,11 +755,14 @@ static void decode_ascii_ssetup(char **pbcc_area, __u16 bleft, if (len > bleft) return; - /* No domain field in LANMAN case. Domain is - returned by old servers in the SMB negprot response */ - /* BB For newer servers which do not support Unicode, - but thus do return domain here we could add parsing - for it later, but it is not very important */ + /* + * No domain field in LANMAN case. Domain is + * returned by old servers in the SMB negprot response + * + * BB For newer servers which do not support Unicode, + * but thus do return domain here, we could add parsing + * for it later, but it is not very important + */ cifs_dbg(FYI, "ascii: bytes left %d\n", bleft); } #endif /* CONFIG_CIFS_ALLOW_INSECURE_LEGACY */ @@ -816,9 +818,12 @@ int decode_ntlmssp_challenge(char *bcc_ptr, int blob_len, ses->ntlmssp->server_flags = server_flags; memcpy(ses->ntlmssp->cryptkey, pblob->Challenge, CIFS_CRYPTO_KEY_SIZE); - /* In particular we can examine sign flags */ - /* BB spec says that if AvId field of MsvAvTimestamp is populated then - we must set the MIC field of the AUTHENTICATE_MESSAGE */ + /* + * In particular we can examine sign flags + * + * BB spec says that if AvId field of MsvAvTimestamp is populated then + * we must set the MIC field of the AUTHENTICATE_MESSAGE + */ tioffset = le32_to_cpu(pblob->TargetInfoArray.BufferOffset); tilen = le16_to_cpu(pblob->TargetInfoArray.Length); |