diff options
author | Vinay Adella <vinay.adella@atheros.com> | 2011-05-19 12:55:47 +0300 |
---|---|---|
committer | Jouni Malinen <j@w1.fi> | 2011-05-19 12:55:47 +0300 |
commit | 86795546f7bf06a306ee288a7c8e7aad6d6a58f3 (patch) | |
tree | 13fe6d40d98d56af3b0df801395aa89f841e5f51 /src | |
parent | fa5165586fa11ecdccb7c81d40979b51c2583fd2 (diff) | |
download | external_wpa_supplicant_8_ti-86795546f7bf06a306ee288a7c8e7aad6d6a58f3.zip external_wpa_supplicant_8_ti-86795546f7bf06a306ee288a7c8e7aad6d6a58f3.tar.gz external_wpa_supplicant_8_ti-86795546f7bf06a306ee288a7c8e7aad6d6a58f3.tar.bz2 |
WPS UPnP: Fix UPnP initialization for non-bridge case with some drivers
If the driver wrapper is setting up the interface up only at commit(),
UPnP initialization fails. Fix that by moving UPnP setup to happen after
the driver commit() call.
Diffstat (limited to 'src')
-rw-r--r-- | src/ap/hostapd.c | 11 | ||||
-rw-r--r-- | src/ap/wps_hostapd.c | 22 | ||||
-rw-r--r-- | src/ap/wps_hostapd.h | 6 |
3 files changed, 35 insertions, 4 deletions
diff --git a/src/ap/hostapd.c b/src/ap/hostapd.c index 4e5eb01..32db668 100644 --- a/src/ap/hostapd.c +++ b/src/ap/hostapd.c @@ -784,6 +784,17 @@ int hostapd_setup_interface_complete(struct hostapd_iface *iface, int err) return -1; } + /* + * WPS UPnP module can be initialized only when the "upnp_iface" is up. + * If "interface" and "upnp_iface" are the same (e.g., non-bridge + * mode), the interface is up only after driver_commit, so initialize + * WPS after driver_commit. + */ + for (j = 0; j < iface->num_bss; j++) { + if (hostapd_init_wps_complete(iface->bss[j])) + return -1; + } + if (hapd->setup_complete_cb) hapd->setup_complete_cb(hapd->setup_complete_cb_ctx); diff --git a/src/ap/wps_hostapd.c b/src/ap/wps_hostapd.c index fcbd89b..fc927f9 100644 --- a/src/ap/wps_hostapd.c +++ b/src/ap/wps_hostapd.c @@ -876,20 +876,34 @@ int hostapd_init_wps(struct hostapd_data *hapd, wps->model_description = hapd->conf->model_description; wps->model_url = hapd->conf->model_url; wps->upc = hapd->conf->upc; +#endif /* CONFIG_WPS_UPNP */ + + hostapd_register_probereq_cb(hapd, hostapd_wps_probe_req_rx, hapd); + + hapd->wps = wps; + + return 0; +} + + +int hostapd_init_wps_complete(struct hostapd_data *hapd) +{ + struct wps_context *wps = hapd->wps; + + if (hapd->wps == NULL) + return 0; +#ifdef CONFIG_WPS_UPNP if (hostapd_wps_upnp_init(hapd, wps) < 0) { wpa_printf(MSG_ERROR, "Failed to initialize WPS UPnP"); wps_registrar_deinit(wps->registrar); os_free(wps->network_key); os_free(wps); + hapd->wps = NULL; return -1; } #endif /* CONFIG_WPS_UPNP */ - hostapd_register_probereq_cb(hapd, hostapd_wps_probe_req_rx, hapd); - - hapd->wps = wps; - return 0; } diff --git a/src/ap/wps_hostapd.h b/src/ap/wps_hostapd.h index 338a220..6b28c13 100644 --- a/src/ap/wps_hostapd.h +++ b/src/ap/wps_hostapd.h @@ -19,6 +19,7 @@ int hostapd_init_wps(struct hostapd_data *hapd, struct hostapd_bss_config *conf); +int hostapd_init_wps_complete(struct hostapd_data *hapd); void hostapd_deinit_wps(struct hostapd_data *hapd); void hostapd_update_wps(struct hostapd_data *hapd); int hostapd_wps_add_pin(struct hostapd_data *hapd, const u8 *addr, @@ -50,6 +51,11 @@ static inline void hostapd_deinit_wps(struct hostapd_data *hapd) { } +static inline int hostapd_init_wps_complete(struct hostapd_data *hapd) +{ + return 0; +} + static inline void hostapd_update_wps(struct hostapd_data *hapd) { } |