diff options
author | Pravin B Shelar <pshelar@nicira.com> | 2013-08-13 01:41:06 -0700 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2013-08-13 16:52:50 -0700 |
commit | 4221f40513233fa8edeef7fc82e44163fde03b9b (patch) | |
tree | b7213a4e780c4c5b61ef4f103bcd5346c960a365 /net/ipv4/ip_tunnel_core.c | |
parent | 50f850fdf91a9ed21bfca982932520ee21a4ccb9 (diff) | |
download | op-kernel-dev-4221f40513233fa8edeef7fc82e44163fde03b9b.zip op-kernel-dev-4221f40513233fa8edeef7fc82e44163fde03b9b.tar.gz |
ip_tunnel: Do not use inner ip-header-id for tunnel ip-header-id.
Using inner-id for tunnel id is not safe in some rare cases.
E.g. packets coming from multiple sources entering same tunnel
can have same id. Therefore on tunnel packet receive we
could have packets from two different stream but with same
source and dst IP with same ip-id which could confuse ip packet
reassembly.
Following patch reverts optimization from commit
490ab08127 (IP_GRE: Fix IP-Identification.)
CC: Jarno Rajahalme <jrajahalme@nicira.com>
CC: Ansis Atteka <aatteka@nicira.com>
Signed-off-by: Pravin B Shelar <pshelar@nicira.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ipv4/ip_tunnel_core.c')
-rw-r--r-- | net/ipv4/ip_tunnel_core.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/net/ipv4/ip_tunnel_core.c b/net/ipv4/ip_tunnel_core.c index 7167b08..850525b 100644 --- a/net/ipv4/ip_tunnel_core.c +++ b/net/ipv4/ip_tunnel_core.c @@ -76,9 +76,7 @@ int iptunnel_xmit(struct net *net, struct rtable *rt, iph->daddr = dst; iph->saddr = src; iph->ttl = ttl; - tunnel_ip_select_ident(skb, - (const struct iphdr *)skb_inner_network_header(skb), - &rt->dst); + __ip_select_ident(iph, &rt->dst, (skb_shinfo(skb)->gso_segs ?: 1) - 1); err = ip_local_out(skb); if (unlikely(net_xmit_eval(err))) |