aboutsummaryrefslogtreecommitdiffstats
path: root/wpa_supplicant/events.c
diff options
context:
space:
mode:
authorJouni Malinen <j@w1.fi>2017-02-05 19:52:26 (GMT)
committerJouni Malinen <j@w1.fi>2017-02-05 19:52:26 (GMT)
commitbcf66493cf189f8fe0ee3281854cd4a5a1354c53 (patch)
tree04a17710f3bb158754b3b7a5e05e4eb44f1f571e /wpa_supplicant/events.c
parent89fa633afbc6eda97a6e196c70f5ae1443b33c2f (diff)
downloadhostap-bcf66493cf189f8fe0ee3281854cd4a5a1354c53.zip
hostap-bcf66493cf189f8fe0ee3281854cd4a5a1354c53.tar.gz
hostap-bcf66493cf189f8fe0ee3281854cd4a5a1354c53.tar.bz2
Fix estimated throughput based skip-roam case
Commit 8d1e693186336f85bf5d86bd094b5c9bd6f8fd02 ('Use estimated throughput to avoid signal based roaming decision') added a check for the current BSS estimated throughput being significantly higher than the selected BSS estimated throughput. However, this case for skipping a roam used "return 1" which actually allows the roam. Fix this by returning 0 in this case. Signed-off-by: Jouni Malinen <j@w1.fi>
Diffstat (limited to 'wpa_supplicant/events.c')
-rw-r--r--wpa_supplicant/events.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/wpa_supplicant/events.c b/wpa_supplicant/events.c
index 76805e0..24dbd3e 100644
--- a/wpa_supplicant/events.c
+++ b/wpa_supplicant/events.c
@@ -1561,7 +1561,7 @@ static int wpa_supplicant_need_to_roam(struct wpa_supplicant *wpa_s,
if (current_bss->est_throughput > selected->est_throughput + 5000) {
wpa_dbg(wpa_s, MSG_DEBUG,
"Skip roam - Current BSS has better estimated throughput");
- return 1;
+ return 0;
}
cur_est = current_bss->est_throughput;