Hi, Two more fixes for the -o/--optimize infrastructure, reported by users after the release: 1) do not hit assert() when concatenation already exists in the ruleset. 2) do not merge rules unless they contain at least one mergeable statement. Both patches come with tests to illustrate the issues. Pablo Neira Ayuso (2): optimize: merging concatenation is unsupported optimize: check for mergeable rules src/optimize.c | 32 ++++++++++++++++++- .../dumps/merge_stmts_concat.nft | 1 + .../optimizations/dumps/not_mergeable.nft | 12 +++++++ .../optimizations/merge_stmts_concat | 1 + .../testcases/optimizations/not_mergeable | 16 ++++++++++ 5 files changed, 61 insertions(+), 1 deletion(-) create mode 100644 tests/shell/testcases/optimizations/dumps/not_mergeable.nft create mode 100755 tests/shell/testcases/optimizations/not_mergeable -- 2.30.2
