Amazing-Mirror/fragattacks
1
0
Fork 0
mirror of https://github.com/vanhoefm/fragattacks.git synced 2024-11-25 08:48:31 -05:00
Code Issues Projects Releases Wiki Activity

EAP-pwd client: Verify received scalar and element

Browse Source 
When processing an EAP-pwd Commit frame, the server's scalar and element
(elliptic curve point) were not validated. This allowed an adversary to
bypass authentication, and act as a rogue Access Point (AP) if the
crypto implementation did not verify the validity of the EC point.

Fix this vulnerability by assuring the received scalar lies within the
valid range, and by checking that the received element is not the point
at infinity and lies on the elliptic curve being used. (CVE-2019-9499)

The vulnerability is only exploitable if OpenSSL version 1.0.2 or lower
is used, or if LibreSSL or wolfssl is used. Newer versions of OpenSSL
(and also BoringSSL) implicitly validate the elliptic curve point in
EC_POINT_set_affine_coordinates_GFp(), preventing the attack.

Signed-off-by: Mathy Vanhoef <mathy.vanhoef@nyu.edu>
This commit is contained in:
Mathy Vanhoef 2019-03-31 17:43:44 +02:00 committed by Jouni Malinen
parent d63edfa902
commit 8ad8585f91
1 changed files with 20 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
src/eap_peer/eap_pwd.c
View File

@ -594,6 +594,26 @@ eap_pwd_perform_commit_exchange(struct eap_sm *sm, struct eap_pwd_data *data,
goto fin; goto fin;
} }
/* verify received scalar */
if (crypto_bignum_is_zero(data->server_scalar) ||
crypto_bignum_is_one(data->server_scalar) ||
crypto_bignum_cmp(data->server_scalar,
crypto_ec_get_order(data->grp->group)) >= 0) {
wpa_printf(MSG_INFO,
"EAP-PWD (peer): received scalar is invalid");
goto fin;
}
/* verify received element */
if (!crypto_ec_point_is_on_curve(data->grp->group,
data->server_element) ||
crypto_ec_point_is_at_infinity(data->grp->group,
data->server_element)) {
wpa_printf(MSG_INFO,
"EAP-PWD (peer): received element is invalid");
goto fin;
}
/* check to ensure server's element is not in a small sub-group */ /* check to ensure server's element is not in a small sub-group */
if (!crypto_bignum_is_one(cofactor)) { if (!crypto_bignum_is_one(cofactor)) {
if (crypto_ec_point_mul(data->grp->group, data->server_element, if (crypto_ec_point_mul(data->grp->group, data->server_element,