|
 |
acfc56 |
From ab62f33df5ef33f6eff8d88d9475a01822a2f625 Mon Sep 17 00:00:00 2001
|
|
|
acfc56 |
From: Phil Sutter <psutter@redhat.com>
|
|
|
acfc56 |
Date: Tue, 30 Jun 2020 16:20:22 +0200
|
|
|
acfc56 |
Subject: [PATCH] segtree: Fix missing expires value in prefixes
|
|
|
acfc56 |
|
|
|
acfc56 |
Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1832235
|
|
|
acfc56 |
Upstream Status: nftables commit 60ba9c22fecc0
|
|
|
acfc56 |
|
|
|
acfc56 |
commit 60ba9c22fecc0ca9bb2a61f6ad39bceed1aee38f
|
|
|
acfc56 |
Author: Phil Sutter <phil@nwl.cc>
|
|
|
acfc56 |
Date: Tue Apr 28 20:54:03 2020 +0200
|
|
|
acfc56 |
|
|
|
acfc56 |
segtree: Fix missing expires value in prefixes
|
|
|
acfc56 |
|
|
|
acfc56 |
This probable copy'n'paste bug prevented 'expiration' field from being
|
|
|
acfc56 |
populated when turning a range into a prefix in
|
|
|
acfc56 |
interval_map_decompose(). Consequently, interval sets with timeout did
|
|
|
acfc56 |
print expiry value for ranges (such as 10.0.0.1-10.0.0.5) but not
|
|
|
acfc56 |
prefixes (10.0.0.0/8, for instance).
|
|
|
acfc56 |
|
|
|
acfc56 |
Fixes: bb0e6d8a2851b ("segtree: incorrect handling of comments and timeouts with mapping")
|
|
|
acfc56 |
Signed-off-by: Phil Sutter <phil@nwl.cc>
|
|
|
acfc56 |
---
|
|
|
acfc56 |
src/segtree.c | 2 +-
|
|
|
acfc56 |
1 file changed, 1 insertion(+), 1 deletion(-)
|
|
|
acfc56 |
|
|
|
acfc56 |
diff --git a/src/segtree.c b/src/segtree.c
|
|
|
acfc56 |
index e859f84..1ba4363 100644
|
|
|
acfc56 |
--- a/src/segtree.c
|
|
|
acfc56 |
+++ b/src/segtree.c
|
|
|
acfc56 |
@@ -1086,7 +1086,7 @@ void interval_map_decompose(struct expr *set)
|
|
|
acfc56 |
prefix->comment = xstrdup(low->comment);
|
|
|
acfc56 |
if (low->timeout)
|
|
|
acfc56 |
prefix->timeout = low->timeout;
|
|
|
acfc56 |
- if (low->left->expiration)
|
|
|
acfc56 |
+ if (low->expiration)
|
|
|
acfc56 |
prefix->expiration = low->expiration;
|
|
|
acfc56 |
}
|
|
|
acfc56 |
|
|
|
acfc56 |
--
|
|
|
acfc56 |
1.8.3.1
|
|
|
acfc56 |
|