aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDavid S. Miller <davem@davemloft.net>2017-12-18 15:11:26 -0500
committerDavid S. Miller <davem@davemloft.net>2017-12-18 15:11:26 -0500
commit0dc6905a6a99cc3f09de7e7b008c52d37c0a273f (patch)
treededc322d3566dac8300b68c3fda955a61006ab58
parent814a17841325643e2d08984d076f976a0b423872 (diff)
parentd91e8db5b629a3c8c81db4dc317a66c7b5591821 (diff)
Merge branch 'erspan-a-couple-fixes'
William Tu says: ==================== net: erspan: a couple fixes Haishuang Yan reports a couple of issues (wrong return value, pskb_may_pull) on erspan V1. Since erspan V2 is in net-next, this series fix the similar issues on v2. ==================== Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r--net/ipv4/ip_gre.c6
-rw-r--r--net/ipv6/ip6_gre.c6
2 files changed, 8 insertions, 4 deletions
diff --git a/net/ipv4/ip_gre.c b/net/ipv4/ip_gre.c
index 9a80d84fc182..fd4d6e96da7e 100644
--- a/net/ipv4/ip_gre.c
+++ b/net/ipv4/ip_gre.c
@@ -279,7 +279,6 @@ static int erspan_rcv(struct sk_buff *skb, struct tnl_ptk_info *tpi,
279 * Use ERSPAN 10-bit session ID as key. 279 * Use ERSPAN 10-bit session ID as key.
280 */ 280 */
281 tpi->key = cpu_to_be32(ntohs(ershdr->session_id) & ID_MASK); 281 tpi->key = cpu_to_be32(ntohs(ershdr->session_id) & ID_MASK);
282 pkt_md = (struct erspan_metadata *)(ershdr + 1);
283 tunnel = ip_tunnel_lookup(itn, skb->dev->ifindex, 282 tunnel = ip_tunnel_lookup(itn, skb->dev->ifindex,
284 tpi->flags | TUNNEL_KEY, 283 tpi->flags | TUNNEL_KEY,
285 iph->saddr, iph->daddr, tpi->key); 284 iph->saddr, iph->daddr, tpi->key);
@@ -287,7 +286,10 @@ static int erspan_rcv(struct sk_buff *skb, struct tnl_ptk_info *tpi,
287 if (tunnel) { 286 if (tunnel) {
288 len = gre_hdr_len + erspan_hdr_len(ver); 287 len = gre_hdr_len + erspan_hdr_len(ver);
289 if (unlikely(!pskb_may_pull(skb, len))) 288 if (unlikely(!pskb_may_pull(skb, len)))
290 return -ENOMEM; 289 return PACKET_REJECT;
290
291 ershdr = (struct erspan_base_hdr *)(skb->data + gre_hdr_len);
292 pkt_md = (struct erspan_metadata *)(ershdr + 1);
291 293
292 if (__iptunnel_pull_header(skb, 294 if (__iptunnel_pull_header(skb,
293 len, 295 len,
diff --git a/net/ipv6/ip6_gre.c b/net/ipv6/ip6_gre.c
index 5c9c65f1d5c2..87b9892dfa23 100644
--- a/net/ipv6/ip6_gre.c
+++ b/net/ipv6/ip6_gre.c
@@ -515,7 +515,6 @@ static int ip6erspan_rcv(struct sk_buff *skb, int gre_hdr_len,
515 515
516 ver = (ntohs(ershdr->ver_vlan) & VER_MASK) >> VER_OFFSET; 516 ver = (ntohs(ershdr->ver_vlan) & VER_MASK) >> VER_OFFSET;
517 tpi->key = cpu_to_be32(ntohs(ershdr->session_id) & ID_MASK); 517 tpi->key = cpu_to_be32(ntohs(ershdr->session_id) & ID_MASK);
518 pkt_md = (struct erspan_metadata *)(ershdr + 1);
519 518
520 tunnel = ip6gre_tunnel_lookup(skb->dev, 519 tunnel = ip6gre_tunnel_lookup(skb->dev,
521 &ipv6h->saddr, &ipv6h->daddr, tpi->key, 520 &ipv6h->saddr, &ipv6h->daddr, tpi->key,
@@ -524,7 +523,10 @@ static int ip6erspan_rcv(struct sk_buff *skb, int gre_hdr_len,
524 int len = erspan_hdr_len(ver); 523 int len = erspan_hdr_len(ver);
525 524
526 if (unlikely(!pskb_may_pull(skb, len))) 525 if (unlikely(!pskb_may_pull(skb, len)))
527 return -ENOMEM; 526 return PACKET_REJECT;
527
528 ershdr = (struct erspan_base_hdr *)skb->data;
529 pkt_md = (struct erspan_metadata *)(ershdr + 1);
528 530
529 if (__iptunnel_pull_header(skb, len, 531 if (__iptunnel_pull_header(skb, len,
530 htons(ETH_P_TEB), 532 htons(ETH_P_TEB),