From e72faadb4955b8206e76a2c6a8e84718fe6a4bac Mon Sep 17 00:00:00 2001 From: Beni Lev Date: Mon, 2 Jan 2017 16:26:26 +0200 Subject: [PATCH] bgscan_simple: Fix short_scan_count comparison Previously, the check was done after we reached the maximum and another scan was already triggered. While at it, remove an irrelevant comment that the previous change in the logic here missed. Signed-off-by: Beni Lev --- wpa_supplicant/bgscan_simple.c | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/wpa_supplicant/bgscan_simple.c b/wpa_supplicant/bgscan_simple.c index a467cc5b9..bae72b934 100644 --- a/wpa_supplicant/bgscan_simple.c +++ b/wpa_supplicant/bgscan_simple.c @@ -56,12 +56,7 @@ static void bgscan_simple_timeout(void *eloop_ctx, void *timeout_ctx) } else { if (data->scan_interval == data->short_interval) { data->short_scan_count++; - /* - * Spend at most the duration of a long scan interval - * scanning at the short scan interval. After that, - * revert to the long scan interval. - */ - if (data->short_scan_count > data->max_short_scans) { + if (data->short_scan_count >= data->max_short_scans) { data->scan_interval = data->long_interval; wpa_printf(MSG_DEBUG, "bgscan simple: Backing " "off to long scan interval");