Search Linux Wireless

Re: [PATCH] mac80211: fixed injection in monitor mode

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

 



On Fri, Oct 16, 2009 at 04:09:21AM +0200, Gabriele Gristina wrote:
> I have a problem with mac80211 injection:
> if i setup interface in monitor mode with iwconfig and i try to send a
> custum 802.11 frame i see twice the packet in monitor interface.
> The first packet is right, the second is different only in the radiotap
> header: rtap len is fixed to 13 bytes...
> If i setup interface with airmon-ng the packet is sended once but the
> radiotap header is fixed to 13 bytes.
> 
> Patch for 2.6.31.4
> 
> --- net/mac80211/main.c.orig    2009-10-16 00:50:00.000000000 +0200
> +++ net/mac80211/main.c    2009-10-15 23:06:03.000000000 +0200
> @@ -607,6 +607,9 @@
>      skb->protocol = htons(ETH_P_802_2);
>      memset(skb->cb, 0, sizeof(skb->cb));
>  
> +    if (!(info->flags & IEEE80211_TX_CTL_REQ_TX_STATUS))
> +        goto out_no_echo;
> +
>      rcu_read_lock();
>      list_for_each_entry_rcu(sdata, &local->interfaces, list) {
>          if (sdata->vif.type == NL80211_IFTYPE_MONITOR) {
> @@ -630,7 +633,8 @@
>          skb = NULL;
>      }
>      rcu_read_unlock();
> -    dev_kfree_skb(skb);
> +out_no_echo:
> +    if (skb) dev_kfree_skb(skb);
>  }
>  EXPORT_SYMBOL(ieee80211_tx_status);
> 

Thanks for the patch!  But I need a Signed-off-by line...

	http://linux.yyz.us/patch-format.html

-- 
John W. Linville		Someday the world will need a hero, and you
linville@xxxxxxxxxxxxx			might be all we have.  Be ready.
--
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

[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