]> git.ipfire.org Git - thirdparty/nftables.git/commitdiff
src: drop obsolete hook argument form hook dump functions
authorFlorian Westphal <fw@strlen.de>
Wed, 31 Jul 2024 16:51:03 +0000 (18:51 +0200)
committerFlorian Westphal <fw@strlen.de>
Mon, 19 Aug 2024 12:00:36 +0000 (14:00 +0200)
since commit b98fee20bfe2 ("mnl: revisit hook listing"), handle.chain is
never set in this path, so 'hook' is always set to -1, so the hook arg
can be dropped.

Signed-off-by: Florian Westphal <fw@strlen.de>
include/mnl.h
src/mnl.c
src/rule.c

index cd5a2053b1661f571d41a0f44cbbbcb4fe4bde06..c9502f328f1cb5e789cd4cc3d5004109248970e7 100644 (file)
@@ -90,7 +90,7 @@ int mnl_nft_flowtable_add(struct netlink_ctx *ctx, struct cmd *cmd,
                          unsigned int flags);
 int mnl_nft_flowtable_del(struct netlink_ctx *ctx, struct cmd *cmd);
 
-int mnl_nft_dump_nf_hooks(struct netlink_ctx *ctx, int family, int hook,
+int mnl_nft_dump_nf_hooks(struct netlink_ctx *ctx, int family,
                          const char *devname);
 
 int mnl_nft_event_listener(struct mnl_socket *nf_sock, unsigned int debug_mask,
index 88475ef4c25ea5181acbfb5cf095cc75cd80782d..1b424e427124eb2e7ef59a33469f38225a0f1db4 100644 (file)
--- a/src/mnl.c
+++ b/src/mnl.c
@@ -2499,7 +2499,7 @@ static void print_hooks(struct netlink_ctx *ctx, int family, struct list_head *h
        fprintf(fp, "}\n");
 }
 
-static int mnl_nft_dump_nf(struct netlink_ctx *ctx, int family, int hook,
+static int mnl_nft_dump_nf(struct netlink_ctx *ctx, int family,
                           const char *devname, struct list_head *hook_list)
 {
        int i, err;
@@ -2515,7 +2515,7 @@ static int mnl_nft_dump_nf(struct netlink_ctx *ctx, int family, int hook,
        return err;
 }
 
-static int mnl_nft_dump_nf_arp(struct netlink_ctx *ctx, int family, int hook,
+static int mnl_nft_dump_nf_arp(struct netlink_ctx *ctx, int family,
                               const char *devname, struct list_head *hook_list)
 {
        int err1, err2;
@@ -2526,7 +2526,7 @@ static int mnl_nft_dump_nf_arp(struct netlink_ctx *ctx, int family, int hook,
        return err1 ? err2 : err1;
 }
 
-static int mnl_nft_dump_nf_netdev(struct netlink_ctx *ctx, int family, int hook,
+static int mnl_nft_dump_nf_netdev(struct netlink_ctx *ctx, int family,
                                  const char *devname, struct list_head *hook_list)
 {
        int err;
@@ -2550,7 +2550,7 @@ static void warn_if_device(struct nft_ctx *nft, const char *devname)
                nft_print(&nft->output, "# device keyword (%s) unexpected for this family\n", devname);
 }
 
-int mnl_nft_dump_nf_hooks(struct netlink_ctx *ctx, int family, int hook, const char *devname)
+int mnl_nft_dump_nf_hooks(struct netlink_ctx *ctx, int family, const char *devname)
 {
        LIST_HEAD(hook_list);
        int ret = -1, tmp;
@@ -2559,16 +2559,16 @@ int mnl_nft_dump_nf_hooks(struct netlink_ctx *ctx, int family, int hook, const c
 
        switch (family) {
        case NFPROTO_UNSPEC:
-               ret = mnl_nft_dump_nf_hooks(ctx, NFPROTO_ARP, hook, NULL);
-               tmp = mnl_nft_dump_nf_hooks(ctx, NFPROTO_INET, hook, NULL);
+               ret = mnl_nft_dump_nf_hooks(ctx, NFPROTO_ARP, NULL);
+               tmp = mnl_nft_dump_nf_hooks(ctx, NFPROTO_INET, NULL);
                if (tmp == 0)
                        ret = 0;
-               tmp = mnl_nft_dump_nf_hooks(ctx, NFPROTO_BRIDGE, hook, NULL);
+               tmp = mnl_nft_dump_nf_hooks(ctx, NFPROTO_BRIDGE, NULL);
                if (tmp == 0)
                        ret = 0;
 
                if (devname) {
-                       tmp = mnl_nft_dump_nf_hooks(ctx, NFPROTO_NETDEV, hook, devname);
+                       tmp = mnl_nft_dump_nf_hooks(ctx, NFPROTO_NETDEV, devname);
                        if (tmp == 0)
                                ret = 0;
                }
@@ -2579,10 +2579,10 @@ int mnl_nft_dump_nf_hooks(struct netlink_ctx *ctx, int family, int hook, const c
                if (devname)
                        ret = __mnl_nft_dump_nf_hooks(ctx, family, NFPROTO_NETDEV,
                                                      NF_NETDEV_INGRESS, devname, &hook_list);
-               tmp = mnl_nft_dump_nf_hooks(ctx, NFPROTO_IPV4, hook, NULL);
+               tmp = mnl_nft_dump_nf_hooks(ctx, NFPROTO_IPV4, NULL);
                if (tmp == 0)
                        ret = 0;
-               tmp = mnl_nft_dump_nf_hooks(ctx, NFPROTO_IPV6, hook, NULL);
+               tmp = mnl_nft_dump_nf_hooks(ctx, NFPROTO_IPV6, NULL);
                if (tmp == 0)
                        ret = 0;
 
@@ -2591,14 +2591,14 @@ int mnl_nft_dump_nf_hooks(struct netlink_ctx *ctx, int family, int hook, const c
        case NFPROTO_IPV6:
        case NFPROTO_BRIDGE:
                warn_if_device(ctx->nft, devname);
-               ret = mnl_nft_dump_nf(ctx, family, hook, devname, &hook_list);
+               ret = mnl_nft_dump_nf(ctx, family, devname, &hook_list);
                break;
        case NFPROTO_ARP:
                warn_if_device(ctx->nft, devname);
-               ret = mnl_nft_dump_nf_arp(ctx, family, hook, devname, &hook_list);
+               ret = mnl_nft_dump_nf_arp(ctx, family, devname, &hook_list);
                break;
        case NFPROTO_NETDEV:
-               ret = mnl_nft_dump_nf_netdev(ctx, family, hook, devname, &hook_list);
+               ret = mnl_nft_dump_nf_netdev(ctx, family, devname, &hook_list);
                break;
        }
 
index 545f9b2b54631120213159a3b1f2cb932c1adf21..0f92ef532ece7a6c8f847f8bb61ab3759b596413 100644 (file)
@@ -2341,12 +2341,8 @@ static int do_list_set(struct netlink_ctx *ctx, struct cmd *cmd,
 static int do_list_hooks(struct netlink_ctx *ctx, struct cmd *cmd)
 {
        const char *devname = cmd->handle.obj.name;
-       int hooknum = -1;
 
-       if (cmd->handle.chain.name)
-               hooknum = cmd->handle.chain_id;
-
-       return mnl_nft_dump_nf_hooks(ctx, cmd->handle.family, hooknum, devname);
+       return mnl_nft_dump_nf_hooks(ctx, cmd->handle.family, devname);
 }
 
 static int do_command_list(struct netlink_ctx *ctx, struct cmd *cmd)