diff options
author | Eric Dumazet <edumazet@google.com> | 2012-10-07 02:28:06 +0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2012-10-07 22:49:17 +0400 |
commit | ca07e43e288956a0ad5e6bd075f7aa1fca3bca00 (patch) | |
tree | fa3b345f2940b257f7c1b5b268f2ee52926bcbf5 /include | |
parent | 51ec04038c113a811b177baa85d293feff9ce995 (diff) | |
download | linux-ca07e43e288956a0ad5e6bd075f7aa1fca3bca00.tar.xz |
net: gro: fix a potential crash in skb_gro_reset_offset
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@google.com>
Cc: Herbert Xu <herbert@gondor.apana.org.au>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'include')
-rw-r--r-- | include/linux/netdevice.h | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/include/linux/netdevice.h b/include/linux/netdevice.h index 01646aa53b0e..a659fd0ba965 100644 --- a/include/linux/netdevice.h +++ b/include/linux/netdevice.h @@ -1663,7 +1663,6 @@ extern int netpoll_trap(void); #endif extern int skb_gro_receive(struct sk_buff **head, struct sk_buff *skb); -extern void skb_gro_reset_offset(struct sk_buff *skb); static inline unsigned int skb_gro_offset(const struct sk_buff *skb) { |