Search Linux Wireless

Re: [PATCH] allow AP interfaces to handle BACK action frames

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, 2008-10-08 at 16:18 -0700, Andrey Yurovsky wrote:
> Add AP mode to the allowed interface check in ieee80211_rx_h_action.  This
> function handles BACK action frames such as ADDBA and AP interfaces need to
> receive those.
> 
> Signed-off-by: Andrey Yurovsky <andrey@xxxxxxxxxxx>

Fine with me, but I think you need to allow VLAN too? Also, how about
simply removing this check? Can anything bad happen when we try this in
an IBSS or on WDS or on MESH?

johannes

> diff --git a/net/mac80211/rx.c b/net/mac80211/rx.c
> index 080b0ce..fe406f5 100644
> --- a/net/mac80211/rx.c
> +++ b/net/mac80211/rx.c
> @@ -1553,7 +1553,8 @@ ieee80211_rx_h_action(struct ieee80211_rx_data *rx)
>  	 *	  of these frames in other modes as well!
>  	 */
>  	if (sdata->vif.type != NL80211_IFTYPE_STATION &&
> -	    sdata->vif.type != NL80211_IFTYPE_ADHOC)
> +	    sdata->vif.type != NL80211_IFTYPE_ADHOC &&
> +	    sdata->vif.type != NL80211_IFTYPE_AP)
>  		return RX_CONTINUE;
>  
>  	switch (mgmt->u.action.category) {
> 
> 
> --
> 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
> 

Attachment: signature.asc
Description: This is a digitally signed message part


[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]
  Powered by Linux