mirror of
https://github.com/espressif/esp-idf.git
synced 2024-10-05 20:47:46 -04:00
Bluedroid: Check only x component of passkey to avoid passkey impersonation attack.
This commit is contained in:
parent
5198a7a46b
commit
fb55f9f397
@ -764,8 +764,7 @@ void smp_process_pairing_public_key(tSMP_CB *p_cb, tSMP_INT_DATA *p_data)
|
||||
/* Check if the peer device's and own public key are not same. If they are same then
|
||||
* return pairing fail. This check is needed to avoid 'Impersonation in Passkey entry
|
||||
* protocol' vulnerability (CVE-2020-26558).*/
|
||||
if ((memcmp(p_cb->loc_publ_key.x, p_cb->peer_publ_key.x, sizeof(BT_OCTET32)) == 0) &&
|
||||
(memcmp(p_cb->loc_publ_key.y, p_cb->peer_publ_key.y, sizeof(BT_OCTET32)) == 0)) {
|
||||
if ((memcmp(p_cb->loc_publ_key.x, p_cb->peer_publ_key.x, sizeof(BT_OCTET32)) == 0)) {
|
||||
p_cb->status = SMP_PAIR_AUTH_FAIL;
|
||||
p_cb->failure = SMP_PAIR_AUTH_FAIL;
|
||||
reason = SMP_PAIR_AUTH_FAIL;
|
||||
|
Loading…
Reference in New Issue
Block a user