gsupplicant: Set the scan callback in g_supplicant_interface_scan
authorJulien Massot <jmassot@aldebaran-robotics.com>
Thu, 21 Feb 2013 17:23:07 +0000 (18:23 +0100)
committerPatrik Flykt <patrik.flykt@linux.intel.com>
Fri, 22 Feb 2013 10:52:29 +0000 (12:52 +0200)
commit5230672c36edeb8ac49a365a10586fb1710071c4
treefcc664734e797d4001583a2fbb68a4bf9836715e
parent896120bead9a86ee1528cd6f1ff8949d183b9ed4
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