diff options
| author | Jouni Malinen <jouni@qca.qualcomm.com> | 2016-09-20 15:00:22 (GMT) |
|---|---|---|
| committer | Jouni Malinen <j@w1.fi> | 2016-09-21 21:34:19 (GMT) |
| commit | 4424aa5d7d1a1624b2a2e0241430da7ee3e80a54 (patch) | |
| tree | 4fa94627b0b685099d521abc7ba8a1da8a535abf /wpa_supplicant | |
| parent | 81258efacb57940a2a902504aec440dc59664c34 (diff) | |
| download | hostap-4424aa5d7d1a1624b2a2e0241430da7ee3e80a54.zip hostap-4424aa5d7d1a1624b2a2e0241430da7ee3e80a54.tar.gz hostap-4424aa5d7d1a1624b2a2e0241430da7ee3e80a54.tar.bz2 | |
P2P: Fix D-Bus persistent parameter in group started event on GO
When starting a P2P GO, the struct p2p_go_neg_results may use
persistent_group == 2 to indicate use of persistent reconnect. Setting
ssid->p2p_persistent_group based on this did not take into account this
special case and that ended up in D-Bus code trying to encode 2 as a
DBUS_TYPE_BOOLEAN value which results in an assert from the library. Fix
this by setting ssid->p2p_persistent_group to 0 or 1 instead of raw
params->persistent_group value without any filtering.
This is similar to an earlier fix in commit
112fdee738d28c4e8bfb66ad7202d4348c4e7771 ('P2P: Fix D-Bus persistent
parameter in group started event') that addressed another code path in
sending out this D-Bus signal.
Signed-off-by: Jouni Malinen <jouni@qca.qualcomm.com>
Diffstat (limited to 'wpa_supplicant')
| -rw-r--r-- | wpa_supplicant/p2p_supplicant.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/wpa_supplicant/p2p_supplicant.c b/wpa_supplicant/p2p_supplicant.c index 31eeb38..b89d43e 100644 --- a/wpa_supplicant/p2p_supplicant.c +++ b/wpa_supplicant/p2p_supplicant.c @@ -1881,7 +1881,7 @@ static void wpas_start_wps_go(struct wpa_supplicant *wpa_s, wpa_config_set_network_defaults(ssid); ssid->temporary = 1; ssid->p2p_group = 1; - ssid->p2p_persistent_group = params->persistent_group; + ssid->p2p_persistent_group = !!params->persistent_group; ssid->mode = group_formation ? WPAS_MODE_P2P_GROUP_FORMATION : WPAS_MODE_P2P_GO; ssid->frequency = params->freq; |