aboutsummaryrefslogtreecommitdiffstats
path: root/fs/cifs
diff options
context:
space:
mode:
authorShirish Pargaonkar <shirishpargaonkar@gmail.com>2010-10-19 11:47:52 -0500
committerSteve French <sfrench@us.ibm.com>2010-10-19 18:56:42 +0000
commit89f150f401c32b0a587dcb98d3bcfafe0b9c1c70 (patch)
treecff93486560754e3c4652bf78ddd120daee5ba8c /fs/cifs
parentb33879aa834ebe03ced3dca4e3b822bd8894a474 (diff)
downloadkernel_samsung_aries-89f150f401c32b0a587dcb98d3bcfafe0b9c1c70.zip
kernel_samsung_aries-89f150f401c32b0a587dcb98d3bcfafe0b9c1c70.tar.gz
kernel_samsung_aries-89f150f401c32b0a587dcb98d3bcfafe0b9c1c70.tar.bz2
Clean up two declarations of blob_len
- Eliminate double declaration of variable blob_len - Modify function build_ntlmssp_auth_blob to return error code as well as length of the blob. Signed-off-by: Shirish Pargaonkar <shirishpargaonkar@gmail.com> Reviewed-by: Jeff Layton <jlayton@samba.org> Signed-off-by: Steve French <sfrench@us.ibm.com>
Diffstat (limited to 'fs/cifs')
-rw-r--r--fs/cifs/sess.c13
1 files changed, 8 insertions, 5 deletions
diff --git a/fs/cifs/sess.c b/fs/cifs/sess.c
index 2111bed..e35dc60 100644
--- a/fs/cifs/sess.c
+++ b/fs/cifs/sess.c
@@ -464,6 +464,7 @@ static void build_ntlmssp_negotiate_blob(unsigned char *pbuffer,
maximum possible size is fixed and small, making this approach cleaner.
This function returns the length of the data in the blob */
static int build_ntlmssp_auth_blob(unsigned char *pbuffer,
+ u16 *buflen,
struct cifsSesInfo *ses,
const struct nls_table *nls_cp)
{
@@ -558,7 +559,8 @@ static int build_ntlmssp_auth_blob(unsigned char *pbuffer,
sec_blob->SessionKey.MaximumLength = 0;
setup_ntlmv2_ret:
- return tmp - pbuffer;
+ *buflen = tmp - pbuffer;
+ return rc;
}
@@ -591,7 +593,7 @@ CIFS_SessSetup(unsigned int xid, struct cifsSesInfo *ses,
int bytes_remaining;
struct key *spnego_key = NULL;
__le32 phase = NtLmNegotiate; /* NTLMSSP, if needed, is multistage */
- int blob_len;
+ u16 blob_len;
char *ntlmsspblob = NULL;
if (ses == NULL)
@@ -848,8 +850,10 @@ ssetup_ntlmssp_authenticate:
goto ssetup_exit;
}
- blob_len = build_ntlmssp_auth_blob(ntlmsspblob,
- ses, nls_cp);
+ rc = build_ntlmssp_auth_blob(ntlmsspblob,
+ &blob_len, ses, nls_cp);
+ if (rc)
+ goto ssetup_exit;
iov[1].iov_len = blob_len;
iov[1].iov_base = ntlmsspblob;
pSMB->req.SecurityBlobLength =
@@ -927,7 +931,6 @@ ssetup_ntlmssp_authenticate:
bcc_ptr = pByteArea(smb_buf);
if (smb_buf->WordCount == 4) {
- __u16 blob_len;
blob_len = le16_to_cpu(pSMB->resp.SecurityBlobLength);
if (blob_len > bytes_remaining) {
cERROR(1, "bad security blob length %d", blob_len);