netfilter-devel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH nft] optimize: payload expression requires inner_desc comparison
@ 2023-01-04 10:32 Pablo Neira Ayuso
  0 siblings, 0 replies; only message in thread
From: Pablo Neira Ayuso @ 2023-01-04 10:32 UTC (permalink / raw)
  To: netfilter-devel

Since 772892a018b4 ("src: add vxlan matching support"), payload
expressions have a inner_desc field that provides the description for
the outer tunnel header.

When searching for common mergeable selectors, compare the inner
description too.

Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
---
 src/optimize.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/src/optimize.c b/src/optimize.c
index 09013efc548c..32aed866eb49 100644
--- a/src/optimize.c
+++ b/src/optimize.c
@@ -46,6 +46,8 @@ static bool __expr_cmp(const struct expr *expr_a, const struct expr *expr_b)
 			return false;
 		if (expr_a->payload.desc != expr_b->payload.desc)
 			return false;
+		if (expr_a->payload.inner_desc != expr_b->payload.inner_desc)
+			return false;
 		if (expr_a->payload.tmpl != expr_b->payload.tmpl)
 			return false;
 		break;
-- 
2.30.2


^ permalink raw reply related	[flat|nested] only message in thread

only message in thread, other threads:[~2023-01-04 10:32 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-01-04 10:32 [PATCH nft] optimize: payload expression requires inner_desc comparison Pablo Neira Ayuso

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).