|
 |
cf8614 |
From f4476f9428a79c5d6d8fe284f0da91c2d4177e66 Mon Sep 17 00:00:00 2001
|
|
|
cf8614 |
From: Phil Sutter <psutter@redhat.com>
|
|
|
cf8614 |
Date: Mon, 12 Jul 2021 17:44:08 +0200
|
|
|
cf8614 |
Subject: [PATCH] tcpopt: rename noop to nop
|
|
|
cf8614 |
|
|
|
cf8614 |
Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1979334
|
|
|
cf8614 |
Upstream Status: nftables commit 8f591eba561ac
|
|
|
cf8614 |
|
|
|
cf8614 |
commit 8f591eba561aceeef605283c693b659a708d1cd3
|
|
|
cf8614 |
Author: Florian Westphal <fw@strlen.de>
|
|
|
cf8614 |
Date: Mon Nov 2 14:58:41 2020 +0100
|
|
|
cf8614 |
|
|
|
cf8614 |
tcpopt: rename noop to nop
|
|
|
cf8614 |
|
|
|
cf8614 |
'nop' is the tcp padding "option". "noop" is retained for compatibility
|
|
|
cf8614 |
on parser side.
|
|
|
cf8614 |
|
|
|
cf8614 |
Signed-off-by: Florian Westphal <fw@strlen.de>
|
|
|
cf8614 |
---
|
|
|
cf8614 |
doc/payload-expression.txt | 4 ++--
|
|
|
cf8614 |
src/tcpopt.c | 2 +-
|
|
|
cf8614 |
tests/py/any/tcpopt.t | 2 +-
|
|
|
cf8614 |
tests/py/any/tcpopt.t.json | 4 ++--
|
|
|
cf8614 |
tests/py/any/tcpopt.t.payload | 16 +---------------
|
|
|
cf8614 |
5 files changed, 7 insertions(+), 21 deletions(-)
|
|
|
cf8614 |
|
|
|
cf8614 |
diff --git a/doc/payload-expression.txt b/doc/payload-expression.txt
|
|
|
cf8614 |
index 27145c3..3a07321 100644
|
|
|
cf8614 |
--- a/doc/payload-expression.txt
|
|
|
cf8614 |
+++ b/doc/payload-expression.txt
|
|
|
cf8614 |
@@ -559,8 +559,8 @@ Segment Routing Header
|
|
|
cf8614 |
|eol|
|
|
|
cf8614 |
End if option list|
|
|
|
cf8614 |
kind
|
|
|
cf8614 |
-|noop|
|
|
|
cf8614 |
-1 Byte TCP No-op options |
|
|
|
cf8614 |
+|nop|
|
|
|
cf8614 |
+1 Byte TCP Nop padding option |
|
|
|
cf8614 |
kind
|
|
|
cf8614 |
|maxseg|
|
|
|
cf8614 |
TCP Maximum Segment Size|
|
|
|
cf8614 |
diff --git a/src/tcpopt.c b/src/tcpopt.c
|
|
|
cf8614 |
index 8d5bdec..17cb580 100644
|
|
|
cf8614 |
--- a/src/tcpopt.c
|
|
|
cf8614 |
+++ b/src/tcpopt.c
|
|
|
cf8614 |
@@ -27,7 +27,7 @@ static const struct exthdr_desc tcpopt_eol = {
|
|
|
cf8614 |
};
|
|
|
cf8614 |
|
|
|
cf8614 |
static const struct exthdr_desc tcpopt_nop = {
|
|
|
cf8614 |
- .name = "noop",
|
|
|
cf8614 |
+ .name = "nop",
|
|
|
cf8614 |
.type = TCPOPT_KIND_NOP,
|
|
|
cf8614 |
.templates = {
|
|
|
cf8614 |
[TCPOPTHDR_FIELD_KIND] = PHT("kind", 0, 8),
|
|
|
cf8614 |
diff --git a/tests/py/any/tcpopt.t b/tests/py/any/tcpopt.t
|
|
|
cf8614 |
index 5f21d49..1d42de8 100644
|
|
|
cf8614 |
--- a/tests/py/any/tcpopt.t
|
|
|
cf8614 |
+++ b/tests/py/any/tcpopt.t
|
|
|
cf8614 |
@@ -5,7 +5,7 @@
|
|
|
cf8614 |
*inet;test-inet;input
|
|
|
cf8614 |
|
|
|
cf8614 |
tcp option eol kind 1;ok
|
|
|
cf8614 |
-tcp option noop kind 1;ok
|
|
|
cf8614 |
+tcp option nop kind 1;ok
|
|
|
cf8614 |
tcp option maxseg kind 1;ok
|
|
|
cf8614 |
tcp option maxseg length 1;ok
|
|
|
cf8614 |
tcp option maxseg size 1;ok
|
|
|
cf8614 |
diff --git a/tests/py/any/tcpopt.t.json b/tests/py/any/tcpopt.t.json
|
|
|
cf8614 |
index 2c6236a..b15e36e 100644
|
|
|
cf8614 |
--- a/tests/py/any/tcpopt.t.json
|
|
|
cf8614 |
+++ b/tests/py/any/tcpopt.t.json
|
|
|
cf8614 |
@@ -14,14 +14,14 @@
|
|
|
cf8614 |
}
|
|
|
cf8614 |
]
|
|
|
cf8614 |
|
|
|
cf8614 |
-# tcp option noop kind 1
|
|
|
cf8614 |
+# tcp option nop kind 1
|
|
|
cf8614 |
[
|
|
|
cf8614 |
{
|
|
|
cf8614 |
"match": {
|
|
|
cf8614 |
"left": {
|
|
|
cf8614 |
"tcp option": {
|
|
|
cf8614 |
"field": "kind",
|
|
|
cf8614 |
- "name": "noop"
|
|
|
cf8614 |
+ "name": "nop"
|
|
|
cf8614 |
}
|
|
|
cf8614 |
},
|
|
|
cf8614 |
"op": "==",
|
|
|
cf8614 |
diff --git a/tests/py/any/tcpopt.t.payload b/tests/py/any/tcpopt.t.payload
|
|
|
cf8614 |
index f63076a..9c480c8 100644
|
|
|
cf8614 |
--- a/tests/py/any/tcpopt.t.payload
|
|
|
cf8614 |
+++ b/tests/py/any/tcpopt.t.payload
|
|
|
cf8614 |
@@ -19,21 +19,7 @@ inet
|
|
|
cf8614 |
[ exthdr load tcpopt 1b @ 0 + 0 => reg 1 ]
|
|
|
cf8614 |
[ cmp eq reg 1 0x00000001 ]
|
|
|
cf8614 |
|
|
|
cf8614 |
-# tcp option noop kind 1
|
|
|
cf8614 |
-ip
|
|
|
cf8614 |
- [ meta load l4proto => reg 1 ]
|
|
|
cf8614 |
- [ cmp eq reg 1 0x00000006 ]
|
|
|
cf8614 |
- [ exthdr load tcpopt 1b @ 1 + 0 => reg 1 ]
|
|
|
cf8614 |
- [ cmp eq reg 1 0x00000001 ]
|
|
|
cf8614 |
-
|
|
|
cf8614 |
-# tcp option noop kind 1
|
|
|
cf8614 |
-ip6
|
|
|
cf8614 |
- [ meta load l4proto => reg 1 ]
|
|
|
cf8614 |
- [ cmp eq reg 1 0x00000006 ]
|
|
|
cf8614 |
- [ exthdr load tcpopt 1b @ 1 + 0 => reg 1 ]
|
|
|
cf8614 |
- [ cmp eq reg 1 0x00000001 ]
|
|
|
cf8614 |
-
|
|
|
cf8614 |
-# tcp option noop kind 1
|
|
|
cf8614 |
+# tcp option nop kind 1
|
|
|
cf8614 |
inet
|
|
|
cf8614 |
[ meta load l4proto => reg 1 ]
|
|
|
cf8614 |
[ cmp eq reg 1 0x00000006 ]
|
|
|
cf8614 |
--
|
|
|
cf8614 |
2.31.1
|
|
|
cf8614 |
|