[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-Id: <20121007.145010.1082292669443081397.davem@davemloft.net>
Date: Sun, 07 Oct 2012 14:50:10 -0400 (EDT)
From: David Miller <davem@...emloft.net>
To: eric.dumazet@...il.com
Cc: netdev@...r.kernel.org, herbert@...dor.apana.org.au
Subject: Re: [PATCH] net: gro: fix a potential crash in skb_gro_reset_offset
From: Eric Dumazet <eric.dumazet@...il.com>
Date: Sun, 07 Oct 2012 10:28:06 +0200
> From: Eric Dumazet <edumazet@...gle.com>
>
> Before accessing skb first fragment, better make sure there
> is one.
>
> This is probably not needed for old kernels, since an ethernet frame
> cannot contain only an ethernet header, but the recent GRO addition
> to tunnels makes this patch needed.
>
> Also skb_gro_reset_offset() can be static, it actually allows
> compiler to inline it.
>
> Signed-off-by: Eric Dumazet <edumazet@...gle.com>
> Cc: Herbert Xu <herbert@...dor.apana.org.au>
Applied, thanks Eric.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists