[PATCH 05/16] WNM: Remove unused age parameter for neighbor comparison
benjamin at sipsolutions.net
benjamin at sipsolutions.net
Mon Apr 29 04:51:46 PDT 2024
From: Benjamin Berg <benjamin.berg at intel.com>
compare_scan_neighbor_results was taking an age paramter to check
whether the BSS has been seen recently. This was used historically in a
codepath when no new scan was done. However, the logic was changed in
commit 20ed289a785c ("WNM: Clean up old scan data processing") and the
parameter is not used at all anymore as it is replaced by a different
logic. Remove it.
Signed-off-by: Benjamin Berg <benjamin.berg at intel.com>
---
wpa_supplicant/wnm_sta.c | 17 ++---------------
1 file changed, 2 insertions(+), 15 deletions(-)
diff --git a/wpa_supplicant/wnm_sta.c b/wpa_supplicant/wnm_sta.c
index 721baeddb..d3bc03202 100644
--- a/wpa_supplicant/wnm_sta.c
+++ b/wpa_supplicant/wnm_sta.c
@@ -740,7 +740,7 @@ static struct wpa_bss * find_better_target(struct wpa_bss *a,
}
static struct wpa_bss *
-compare_scan_neighbor_results(struct wpa_supplicant *wpa_s, os_time_t age_secs,
+compare_scan_neighbor_results(struct wpa_supplicant *wpa_s,
enum mbo_transition_reject_reason *reason)
{
u8 i;
@@ -777,19 +777,6 @@ compare_scan_neighbor_results(struct wpa_supplicant *wpa_s, os_time_t age_secs,
continue;
}
- if (age_secs) {
- struct os_reltime now;
-
- if (os_get_reltime(&now) == 0 &&
- os_reltime_expired(&now, &target->last_update,
- age_secs)) {
- wpa_printf(MSG_DEBUG,
- "Candidate BSS is more than %ld seconds old",
- age_secs);
- continue;
- }
- }
-
/*
* TODO: Could consider allowing transition to another ESS if
* PMF was enabled for the association.
@@ -1198,7 +1185,7 @@ int wnm_scan_process(struct wpa_supplicant *wpa_s, bool pre_scan_check)
wpa_s->wnm_transition_scan = false;
/* Compare the Neighbor Report and scan results */
- bss = compare_scan_neighbor_results(wpa_s, 0, &reason);
+ bss = compare_scan_neighbor_results(wpa_s, &reason);
/*
* If this is a pre-scan check, returning 0 will trigger a scan and
--
2.44.0
More information about the Hostap
mailing list