diff options
author | Luis de Bethencourt <luisbg@osg.samsung.com> | 2016-06-08 12:02:32 -0400 |
---|---|---|
committer | Steve French <smfrench@gmail.com> | 2016-06-24 00:45:17 -0400 |
commit | a6b6befbb2806697461962edb044e3376a771ebb (patch) | |
tree | 6881983b05ca1f23ba81788d7d4c829eaebbf839 /fs/cifs | |
parent | b8da344b74c822e966c6d19d6b2321efe82c5d97 (diff) |
cifs: check hash calculating succeeded
calc_lanman_hash() could return -ENOMEM or other errors, we should check
that everything went fine before using the calculated key.
Signed-off-by: Luis de Bethencourt <luisbg@osg.samsung.com>
Signed-off-by: Steve French <smfrench@gmail.com>
Diffstat (limited to 'fs/cifs')
-rw-r--r-- | fs/cifs/sess.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/fs/cifs/sess.c b/fs/cifs/sess.c index a42e99c8e00e..538d9b55699a 100644 --- a/fs/cifs/sess.c +++ b/fs/cifs/sess.c | |||
@@ -710,6 +710,8 @@ sess_auth_lanman(struct sess_data *sess_data) | |||
710 | rc = calc_lanman_hash(ses->password, ses->server->cryptkey, | 710 | rc = calc_lanman_hash(ses->password, ses->server->cryptkey, |
711 | ses->server->sec_mode & SECMODE_PW_ENCRYPT ? | 711 | ses->server->sec_mode & SECMODE_PW_ENCRYPT ? |
712 | true : false, lnm_session_key); | 712 | true : false, lnm_session_key); |
713 | if (rc) | ||
714 | goto out; | ||
713 | 715 | ||
714 | memcpy(bcc_ptr, (char *)lnm_session_key, CIFS_AUTH_RESP_SIZE); | 716 | memcpy(bcc_ptr, (char *)lnm_session_key, CIFS_AUTH_RESP_SIZE); |
715 | bcc_ptr += CIFS_AUTH_RESP_SIZE; | 717 | bcc_ptr += CIFS_AUTH_RESP_SIZE; |