diff options
| author | Timo Teräs <timo.teras@iki.fi> | 2013-12-16 11:02:09 +0200 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2013-12-18 17:44:33 -0500 |
| commit | 0e3da5bb8da45890b1dc413404e0f978ab71173e (patch) | |
| tree | 2d3bd7a6e9c9c38520ac3bbe132e0f8940a9a18a /net/ipv4/ip_gre.c | |
| parent | a81d8762d71396f69d27187a909fcc69f1a7be75 (diff) | |
| download | lwn-0e3da5bb8da45890b1dc413404e0f978ab71173e.tar.gz lwn-0e3da5bb8da45890b1dc413404e0f978ab71173e.zip | |
ip_gre: fix msg_name parsing for recvfrom/recvmsg
ipgre_header_parse() needs to parse the tunnel's ip header and it
uses mac_header to locate the iphdr. This got broken when gre tunneling
was refactored as mac_header is no longer updated to point to iphdr.
Introduce skb_pop_mac_header() helper to do the mac_header assignment
and use it in ipgre_rcv() to fix msg_name parsing.
Bug introduced in commit c54419321455 (GRE: Refactor GRE tunneling code.)
Cc: Pravin B Shelar <pshelar@nicira.com>
Signed-off-by: Timo Teräs <timo.teras@iki.fi>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ipv4/ip_gre.c')
| -rw-r--r-- | net/ipv4/ip_gre.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/net/ipv4/ip_gre.c b/net/ipv4/ip_gre.c index d7aea4c5b940..e560ef34cf4b 100644 --- a/net/ipv4/ip_gre.c +++ b/net/ipv4/ip_gre.c @@ -217,6 +217,7 @@ static int ipgre_rcv(struct sk_buff *skb, const struct tnl_ptk_info *tpi) iph->saddr, iph->daddr, tpi->key); if (tunnel) { + skb_pop_mac_header(skb); ip_tunnel_rcv(tunnel, skb, tpi, log_ecn_error); return PACKET_RCVD; } |