From 407be00b6ca299e04c34f385df5e538c912fc0ec Mon Sep 17 00:00:00 2001 From: Jouni Malinen Date: Mon, 18 Mar 2013 17:11:15 +0200 Subject: [PATCH] P2P: Postpone P2P scan only if station mode scan is pending sta_scan_pending was supposed to be accurate enough for determining whether a P2P scan needs to be postponed. However, it has turned out that there were cases where sta_scan_pending was not cleared properly. While the known cases have now been addressed, it is possible that some other cases may still exist. To avoid issues with P2P operationg getting stuck, verify more carefully that there is a real pending station mode scan (either in progress or scheduled to be requested). Signed-hostap: Jouni Malinen --- wpa_supplicant/p2p_supplicant.c | 1 + wpa_supplicant/scan.c | 6 ++++++ wpa_supplicant/scan.h | 1 + 3 files changed, 8 insertions(+) diff --git a/wpa_supplicant/p2p_supplicant.c b/wpa_supplicant/p2p_supplicant.c index 76ddf0f25..94a8219aa 100644 --- a/wpa_supplicant/p2p_supplicant.c +++ b/wpa_supplicant/p2p_supplicant.c @@ -151,6 +151,7 @@ static int wpas_p2p_scan(void *ctx, enum p2p_scan_type type, int freq, for (ifs = wpa_s->global->ifaces; ifs; ifs = ifs->next) { if (ifs->sta_scan_pending && + (wpas_scan_scheduled(ifs) || ifs->scanning) && wpas_p2p_in_progress(wpa_s) == 2) { wpa_printf(MSG_DEBUG, "Delaying P2P scan to allow " "pending station mode scan to be " diff --git a/wpa_supplicant/scan.c b/wpa_supplicant/scan.c index 29a430c97..1a4f6955d 100644 --- a/wpa_supplicant/scan.c +++ b/wpa_supplicant/scan.c @@ -1613,3 +1613,9 @@ void scan_only_handler(struct wpa_supplicant *wpa_s, wpas_notify_scan_results(wpa_s); wpas_notify_scan_done(wpa_s, 1); } + + +int wpas_scan_scheduled(struct wpa_supplicant *wpa_s) +{ + return eloop_is_timeout_registered(wpa_supplicant_scan, wpa_s, NULL); +} diff --git a/wpa_supplicant/scan.h b/wpa_supplicant/scan.h index 5ecfa00cf..e892479f6 100644 --- a/wpa_supplicant/scan.h +++ b/wpa_supplicant/scan.h @@ -35,5 +35,6 @@ int wpa_supplicant_filter_bssid_match(struct wpa_supplicant *wpa_s, void wpa_supplicant_update_scan_int(struct wpa_supplicant *wpa_s, int sec); void scan_only_handler(struct wpa_supplicant *wpa_s, struct wpa_scan_results *scan_res); +int wpas_scan_scheduled(struct wpa_supplicant *wpa_s); #endif /* SCAN_H */