DGNum/hostapd
6
0
Fork 0
Code Issues Pull requests 1 Projects Releases Packages Wiki Activity Actions

WPS: Fix authorized MAC removal

Browse source 
Missing array referencing ended up only the first entry in the list to
be compared.

Fixes: 31fcea931d ("WPS 2.0: Add support for AuthorizedMACs attribute")
Signed-off-by: Jouni Malinen <j@w1.fi>
This commit is contained in:
Jouni Malinen 2024-01-13 23:11:08 +02:00
parent 95123ab3b3
commit 58027cfecc
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
src/wps/wps_registrar.c
View file

@ -259,7 +259,7 @@ static void wps_registrar_remove_authorized_mac(struct wps_registrar *reg,
wpa_printf(MSG_DEBUG, "WPS: Remove authorized MAC " MACSTR, wpa_printf(MSG_DEBUG, "WPS: Remove authorized MAC " MACSTR,
MAC2STR(addr)); MAC2STR(addr));
for (i = 0; i < WPS_MAX_AUTHORIZED_MACS; i++) { for (i = 0; i < WPS_MAX_AUTHORIZED_MACS; i++) {
if (os_memcmp(reg->authorized_macs, addr, ETH_ALEN) == 0) if (os_memcmp(reg->authorized_macs[i], addr, ETH_ALEN) == 0)
break; break;
} }
if (i == WPS_MAX_AUTHORIZED_MACS) { if (i == WPS_MAX_AUTHORIZED_MACS) {