From c32e45be8320abfc6309934d57b0bafa38c1c490 Mon Sep 17 00:00:00 2001 From: Shreyas Sheth Date: Thu, 6 Aug 2026 16:35:32 +0530 Subject: [PATCH] fix(esp_wifi): Harden dpp Auth confirm and drop mismatched auth confirms --- components/esp_wifi/lib | 2 +- .../wpa_supplicant/esp_supplicant/src/esp_dpp.c | 13 +++++++++++-- components/wpa_supplicant/src/common/dpp.c | 6 ++++++ 3 files changed, 18 insertions(+), 3 deletions(-) diff --git a/components/esp_wifi/lib b/components/esp_wifi/lib index b3074d18b34..f54999f6d0c 160000 --- a/components/esp_wifi/lib +++ b/components/esp_wifi/lib @@ -1 +1 @@ -Subproject commit b3074d18b3472e1e0e30d7f2f30b0b334387616b +Subproject commit f54999f6d0cf6965d2ce3758da748bebdd606663 diff --git a/components/wpa_supplicant/esp_supplicant/src/esp_dpp.c b/components/wpa_supplicant/esp_supplicant/src/esp_dpp.c index 0dd8f878689..30d84edfd40 100644 --- a/components/wpa_supplicant/esp_supplicant/src/esp_dpp.c +++ b/components/wpa_supplicant/esp_supplicant/src/esp_dpp.c @@ -286,8 +286,12 @@ static void esp_dpp_rx_auth_conf(struct action_rx_param *rx_param, uint8_t *dpp_ if (os_memcmp(rx_param->sa, auth->peer_mac_addr, ETH_ALEN) != 0) { wpa_printf(MSG_DEBUG, "DPP: MAC address mismatch (expected " MACSTR ") - drop", MAC2STR(auth->peer_mac_addr)); - rc = ESP_ERR_DPP_FAILURE; - goto fail; + return; + } + + if (auth->auth_success || !auth->waiting_auth_conf) { + wpa_printf(MSG_DEBUG, "DPP: Not waiting for Auth Confirm - drop"); + return; } eloop_cancel_timeout(esp_dpp_auth_conf_wait_timeout, NULL, NULL); @@ -328,6 +332,11 @@ static esp_err_t esp_dpp_rx_peer_disc_resp(struct action_rx_param *rx_param) return ESP_ERR_INVALID_ARG; } + if (!auth) { + wpa_printf(MSG_DEBUG, "DPP: No DPP Authentication in progress - drop"); + return ESP_OK; + } + if (rx_param->vendor_data_len < 2) { wpa_printf(MSG_INFO, "DPP: Too short vendor specific data"); return ESP_FAIL; diff --git a/components/wpa_supplicant/src/common/dpp.c b/components/wpa_supplicant/src/common/dpp.c index 0ac0d044b33..e88a1a7dc72 100644 --- a/components/wpa_supplicant/src/common/dpp.c +++ b/components/wpa_supplicant/src/common/dpp.c @@ -2687,6 +2687,12 @@ int dpp_auth_conf_rx(struct dpp_authentication *auth, const u8 *hdr, return -1; } + if (!auth->waiting_auth_conf) { + wpa_printf(MSG_DEBUG, + "DPP: Not waiting for Auth Confirm - drop"); + return -1; + } + auth->waiting_auth_conf = 0; wrapped_data = dpp_get_attr(attr_start, attr_len, DPP_ATTR_WRAPPED_DATA,