commit b9aeb5ddb112bc24ec63de9a2c8c16ad98519b3c
author Jimmy Chen <jimmycmchen@google.com> Wed Jan 15 16:52:29 2020 +0800
committer Jimmy Chen <jimmycmchen@google.com> Wed Jan 15 17:45:53 2020 +0800
tree 117837896cd8f8a981e6ce3c6c21e372a4b6552f
parent dd48efd31e580eeb88c82351d755f2d2e42c7e97

p2p: clean p2p scan work in scan result handler

If p2p find is running and a fast join is triggered,
our scan result handler would replace original one.
If our handler does not clean p2p scan work, p2p would
consider there is still a running p2p find and reject new
p2p find request.

Bug: 147720382
Test: CtsVerifier - WiFi Direct
Change-Id: Ib05eafeba570cc0224cf318c12269be8d397075e

wpa_supplicant/hidl/1.3/p2p_iface.cpp

diff --git a/wpa_supplicant/hidl/1.3/p2p_iface.cpp b/wpa_supplicant/hidl/1.3/p2p_iface.cpp
index 833d624..bd893a3 100644
--- a/wpa_supplicant/hidl/1.3/p2p_iface.cpp
+++ b/wpa_supplicant/hidl/1.3/p2p_iface.cpp
@@ -209,6 +209,12 @@
     struct wpa_supplicant *wpa_s,
     struct wpa_scan_results *scan_res)
 {
+	if (wpa_s->p2p_scan_work) {
+		struct wpa_radio_work *work = wpa_s->p2p_scan_work;
+		wpa_s->p2p_scan_work = NULL;
+		radio_work_done(work);
+	}
+
 	if (pending_scan_res_join_callback) {
 		pending_scan_res_join_callback();
 	}
@@ -310,6 +316,9 @@
 	ret = wpa_drv_scan(wpa_s, &params);
 	if (!ret) {
 		os_get_reltime(&wpa_s->scan_trigger_time);
+		if (wpa_s->scan_res_handler) {
+			wpa_printf(MSG_DEBUG, "Replace current running scan result handler");
+		}
 		wpa_s->scan_res_handler = scanResJoinWrapper;
 		wpa_s->own_scan_requested = 1;
 		wpa_s->clear_driver_scan_cache = 0;
@@ -382,6 +391,13 @@
 
 void scanResJoinIgnore(struct wpa_supplicant *wpa_s, struct wpa_scan_results *scan_res) {
 	wpa_printf(MSG_DEBUG, "P2P: Ignore group join scan results.");
+
+	if (wpa_s->p2p_scan_work) {
+		struct wpa_radio_work *work = wpa_s->p2p_scan_work;
+		wpa_s->p2p_scan_work = NULL;
+		radio_work_done(work);
+	}
+
 }
 
 }  // namespace