On Thu, 2011-09-15 at 13:25 +0300, Arik Nemtsov wrote:
> + case NL80211_TDLS_ENABLE_LINK:
> + rcu_read_lock();
> + sta = sta_info_get(sdata, peer);
> + if (sta) {
> + set_sta_flags(sta, WLAN_STA_AUTHORIZED);
> + sta->tdls_link_enabled = true;
> + }
> + rcu_read_unlock();
> + break;
This seems to require the station already having been added, but
couldn't this create the data race you were worried about? Could you not
simply just create the station once it is authorized?
> + case NL80211_TDLS_DISABLE_LINK:
> + rcu_read_lock();
> + sta = sta_info_get(sdata, peer);
> + if (sta) {
> + sta->tdls_link_enabled = false;
> + sta_info_destroy_addr(sdata, peer);
> + }
> + rcu_read_unlock();
> + break;
Isn't that equivalent to just deleting the station?
johannes
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
