[PATCH] bgscan_simple: Fix short_scan_count comparison

Andrei Otcheretianski andrei.otcheretianski at intel.com
Mon Jan 2 06:26:26 PST 2017


From: Beni Lev <beni.lev at intel.com>

Currently, the check is done after we reached the maximum and another
scan is already triggered.
While at it, remove irrelevant comment.

Signed-off-by: Beni Lev <beni.lev at intel.com>
---
 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 a467cc5..bae72b9 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");
-- 
1.9.1




More information about the Hostap mailing list