diff options
| author | Jouni Malinen <j@w1.fi> | 2011-12-04 16:59:16 +0200 |
|---|---|---|
| committer | Jouni Malinen <j@w1.fi> | 2011-12-04 16:59:16 +0200 |
| commit | 607bcf61a4d8e202fdf8e2aefdd7f72eca12f2a5 (patch) | |
| tree | 8a65083fe87f27cc1003227a8fd186965180eeba /src/eap_server/eap_server_mschapv2.c | |
| parent | d627a9395d78eb41f87f9b8df6ed629f606157ba (diff) | |
| download | external_wpa_supplicant_8_ti-607bcf61a4d8e202fdf8e2aefdd7f72eca12f2a5.zip external_wpa_supplicant_8_ti-607bcf61a4d8e202fdf8e2aefdd7f72eca12f2a5.tar.gz external_wpa_supplicant_8_ti-607bcf61a4d8e202fdf8e2aefdd7f72eca12f2a5.tar.bz2 | |
Check nt_password_hash() return code
While this is unlikely to fail in practice, better be more consistent
in validating nt_password_hash() result.
Signed-hostap: Jouni Malinen <j@w1.fi>
Diffstat (limited to 'src/eap_server/eap_server_mschapv2.c')
| -rw-r--r-- | src/eap_server/eap_server_mschapv2.c | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/src/eap_server/eap_server_mschapv2.c b/src/eap_server/eap_server_mschapv2.c index 64120a4..f5ee7f4 100644 --- a/src/eap_server/eap_server_mschapv2.c +++ b/src/eap_server/eap_server_mschapv2.c @@ -405,9 +405,12 @@ static void eap_mschapv2_process_response(struct eap_sm *sm, if (sm->user->password_hash) { pw_hash = sm->user->password; } else { - nt_password_hash(sm->user->password, - sm->user->password_len, - pw_hash_buf); + if (nt_password_hash(sm->user->password, + sm->user->password_len, + pw_hash_buf) < 0) { + data->state = FAILURE; + return; + } pw_hash = pw_hash_buf; } generate_authenticator_response_pwhash( |