|
 |
acfc56 |
From 9d67918643e7d17c433e82eb6cdb039cb103c50f Mon Sep 17 00:00:00 2001
|
|
|
acfc56 |
From: Phil Sutter <psutter@redhat.com>
|
|
|
acfc56 |
Date: Mon, 7 Dec 2020 18:26:24 +0100
|
|
|
acfc56 |
Subject: [PATCH] mergesort: unbreak listing with binops
|
|
|
acfc56 |
|
|
|
acfc56 |
Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1891790
|
|
|
acfc56 |
Upstream Status: nftables commit 3926a3369bb5a
|
|
|
acfc56 |
|
|
|
acfc56 |
commit 3926a3369bb5ada5c0706dadcbcf938517822a35
|
|
|
acfc56 |
Author: Pablo Neira Ayuso <pablo@netfilter.org>
|
|
|
acfc56 |
Date: Thu Aug 20 01:05:04 2020 +0200
|
|
|
acfc56 |
|
|
|
acfc56 |
mergesort: unbreak listing with binops
|
|
|
acfc56 |
|
|
|
acfc56 |
tcp flags == {syn, syn|ack}
|
|
|
acfc56 |
tcp flags & (fin|syn|rst|psh|ack|urg) == {ack, psh|ack, fin, fin|psh|ack}
|
|
|
acfc56 |
|
|
|
acfc56 |
results in:
|
|
|
acfc56 |
|
|
|
acfc56 |
BUG: Unknown expression binop
|
|
|
acfc56 |
nft: mergesort.c:47: expr_msort_cmp: Assertion `0' failed.
|
|
|
acfc56 |
Aborted (core dumped)
|
|
|
acfc56 |
|
|
|
acfc56 |
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
|
|
|
acfc56 |
---
|
|
|
acfc56 |
src/mergesort.c | 2 ++
|
|
|
acfc56 |
tests/py/inet/tcp.t | 2 ++
|
|
|
acfc56 |
tests/py/inet/tcp.t.payload | 21 +++++++++++++++++++++
|
|
|
acfc56 |
3 files changed, 25 insertions(+)
|
|
|
acfc56 |
|
|
|
acfc56 |
diff --git a/src/mergesort.c b/src/mergesort.c
|
|
|
acfc56 |
index 649b780..02094b4 100644
|
|
|
acfc56 |
--- a/src/mergesort.c
|
|
|
acfc56 |
+++ b/src/mergesort.c
|
|
|
acfc56 |
@@ -43,6 +43,8 @@ static int expr_msort_cmp(const struct expr *e1, const struct expr *e2)
|
|
|
acfc56 |
return concat_expr_msort_cmp(e1, e2);
|
|
|
acfc56 |
case EXPR_MAPPING:
|
|
|
acfc56 |
return expr_msort_cmp(e1->left, e2->left);
|
|
|
acfc56 |
+ case EXPR_BINOP:
|
|
|
acfc56 |
+ return expr_msort_cmp(e1->left, e2->left);
|
|
|
acfc56 |
default:
|
|
|
acfc56 |
BUG("Unknown expression %s\n", expr_name(e1));
|
|
|
acfc56 |
}
|
|
|
acfc56 |
diff --git a/tests/py/inet/tcp.t b/tests/py/inet/tcp.t
|
|
|
acfc56 |
index e0a83e2..29f06f5 100644
|
|
|
acfc56 |
--- a/tests/py/inet/tcp.t
|
|
|
acfc56 |
+++ b/tests/py/inet/tcp.t
|
|
|
acfc56 |
@@ -79,6 +79,8 @@ tcp flags != cwr;ok
|
|
|
acfc56 |
tcp flags == syn;ok
|
|
|
acfc56 |
tcp flags & (syn|fin) == (syn|fin);ok;tcp flags & (fin | syn) == fin | syn
|
|
|
acfc56 |
tcp flags & (fin | syn | rst | psh | ack | urg | ecn | cwr) == fin | syn | rst | psh | ack | urg | ecn | cwr;ok;tcp flags == 0xff
|
|
|
acfc56 |
+tcp flags { syn, syn | ack };ok
|
|
|
acfc56 |
+tcp flags & (fin | syn | rst | psh | ack | urg) == { fin, ack, psh | ack, fin | psh | ack };ok
|
|
|
acfc56 |
|
|
|
acfc56 |
tcp window 22222;ok
|
|
|
acfc56 |
tcp window 22;ok
|
|
|
acfc56 |
diff --git a/tests/py/inet/tcp.t.payload b/tests/py/inet/tcp.t.payload
|
|
|
acfc56 |
index 55f1bc2..076e562 100644
|
|
|
acfc56 |
--- a/tests/py/inet/tcp.t.payload
|
|
|
acfc56 |
+++ b/tests/py/inet/tcp.t.payload
|
|
|
acfc56 |
@@ -680,3 +680,24 @@ inet test-inet input
|
|
|
acfc56 |
[ bitwise reg 1 = (reg=1 & 0x000000f0 ) ^ 0x00000000 ]
|
|
|
acfc56 |
[ cmp eq reg 1 0x00000080 ]
|
|
|
acfc56 |
|
|
|
acfc56 |
+# tcp flags & (fin | syn | rst | psh | ack | urg) == { fin, ack, psh | ack, fin | psh | ack }
|
|
|
acfc56 |
+__set%d test-inet 3
|
|
|
acfc56 |
+__set%d test-inet 0
|
|
|
acfc56 |
+ element 00000001 : 0 [end] element 00000010 : 0 [end] element 00000018 : 0 [end] element 00000019 : 0 [end]
|
|
|
acfc56 |
+ip
|
|
|
acfc56 |
+ [ meta load l4proto => reg 1 ]
|
|
|
acfc56 |
+ [ cmp eq reg 1 0x00000006 ]
|
|
|
acfc56 |
+ [ payload load 1b @ transport header + 13 => reg 1 ]
|
|
|
acfc56 |
+ [ bitwise reg 1 = (reg=1 & 0x0000003f ) ^ 0x00000000 ]
|
|
|
acfc56 |
+ [ lookup reg 1 set __set%d ]
|
|
|
acfc56 |
+
|
|
|
acfc56 |
+# tcp flags { syn, syn | ack }
|
|
|
acfc56 |
+__set%d test-inet 3
|
|
|
acfc56 |
+__set%d test-inet 0
|
|
|
acfc56 |
+ element 00000002 : 0 [end] element 00000012 : 0 [end]
|
|
|
acfc56 |
+inet
|
|
|
acfc56 |
+ [ meta load l4proto => reg 1 ]
|
|
|
acfc56 |
+ [ cmp eq reg 1 0x00000006 ]
|
|
|
acfc56 |
+ [ payload load 1b @ transport header + 13 => reg 1 ]
|
|
|
acfc56 |
+ [ lookup reg 1 set __set%d ]
|
|
|
acfc56 |
+
|
|
|
acfc56 |
--
|
|
|
acfc56 |
1.8.3.1
|
|
|
acfc56 |
|