Patchwork wpa_supplicant: don't save prev_sched_ssid if not needed

login
register
mail settings
Submitter dvdspndl@gmail.com
Date March 7, 2013, 9:26 a.m.
Message ID <1362648393-5832-1-git-send-email-dvdspndl@gmail.com>
Download mbox | patch
Permalink /patch/225774/
State Accepted
Commit 7ed52f678bffecd0891562035a50a69a1251b788
Headers show

Comments

dvdspndl@gmail.com - March 7, 2013, 9:26 a.m.
From: David Spinadel <david.spinadel@intel.com>

If the whole ssid list was scanned, don't save the prev
sheduled ssid since we are going to start from beggining.
It avoids starting sched scan from prev_sched_ssid after
scanning the full list, in case of adding network to ssid
during scheduled scan.

Signed-hostap: David Spinadel <david.spinadel@intel.com>
---
 wpa_supplicant/scan.c |    4 ++++
 1 file changed, 4 insertions(+)
Jouni Malinen - March 10, 2013, 3:33 p.m.
On Thu, Mar 07, 2013 at 11:26:33AM +0200, David Spinadel wrote:
> If the whole ssid list was scanned, don't save the prev
> sheduled ssid since we are going to start from beggining.
> It avoids starting sched scan from prev_sched_ssid after
> scanning the full list, in case of adding network to ssid
> during scheduled scan.

Thanks, applied.

Patch

diff --git a/wpa_supplicant/scan.c b/wpa_supplicant/scan.c
index c3f9981..0e45079 100644
--- a/wpa_supplicant/scan.c
+++ b/wpa_supplicant/scan.c
@@ -1120,6 +1120,10 @@  scan:
 		wpa_s->sched_scan_interval *= 2;
 	}
 
+	/* If there is no more ssids, start next time from the beginning */
+	if (!ssid)
+		wpa_s->prev_sched_ssid = NULL;
+
 	return 0;
 }