The evaluation step already updates the cache for each command in this batch. There is no need to update the cache again from the echo path, otherwise the cache is populated twice with the same object. Fixes: b99c4d072d99 ("Implement --echo option") Signed-off-by: Pablo Neira Ayuso <pablo@xxxxxxxxxxxxx> --- This fixes a crash when combining -f and -e, the follow up patch introduces a test for such combination. src/monitor.c | 1 - 1 file changed, 1 deletion(-) diff --git a/src/monitor.c b/src/monitor.c index a68d960bfd4e..5b25c9d4854e 100644 --- a/src/monitor.c +++ b/src/monitor.c @@ -900,7 +900,6 @@ int netlink_echo_callback(const struct nlmsghdr *nlh, void *data) .ctx = ctx, .loc = &netlink_location, .monitor_flags = 0xffffffff, - .cache_needed = true, }; if (!nft_output_echo(&echo_monh.ctx->nft->output)) -- 2.11.0