From: Adil Saeed Musthafa Date: Mon, 24 Nov 2025 21:13:15 +0000 (-0800) Subject: FILS: Remove unused arguments from writing Association Response elements X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=779ddb4e4566dbab6118e2cc97583e4b066f75a8;p=thirdparty%2Fhostap.git FILS: Remove unused arguments from writing Association Response elements The elements from the (Re)Association Request frame were never needed for the FILS case and were likely originally included just because a similar function for FT included them. Signed-off-by: Adil Saeed Musthafa --- diff --git a/src/ap/ieee802_11.c b/src/ap/ieee802_11.c index 34987b568..3b21459ad 100644 --- a/src/ap/ieee802_11.c +++ b/src/ap/ieee802_11.c @@ -5477,8 +5477,7 @@ static u16 send_assoc_resp(struct hostapd_data *hapd, struct sta_info *sta, sta->auth_alg == WLAN_AUTH_FILS_SK_PFS || sta->auth_alg == WLAN_AUTH_FILS_PK)) p = wpa_auth_write_assoc_resp_fils(sta->wpa_sm, p, - buf + buflen - p, - ies, ies_len); + buf + buflen - p); #endif /* CONFIG_FILS */ #ifdef CONFIG_OWE diff --git a/src/ap/wpa_auth.h b/src/ap/wpa_auth.h index fc95019e7..12a8b6b1a 100644 --- a/src/ap/wpa_auth.h +++ b/src/ap/wpa_auth.h @@ -639,8 +639,7 @@ void wpa_auth_add_fils_pmk_pmkid(struct wpa_state_machine *sm, const u8 *pmk, u8 * wpa_auth_write_assoc_resp_owe(struct wpa_state_machine *sm, u8 *pos, size_t max_len); u8 * wpa_auth_write_assoc_resp_fils(struct wpa_state_machine *sm, - u8 *pos, size_t max_len, - const u8 *req_ies, size_t req_ies_len); + u8 *pos, size_t max_len); bool wpa_auth_write_fd_rsn_info(struct wpa_authenticator *wpa_auth, u8 *fd_rsn_info); void wpa_auth_set_auth_alg(struct wpa_state_machine *sm, u16 auth_alg); diff --git a/src/ap/wpa_auth_ie.c b/src/ap/wpa_auth_ie.c index 21ed31a41..220ac809f 100644 --- a/src/ap/wpa_auth_ie.c +++ b/src/ap/wpa_auth_ie.c @@ -1442,8 +1442,7 @@ u8 * wpa_auth_write_assoc_resp_owe(struct wpa_state_machine *sm, #ifdef CONFIG_FILS u8 * wpa_auth_write_assoc_resp_fils(struct wpa_state_machine *sm, - u8 *pos, size_t max_len, - const u8 *req_ies, size_t req_ies_len) + u8 *pos, size_t max_len) { int res;