tests: add test coverage for firewall includes
authorJo-Philipp Wich <jo@mein.io>
Wed, 15 Jun 2022 11:52:56 +0000 (13:52 +0200)
committerJo-Philipp Wich <jo@mein.io>
Wed, 15 Jun 2022 11:52:56 +0000 (13:52 +0200)
Signed-off-by: Jo-Philipp Wich <jo@mein.io>
tests/05_includes/01_nft_includes [new file with mode: 0644]
tests/05_includes/02_firewall.user_include [new file with mode: 0644]
tests/05_includes/03_script_includes [new file with mode: 0644]

diff --git a/tests/05_includes/01_nft_includes b/tests/05_includes/01_nft_includes
new file mode 100644 (file)
index 0000000..314a428
--- /dev/null
@@ -0,0 +1,266 @@
+Testing the correct placement of potential include positions.
+
+-- Testcase --
+{%
+       include("./root/usr/share/firewall4/main.uc", {
+               getenv: function(varname) {
+                       switch (varname) {
+                       case 'ACTION':
+                               return 'print';
+                       }
+               }
+       })
+%}
+-- End --
+
+-- File uci/helpers.json --
+{}
+-- End --
+
+-- File fs/open~_sys_class_net_eth0_flags.txt --
+0x1103
+-- End --
+
+-- File fs/open~_usr_share_nftables_d_include-ruleset-start_nft.txt --
+# dummy
+-- End --
+
+-- File fs/open~_usr_share_nftables_d_include-table-start_nft.txt --
+# dummy
+-- End --
+
+-- File fs/open~_usr_share_nftables_d_include-chain-start-forward_nft.txt --
+# dummy
+-- End --
+
+-- File fs/open~_usr_share_nftables_d_include-chain-end-forward_nft.txt --
+# dummy
+-- End --
+
+-- File fs/open~_usr_share_nftables_d_include-table-end-1_nft.txt --
+# dummy
+-- End --
+
+-- File fs/open~_usr_share_nftables_d_include-table-end-2_nft.txt --
+# dummy
+-- End --
+
+-- File fs/open~_usr_share_nftables_d_include-ruleset-end_nft.txt --
+# dummy
+-- End --
+
+-- File uci/firewall.json --
+{
+       "zone": [
+               {
+                       "name": "test",
+                       "device": [ "eth0" ],
+                       "auto_helper": 0
+               }
+       ],
+       "include": [
+               {
+                       ".description": "Position 'table-pre' (or 'table-prepend') will place an include before the first chain",
+                       "path": "/usr/share/nftables.d/include-table-start.nft",
+                       "type": "nftables",
+                       "position": "table-pre"
+               },
+
+               {
+                       ".description": "Position defaults to 'table-append', means after the last chain in the table scope",
+                       "path": "/usr/share/nftables.d/include-table-end-1.nft",
+                       "type": "nftables"
+               },
+
+               {
+                       ".description": "Position 'table-post' (or 'table-postpend') may be used as alias for 'table-append'",
+                       "path": "/usr/share/nftables.d/include-table-end-2.nft",
+                       "type": "nftables",
+                       "position": "table-post"
+               },
+
+               {
+                       ".description": "Position 'ruleset-pre' (or 'ruleset-prepend') will place an include before the table declaration",
+                       "path": "/usr/share/nftables.d/include-ruleset-start.nft",
+                       "type": "nftables",
+                       "position": "ruleset-pre"
+               },
+
+               {
+                       ".description": "Position 'ruleset-post' (or 'ruleset-append') will place an include after the table declaration",
+                       "path": "/usr/share/nftables.d/include-ruleset-end.nft",
+                       "type": "nftables",
+                       "position": "ruleset-post"
+               },
+
+               {
+                       ".description": "Position 'chain-pre' (or 'chain-prepend') will place an include at the top of a specified chain",
+                       "path": "/usr/share/nftables.d/include-chain-start-forward.nft",
+                       "type": "nftables",
+                       "position": "chain-pre",
+                       "chain": "forward"
+               },
+
+               {
+                       ".description": "Position 'chain-post' (or 'chain-append') will place an include at the bottom of a specified chain",
+                       "path": "/usr/share/nftables.d/include-chain-end-forward.nft",
+                       "type": "nftables",
+                       "position": "chain-post",
+                       "chain": "forward"
+               },
+
+               {
+                       ".description": "Position 'chain-pre' or 'chain-post' without chain option will yield and error",
+                       "path": "/usr/share/nftables.d/include-chain-end-forward.nft",
+                       "type": "nftables",
+                       "position": "chain-post"
+               },
+       ]
+}
+-- End --
+
+-- Expect stderr --
+[!] Section @include[7] must specify 'chain' for position chain-append, ignoring section
+-- End --
+
+-- Expect stdout --
+table inet fw4
+flush table inet fw4
+
+include "/usr/share/nftables.d/include-ruleset-start.nft"
+
+table inet fw4 {
+       #
+       # Defines
+       #
+
+       define test_devices = { "eth0" }
+       define test_subnets = {  }
+
+
+       #
+       # User includes
+       #
+
+       include "/etc/nftables.d/*.nft"
+
+       include "/usr/share/nftables.d/include-table-start.nft"
+
+
+       #
+       # Filter rules
+       #
+
+       chain input {
+               type filter hook input priority filter; policy drop;
+
+               iifname "lo" accept comment "!fw4: Accept traffic from loopback"
+
+               ct state established,related accept comment "!fw4: Allow inbound established and related flows"
+               iifname "eth0" jump input_test comment "!fw4: Handle test IPv4/IPv6 input traffic"
+       }
+
+       chain forward {
+               type filter hook forward priority filter; policy drop;
+
+               include "/usr/share/nftables.d/include-chain-start-forward.nft"
+               ct state established,related accept comment "!fw4: Allow forwarded established and related flows"
+               iifname "eth0" jump forward_test comment "!fw4: Handle test IPv4/IPv6 forward traffic"
+               include "/usr/share/nftables.d/include-chain-end-forward.nft"
+       }
+
+       chain output {
+               type filter hook output priority filter; policy drop;
+
+               oifname "lo" accept comment "!fw4: Accept traffic towards loopback"
+
+               ct state established,related accept comment "!fw4: Allow outbound established and related flows"
+               oifname "eth0" jump output_test comment "!fw4: Handle test IPv4/IPv6 output traffic"
+       }
+
+       chain prerouting {
+               type filter hook prerouting priority filter; policy accept;
+       }
+
+       chain handle_reject {
+               meta l4proto tcp reject with tcp reset comment "!fw4: Reject TCP traffic"
+               reject with icmpx type port-unreachable comment "!fw4: Reject any other traffic"
+       }
+
+       chain input_test {
+               jump drop_from_test
+       }
+
+       chain output_test {
+               jump drop_to_test
+       }
+
+       chain forward_test {
+               jump drop_to_test
+       }
+
+       chain drop_from_test {
+               iifname "eth0" counter drop comment "!fw4: drop test IPv4/IPv6 traffic"
+       }
+
+       chain drop_to_test {
+               oifname "eth0" counter drop comment "!fw4: drop test IPv4/IPv6 traffic"
+       }
+
+
+       #
+       # NAT rules
+       #
+
+       chain dstnat {
+               type nat hook prerouting priority dstnat; policy accept;
+       }
+
+       chain srcnat {
+               type nat hook postrouting priority srcnat; policy accept;
+       }
+
+
+       #
+       # Raw rules (notrack)
+       #
+
+       chain raw_prerouting {
+               type filter hook prerouting priority raw; policy accept;
+       }
+
+       chain raw_output {
+               type filter hook output priority raw; policy accept;
+       }
+
+
+       #
+       # Mangle rules
+       #
+
+       chain mangle_prerouting {
+               type filter hook prerouting priority mangle; policy accept;
+       }
+
+       chain mangle_postrouting {
+               type filter hook postrouting priority mangle; policy accept;
+       }
+
+       chain mangle_input {
+               type filter hook input priority mangle; policy accept;
+       }
+
+       chain mangle_output {
+               type route hook output priority mangle; policy accept;
+       }
+
+       chain mangle_forward {
+               type filter hook forward priority mangle; policy accept;
+       }
+
+       include "/usr/share/nftables.d/include-table-end-1.nft"
+       include "/usr/share/nftables.d/include-table-end-2.nft"
+}
+
+include "/usr/share/nftables.d/include-ruleset-end.nft"
+-- End --
diff --git a/tests/05_includes/02_firewall.user_include b/tests/05_includes/02_firewall.user_include
new file mode 100644 (file)
index 0000000..fa398ff
--- /dev/null
@@ -0,0 +1,196 @@
+Testing that /etc/firewall.user is treated specially and requires an extra
+option to be marked compatible.
+
+-- Testcase --
+{%
+       include("./root/usr/share/firewall4/main.uc", {
+               getenv: function(varname) {
+                       switch (varname) {
+                       case 'ACTION':
+                               return 'print';
+                       }
+               }
+       })
+%}
+-- End --
+
+-- File uci/helpers.json --
+{}
+-- End --
+
+-- File fs/open~_sys_class_net_eth0_flags.txt --
+0x1103
+-- End --
+
+-- File fs/open~_etc_firewall_user.txt --
+# dummy
+-- End --
+
+-- File fs/open~_usr_share_miniupnpd_firewall_include.txt --
+# dummy
+-- End --
+
+-- File uci/firewall.json --
+{
+       "zone": [
+               {
+                       "name": "test",
+                       "device": [ "eth0" ],
+                       "auto_helper": 0
+               }
+       ],
+       "include": [
+               {
+                       ".description": "By default, this /etc/firewall.user include should be skipped with a warning",
+                       "path": "/etc/firewall.user"
+               },
+
+               {
+                       ".description": "This /etc/firewall.user include should be added due to the compatible flag",
+                       "path": "/etc/firewall.user",
+                       "fw4_compatible": 1
+               },
+
+               {
+                       ".description": "An include of another path should not require a compatible flag",
+                       "path": "/usr/share/miniupnpd/firewall.include"
+               }
+       ]
+}
+-- End --
+
+-- Expect stderr --
+[!] Section @include[0] is not marked as compatible with fw4, ignoring section
+[!] Section @include[0] requires 'option fw4_compatible 1' to be considered compatible
+-- End --
+
+-- Expect stdout --
+table inet fw4
+flush table inet fw4
+
+table inet fw4 {
+       #
+       # Defines
+       #
+
+       define test_devices = { "eth0" }
+       define test_subnets = {  }
+
+
+       #
+       # User includes
+       #
+
+       include "/etc/nftables.d/*.nft"
+
+
+       #
+       # Filter rules
+       #
+
+       chain input {
+               type filter hook input priority filter; policy drop;
+
+               iifname "lo" accept comment "!fw4: Accept traffic from loopback"
+
+               ct state established,related accept comment "!fw4: Allow inbound established and related flows"
+               iifname "eth0" jump input_test comment "!fw4: Handle test IPv4/IPv6 input traffic"
+       }
+
+       chain forward {
+               type filter hook forward priority filter; policy drop;
+
+               ct state established,related accept comment "!fw4: Allow forwarded established and related flows"
+               iifname "eth0" jump forward_test comment "!fw4: Handle test IPv4/IPv6 forward traffic"
+       }
+
+       chain output {
+               type filter hook output priority filter; policy drop;
+
+               oifname "lo" accept comment "!fw4: Accept traffic towards loopback"
+
+               ct state established,related accept comment "!fw4: Allow outbound established and related flows"
+               oifname "eth0" jump output_test comment "!fw4: Handle test IPv4/IPv6 output traffic"
+       }
+
+       chain prerouting {
+               type filter hook prerouting priority filter; policy accept;
+       }
+
+       chain handle_reject {
+               meta l4proto tcp reject with tcp reset comment "!fw4: Reject TCP traffic"
+               reject with icmpx type port-unreachable comment "!fw4: Reject any other traffic"
+       }
+
+       chain input_test {
+               jump drop_from_test
+       }
+
+       chain output_test {
+               jump drop_to_test
+       }
+
+       chain forward_test {
+               jump drop_to_test
+       }
+
+       chain drop_from_test {
+               iifname "eth0" counter drop comment "!fw4: drop test IPv4/IPv6 traffic"
+       }
+
+       chain drop_to_test {
+               oifname "eth0" counter drop comment "!fw4: drop test IPv4/IPv6 traffic"
+       }
+
+
+       #
+       # NAT rules
+       #
+
+       chain dstnat {
+               type nat hook prerouting priority dstnat; policy accept;
+       }
+
+       chain srcnat {
+               type nat hook postrouting priority srcnat; policy accept;
+       }
+
+
+       #
+       # Raw rules (notrack)
+       #
+
+       chain raw_prerouting {
+               type filter hook prerouting priority raw; policy accept;
+       }
+
+       chain raw_output {
+               type filter hook output priority raw; policy accept;
+       }
+
+
+       #
+       # Mangle rules
+       #
+
+       chain mangle_prerouting {
+               type filter hook prerouting priority mangle; policy accept;
+       }
+
+       chain mangle_postrouting {
+               type filter hook postrouting priority mangle; policy accept;
+       }
+
+       chain mangle_input {
+               type filter hook input priority mangle; policy accept;
+       }
+
+       chain mangle_output {
+               type route hook output priority mangle; policy accept;
+       }
+
+       chain mangle_forward {
+               type filter hook forward priority mangle; policy accept;
+       }
+}
+-- End --
diff --git a/tests/05_includes/03_script_includes b/tests/05_includes/03_script_includes
new file mode 100644 (file)
index 0000000..3eeea31
--- /dev/null
@@ -0,0 +1,44 @@
+Testing the execution of include scripts.
+
+-- Testcase --
+{%
+       include("./root/usr/share/firewall4/main.uc", {
+               TRACE_CALLS: "stderr",
+
+               getenv: function(varname) {
+                       switch (varname) {
+                       case 'ACTION':
+                               return 'includes';
+                       }
+               }
+       })
+%}
+-- End --
+
+-- File fs/open~_var_run_fw4_state.txt --
+{
+       "includes": [
+               {
+                       "enabled": true,
+                       "path": "/usr/share/miniupnpd/firewall.include",
+                       "type": "script",
+                       "fw4_compatible": true,
+                       "position": "table-append"
+               },
+
+               {
+                       "enabled": true,
+                       "path": "/etc/example.sh",
+                       "type": "script",
+                       "fw4_compatible": true,
+                       "position": "table-append"
+               }
+       ]
+}
+-- End --
+
+-- Expect stderr --
+[call] fs.open path </var/run/fw4.state> mode <r>
+[call] system command <[ "sh", "-c", "exec 1000>&-; config() { echo \"You cannot use UCI in firewall in..." ]> timeout <30000>
+[call] system command <[ "sh", "-c", "exec 1000>&-; config() { echo \"You cannot use UCI in firewall in..." ]> timeout <30000>
+-- End --