Blame SOURCES/0051-tcpopt-rename-noop-to-nop.patch

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