[PATCH v2 00/20] mesh support for wpa_supplicant

Bob Copeland me
Sun Sep 28 19:07:19 PDT 2014


On Sun, Sep 28, 2014 at 09:45:12PM +0300, Jouni Malinen wrote:
> On Mon, Sep 01, 2014 at 12:23:19AM -0400, Bob Copeland wrote:
> > This is an updated posting of the mesh-for-wpa_supplicant patchset,
> > which allows creating an open or secure (SAE) 802.11s mesh.
> 
> It looks like this breaks number of TDLS hwsim test cases (both with and
> without CONFIG_MESH=y). This seems to be due to set_key failing for peer
> with "No such file or directory", i.e., likely missing a proper STA
> entry for the peer. Any idea why? driver_nl80211.c has changes in
> sta_add handler, so I'd assume those are related.

Hmm, yes, it looks like this restores passing on TDLS cases (but breaks
the mesh test cases).  I guess mesh MPM should just be setting supp_rates
when marking the sta authenticated.

diff --git a/src/drivers/driver_nl80211.c b/src/drivers/driver_nl80211.c
index 44d07fe..4b0dad0 100644
--- a/src/drivers/driver_nl80211.c
+++ b/src/drivers/driver_nl80211.c
@@ -7754,11 +7754,12 @@ static int wpa_driver_nl80211_sta_add(void *priv,
 	NLA_PUT_U32(msg, NL80211_ATTR_IFINDEX, if_nametoindex(bss->ifname));
 	NLA_PUT(msg, NL80211_ATTR_MAC, ETH_ALEN, params->addr);
 
+	NLA_PUT(msg, NL80211_ATTR_STA_SUPPORTED_RATES,
+		params->supp_rates_len, params->supp_rates);
+	wpa_hexdump(MSG_DEBUG, "  * supported rates",
+		    params->supp_rates, params->supp_rates_len);
+
 	if (!params->set) {
-		NLA_PUT(msg, NL80211_ATTR_STA_SUPPORTED_RATES,
-			params->supp_rates_len, params->supp_rates);
-		wpa_hexdump(MSG_DEBUG, "  * supported rates",
-			    params->supp_rates, params->supp_rates_len);
 
 		if (params->aid) {
 			wpa_printf(MSG_DEBUG, "  * aid=%u", params->aid);


> I pushed my temporary work branch ('mesh') into hostap.git to make it
> easier to see where I'm with the review and to send any additional
> patches on top of that version, if needed.

Great, thanks!

-- 
Bob Copeland %% www.bobcopeland.com



More information about the Hostap mailing list