On Sat, 2010-03-13 at 21:53 +0200, Jouni Malinen wrote:
> static void ieee80211_send_deauth_disassoc(struct ieee80211_sub_if_data *sdata,
> const u8 *bssid, u16 stype, u16 reason,
> - void *cookie)
> + void *cookie, bool send_frame)
> {
> struct ieee80211_local *local = sdata->local;
> struct ieee80211_if_managed *ifmgd = &sdata->u.mgd;
> @@ -240,7 +240,11 @@ static void ieee80211_send_deauth_disass
> cfg80211_send_disassoc(sdata->dev, (u8 *)mgmt, skb->len);
> if (!(ifmgd->flags & IEEE80211_STA_MFP_ENABLED))
> IEEE80211_SKB_CB(skb)->flags |= IEEE80211_TX_INTFL_DONT_ENCRYPT;
> - ieee80211_tx_skb(sdata, skb);
> +
> + if (send_frame)
> + ieee80211_tx_skb(sdata, skb);
> + else
> + kfree_skb(skb);
> }
I'm wondering now why we even bother sending events to userspace when it
itself triggered the transition?
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
