18074eb
From 617564b04485223a313164cc6b6c7febad23f8ad Mon Sep 17 00:00:00 2001
18074eb
From: Phil Sutter <phil@nwl.cc>
18074eb
Date: Tue, 4 May 2021 16:03:24 +0200
18074eb
Subject: [PATCH] extensions: sctp: Fix nftables translation
18074eb
18074eb
If both sport and dport was present, incorrect nft syntax was generated.
18074eb
18074eb
Fixes: defc7bd2bac89 ("extensions: libxt_sctp: Add translation to nft")
18074eb
Signed-off-by: Phil Sutter <phil@nwl.cc>
18074eb
(cherry picked from commit a61282ec6a1697bfb40f19d13a28a74559050167)
18074eb
---
18074eb
 extensions/libxt_sctp.c      | 10 ++++------
18074eb
 extensions/libxt_sctp.txlate | 10 +++++-----
18074eb
 2 files changed, 9 insertions(+), 11 deletions(-)
18074eb
18074eb
diff --git a/extensions/libxt_sctp.c b/extensions/libxt_sctp.c
18074eb
index 140de2653b1ef..ee4e99ebf11bf 100644
18074eb
--- a/extensions/libxt_sctp.c
18074eb
+++ b/extensions/libxt_sctp.c
18074eb
@@ -495,15 +495,13 @@ static int sctp_xlate(struct xt_xlate *xl,
18074eb
 	if (!einfo->flags)
18074eb
 		return 0;
18074eb
 
18074eb
-	xt_xlate_add(xl, "sctp ");
18074eb
-
18074eb
 	if (einfo->flags & XT_SCTP_SRC_PORTS) {
18074eb
 		if (einfo->spts[0] != einfo->spts[1])
18074eb
-			xt_xlate_add(xl, "sport%s %u-%u",
18074eb
+			xt_xlate_add(xl, "sctp sport%s %u-%u",
18074eb
 				     einfo->invflags & XT_SCTP_SRC_PORTS ? " !=" : "",
18074eb
 				     einfo->spts[0], einfo->spts[1]);
18074eb
 		else
18074eb
-			xt_xlate_add(xl, "sport%s %u",
18074eb
+			xt_xlate_add(xl, "sctp sport%s %u",
18074eb
 				     einfo->invflags & XT_SCTP_SRC_PORTS ? " !=" : "",
18074eb
 				     einfo->spts[0]);
18074eb
 		space = " ";
18074eb
@@ -511,11 +509,11 @@ static int sctp_xlate(struct xt_xlate *xl,
18074eb
 
18074eb
 	if (einfo->flags & XT_SCTP_DEST_PORTS) {
18074eb
 		if (einfo->dpts[0] != einfo->dpts[1])
18074eb
-			xt_xlate_add(xl, "%sdport%s %u-%u", space,
18074eb
+			xt_xlate_add(xl, "%ssctp dport%s %u-%u", space,
18074eb
 				     einfo->invflags & XT_SCTP_DEST_PORTS ? " !=" : "",
18074eb
 				     einfo->dpts[0], einfo->dpts[1]);
18074eb
 		else
18074eb
-			xt_xlate_add(xl, "%sdport%s %u", space,
18074eb
+			xt_xlate_add(xl, "%ssctp dport%s %u", space,
18074eb
 				     einfo->invflags & XT_SCTP_DEST_PORTS ? " !=" : "",
18074eb
 				     einfo->dpts[0]);
18074eb
 	}
18074eb
diff --git a/extensions/libxt_sctp.txlate b/extensions/libxt_sctp.txlate
18074eb
index 72f4641ab021c..0d6c59e183675 100644
18074eb
--- a/extensions/libxt_sctp.txlate
18074eb
+++ b/extensions/libxt_sctp.txlate
18074eb
@@ -23,16 +23,16 @@ iptables-translate -A INPUT -p sctp ! --dport 50:56 -j ACCEPT
18074eb
 nft add rule ip filter INPUT sctp dport != 50-56 counter accept
18074eb
 
18074eb
 iptables-translate -A INPUT -p sctp --dport 80 --sport 50 -j ACCEPT
18074eb
-nft add rule ip filter INPUT sctp sport 50 dport 80 counter accept
18074eb
+nft add rule ip filter INPUT sctp sport 50 sctp dport 80 counter accept
18074eb
 
18074eb
 iptables-translate -A INPUT -p sctp --dport 80:100 --sport 50 -j ACCEPT
18074eb
-nft add rule ip filter INPUT sctp sport 50 dport 80-100 counter accept
18074eb
+nft add rule ip filter INPUT sctp sport 50 sctp dport 80-100 counter accept
18074eb
 
18074eb
 iptables-translate -A INPUT -p sctp --dport 80 --sport 50:55 -j ACCEPT
18074eb
-nft add rule ip filter INPUT sctp sport 50-55 dport 80 counter accept
18074eb
+nft add rule ip filter INPUT sctp sport 50-55 sctp dport 80 counter accept
18074eb
 
18074eb
 iptables-translate -A INPUT -p sctp ! --dport 80:100 --sport 50 -j ACCEPT
18074eb
-nft add rule ip filter INPUT sctp sport 50 dport != 80-100 counter accept
18074eb
+nft add rule ip filter INPUT sctp sport 50 sctp dport != 80-100 counter accept
18074eb
 
18074eb
 iptables-translate -A INPUT -p sctp --dport 80 ! --sport 50:55 -j ACCEPT
18074eb
-nft add rule ip filter INPUT sctp sport != 50-55 dport 80 counter accept
18074eb
+nft add rule ip filter INPUT sctp sport != 50-55 sctp dport 80 counter accept
18074eb
-- 
18074eb
2.31.1
18074eb