From 5230672c36edeb8ac49a365a10586fb1710071c4 Mon Sep 17 00:00:00 2001 From: Julien Massot Date: Thu, 21 Feb 2013 18:23:07 +0100 Subject: gsupplicant: Set the scan callback in g_supplicant_interface_scan The callback is required to properly handle scanning errors reported by wpa_supplicant through the scan_done signal. Steps to reproduce: 1. Set a country code to the WiFi card which have more frequencies allowed than the world roaming allow. 2. Connect to a WiFi network not available in the world roaming setup. 3. Stop connman. 4. Change the country code to 00, be sure the frequencies use in step 2. is no longer allowed. Take care of cfg80211 authorize frequencies when beacons are received, I prefer using mac80211_hwsim from here to avoid side effects about beacon reception. 5. Start connman again. Now ConnMan is stuck in scanning state as the scan_fast method of plugins/wifi.c is called with a forbidden frequency. wpa_supplicant returns an error received in "gsupplicant/supplicant.c: signal_scan_done", where the code enters 'if (success == FALSE)' but unfortunately scan_callback is NULL at this point as the scan_callback is normally set in interface_scan_result(). --- gsupplicant/supplicant.c | 3 +++ 1 file changed, 3 insertions(+) (limited to 'gsupplicant') diff --git a/gsupplicant/supplicant.c b/gsupplicant/supplicant.c index f33c2df1..be5e72bf 100644 --- a/gsupplicant/supplicant.c +++ b/gsupplicant/supplicant.c @@ -2886,6 +2886,9 @@ int g_supplicant_interface_scan(GSupplicantInterface *interface, data->user_data = user_data; data->scan_params = scan_data; + interface->scan_callback = callback; + interface->scan_data = user_data; + ret = supplicant_dbus_method_call(interface->path, SUPPLICANT_INTERFACE ".Interface", "Scan", interface_scan_params, interface_scan_result, data); -- cgit v1.2.3