GRE: fix single loop decap and add test 21/9821/2
authorNeale Ranns <nranns@cisco.com>
Wed, 13 Dec 2017 10:47:27 +0000 (02:47 -0800)
committerDamjan Marion <dmarion.lists@gmail.com>
Wed, 13 Dec 2017 15:14:49 +0000 (15:14 +0000)
Change-Id: I64e8a76a17057ae69de72a5a80c0a194cd0c21cb
Signed-off-by: Neale Ranns <nranns@cisco.com>
src/vnet/gre/node.c
test/patches/scapy-2.3.3/inet6.py.patch
test/test_gre.py

index e10f116..1a5fc87 100644 (file)
@@ -398,7 +398,7 @@ gre_input (vlib_main_t * vm,
          int verr0;
          u32 i0, next0;
          u32 ip4_tun_src0, ip4_tun_dst0;
-         u32 ip6_tun_src0[4], ip6_tun_dst0[4];
+         u64 ip6_tun_src0[2], ip6_tun_dst0[2];
 
          bi0 = from[0];
          to_next[0] = bi0;
index f98e709..cf3217e 100644 (file)
@@ -183,3 +183,12 @@ diff --git a/scapy/layers/inet6.py b/scapy/layers/inet6.py
  ########################### Fragmentation Header ############################
 
  class IPv6ExtHdrFragment(_IPv6ExtHdr):
+diff --git a/scapy/layers/inet6.py b/scapy/layers/inet6.py
+index 20afedf..ae3c4dd 100644
+--- a/scapy/layers/inet6.py
++++ b/scapy/layers/inet6.py
+@@ -3888,3 +3888,4 @@ bind_layers(IPv6,      UDP,      nh = socket.IPPROTO_UDP )
+ bind_layers(IP,        IPv6,     proto = socket.IPPROTO_IPV6 )
+ bind_layers(IPv6,      IPv6,     nh = socket.IPPROTO_IPV6 )
+ bind_layers(IPv6,      IP,       nh = socket.IPPROTO_IPIP )
++bind_layers(IPv6,      GRE,      nh = socket.IPPROTO_GRE )
index 9046b05..5cdc063 100644 (file)
@@ -507,6 +507,9 @@ class TestGRE(VppTestCase):
     def test_gre6(self):
         """ GRE IPv6 tunnel Tests """
 
+        self.pg1.config_ip6()
+        self.pg1.resolve_ndp()
+
         #
         # Create an L3 GRE tunnel.
         #  - set it admin up
@@ -570,6 +573,27 @@ class TestGRE(VppTestCase):
         self.verify_tunneled_6o6(self.pg2, rx, tx,
                                  self.pg2.local_ip6, "1002::1")
 
+        #
+        # Test decap. decapped packets go out pg1
+        #
+        tx = self.create_tunnel_stream_6o6(self.pg2,
+                                           "1002::1",
+                                           self.pg2.local_ip6,
+                                           "2001::1",
+                                           self.pg1.remote_ip6)
+        self.vapi.cli("clear trace")
+        self.pg2.add_stream(tx)
+
+        self.pg_enable_capture(self.pg_interfaces)
+        self.pg_start()
+        rx = self.pg1.get_capture(len(tx))
+
+        #
+        # RX'd packet is UDP over IPv6, test the GRE header is gone.
+        #
+        self.assertFalse(rx[0].haslayer(GRE))
+        self.assertEqual(rx[0][IPv6].dst, self.pg1.remote_ip6)
+
         #
         # test case cleanup
         #
@@ -578,6 +602,7 @@ class TestGRE(VppTestCase):
         gre_if.remove_vpp_config()
 
         self.pg2.unconfig_ip6()
+        self.pg1.unconfig_ip6()
 
     def test_gre_vrf(self):
         """ GRE tunnel VRF Tests """