linux-kernel - Re: [34/49] igb: only use vlan_gro

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <4D40A223.8020507@intel.com>
Date:	Wed, 26 Jan 2011 14:37:23 -0800
From:	Alexander Duyck <alexander.h.duyck@...el.com>
To:	Greg KH <gregkh@...e.de>
CC:	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	"stable@...nel.org" <stable@...nel.org>,
	"stable-review@...nel.org" <stable-review@...nel.org>,
	"torvalds@...ux-foundation.org" <torvalds@...ux-foundation.org>,
	"akpm@...ux-foundation.org" <akpm@...ux-foundation.org>,
	"alan@...rguk.ukuu.org.uk" <alan@...rguk.ukuu.org.uk>,
	"Kirsher, Jeffrey T" <jeffrey.t.kirsher@...el.com>,
	Eric Dumazet <eric.dumazet@...il.com>,
	"David S. Miller" <davem@...emloft.net>
Subject: Re: [34/49] igb: only use vlan_gro_receive if vlans are registered

On 1/5/2011 3:00 PM, Greg KH wrote:
> 2.6.32-longterm review patch.  If anyone has any objections, please let us know.
>
> ------------------
>
> From: Alexander Duyck<alexander.h.duyck@...el.com>
>
> commit 31b24b955c3ebbb6f3008a6374e61cf7c05a193c upstream.
>
> This change makes it so that vlan_gro_receive is only used if vlans have been
> registered to the adapter structure.  Previously we were just sending all vlan
> tagged frames in via this function but this results in a null pointer
> dereference when vlans are not registered.
>
> [ This fixes bugzilla entry 15582 -Eric Dumazet]
>
> Signed-off-by: Alexander Duyck<alexander.h.duyck@...el.com>
> Signed-off-by: Jeff Kirsher<jeffrey.t.kirsher@...el.com>
> Acked-by: Eric Dumazet<eric.dumazet@...il.com>
> Signed-off-by: David S. Miller<davem@...emloft.net>
> Signed-off-by: Greg Kroah-Hartman<gregkh@...e.de>
>
> ---
>   drivers/net/igb/igb_main.c |    2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> --- a/drivers/net/igb/igb_main.c
> +++ b/drivers/net/igb/igb_main.c
> @@ -4560,7 +4560,7 @@ static void igb_receive_skb(struct igb_r
>   	bool vlan_extracted = (adapter->vlgrp&&  (status&  E1000_RXD_STAT_VP));
>
>   	skb_record_rx_queue(skb, ring->queue_index);
> -	if (vlan_extracted)
> +	if (vlan_extracted && adapter->vlgrp)
>   		vlan_gro_receive(&ring->napi, adapter->vlgrp,
>   		                 le16_to_cpu(rx_desc->wb.upper.vlan),
>   		                 skb);
>
>

I was just reviewing some history on this and I noticed that this patch 
appears to be applied to the wrong kernel.  The change isn't needed for 
2.6.32 as evidenced by the fact that vlan_extracted already includes 
adapter->vlgrp as part of computing it's value.  However, the change is 
needed for 2.6.33.  The original bugzilla include the patch for that kernel.

Thanks,

Alex

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/