diff options
| author |   Neale Ranns <nranns@cisco.com> | 2019-10-30 17:34:14 +0000 |
|---|---|---|
| committer |   Ole Trøan <otroan@employees.org> | 2019-11-05 15:34:00 +0000 |
| commit | 0b6a857d85df97e887de7aaf00fd6bd2dae39bf8 (patch) | |
| tree | 9494d7544d7af1fb5381bfb0aea51f731a661afd /test/test_mtu.py | |
| parent | 3ea17d54a9a00c81bc672a7be1d48b765ac87ed2 (diff) | |
ip: Fragmentation fixes
Type: fix
if the packet is about to be fragmented, then don't call any of the
actions that expect the rewrite to have been written.
1) don't double count packets thru the adjacency (original & fragments)
2) don't double decrement the TTL for fragments
3) return to ip4-midchain post ip-frag if that's where we started.
4) only run midchain/mcast fixups if not fragmenting (if no errors)
Change-Id: Ib2866787a42713ee5871b87b597d8f74b901044b
Signed-off-by: Neale Ranns <nranns@cisco.com>
Diffstat (limited to 'test/test_mtu.py')
| -rw-r--r-- | test/test_mtu.py | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/test/test_mtu.py b/test/test_mtu.py index 568a147a9a4..c18ef9f4cdf 100644 --- a/test/test_mtu.py +++ b/test/test_mtu.py @@ -101,7 +101,6 @@ class TestMTU(VppTestCase): dst=self.pg0.remote_ip4, ttl=254, len=576, id=0) / p_icmp4 / p_ip4 / p_payload) - icmp4_reply[1].ttl -= 1 n = icmp4_reply.__class__(icmp4_reply) s = bytes(icmp4_reply) icmp4_reply = s[0:576] @@ -118,7 +117,7 @@ class TestMTU(VppTestCase): p4 = p_ether / p_ip4 / p_payload p4.flags = 0 p4_reply = p_ip4 / p_payload - p4_reply.ttl = 62 # check this + p4_reply.ttl = p_ip4.ttl - 1 p4_reply.flags = 0 p4_reply.id = 256 self.pg_enable_capture() |