diff options
author | Phil Sutter <phil@nwl.cc> | 2022-03-15 12:45:04 +0100 |
---|---|---|
committer | Phil Sutter <phil@nwl.cc> | 2022-03-15 14:53:52 +0100 |
commit | c293e116fc6130baae9ed82ca122b69ee6ff8390 (patch) | |
tree | 97ec9cc53ced82179bd31a61451c319ad826df0d /iptables/xtables-save.c | |
parent | 0836524f093c0fd9c39604a46a949e43d9b47ef2 (diff) |
nft: Review static extension loading
Combine the init_extensions() call common to all families, do not load
IPv6 extensions for iptables and vice versa, drop the outdated comment
about "same table".
Signed-off-by: Phil Sutter <phil@nwl.cc>
Diffstat (limited to 'iptables/xtables-save.c')
-rw-r--r-- | iptables/xtables-save.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/iptables/xtables-save.c b/iptables/xtables-save.c index 3b6b7e25..5a82cac5 100644 --- a/iptables/xtables-save.c +++ b/iptables/xtables-save.c @@ -199,16 +199,17 @@ xtables_save_main(int family, int argc, char *argv[], exit(1); } + init_extensions(); switch (family) { case NFPROTO_IPV4: - case NFPROTO_IPV6: /* fallthough, same table */ - init_extensions(); init_extensions4(); + d.commit = true; + break; + case NFPROTO_IPV6: init_extensions6(); d.commit = true; break; case NFPROTO_ARP: - init_extensions(); init_extensionsa(); break; case NFPROTO_BRIDGE: { @@ -220,7 +221,6 @@ xtables_save_main(int family, int argc, char *argv[], d.format &= ~FMT_NOCOUNTS; d.format |= FMT_C_COUNTS | FMT_EBT_SAVE; } - init_extensions(); init_extensionsb(); break; } |