netfilter: nf_tables: Allow chain name of up to 255 chars
authorPhil Sutter <phil@nwl.cc>
Thu, 27 Jul 2017 14:56:42 +0000 (16:56 +0200)
committerPablo Neira Ayuso <pablo@netfilter.org>
Mon, 31 Jul 2017 18:41:57 +0000 (20:41 +0200)
Same conversion as for table names, use NFT_NAME_MAXLEN as upper
boundary as well.

Signed-off-by: Phil Sutter <phil@nwl.cc>
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
include/net/netfilter/nf_tables.h
include/uapi/linux/netfilter/nf_tables.h
net/netfilter/nf_tables_api.c
net/netfilter/nf_tables_trace.c

index 05ecf78ec078740d5a590f545bfb23af3d935f76..be1610162ee02818dfa85033ceffbd1fca3d964b 100644 (file)
@@ -859,7 +859,7 @@ struct nft_chain {
        u16                             level;
        u8                              flags:6,
                                        genmask:2;
-       char                            name[NFT_CHAIN_MAXNAMELEN];
+       char                            *name;
 };
 
 enum nft_chain_type {
@@ -1272,7 +1272,7 @@ struct nft_trans_set {
 
 struct nft_trans_chain {
        bool                            update;
-       char                            name[NFT_CHAIN_MAXNAMELEN];
+       char                            *name;
        struct nft_stats __percpu       *stats;
        u8                              policy;
 };
index 0b94e572ef1665853a97f840d91e5684fbeffa89..d9c03a8608ee3a6dafa5ebbce931b886c623c246 100644 (file)
@@ -3,7 +3,7 @@
 
 #define NFT_NAME_MAXLEN                256
 #define NFT_TABLE_MAXNAMELEN   NFT_NAME_MAXLEN
-#define NFT_CHAIN_MAXNAMELEN   32
+#define NFT_CHAIN_MAXNAMELEN   NFT_NAME_MAXLEN
 #define NFT_SET_MAXNAMELEN     32
 #define NFT_OBJ_MAXNAMELEN     32
 #define NFT_USERDATA_MAXLEN    256
index c2e392d5e512faeb8cebf714ef078edd74b25970..747499039709e0c9c09eeccaebfb329f853871f5 100644 (file)
@@ -1250,8 +1250,10 @@ static void nf_tables_chain_destroy(struct nft_chain *chain)
                        static_branch_dec(&nft_counters_enabled);
                if (basechain->ops[0].dev != NULL)
                        dev_put(basechain->ops[0].dev);
+               kfree(chain->name);
                kfree(basechain);
        } else {
+               kfree(chain->name);
                kfree(chain);
        }
 }
@@ -1476,8 +1478,13 @@ static int nf_tables_newchain(struct net *net, struct sock *nlsk,
                        nft_trans_chain_policy(trans) = -1;
 
                if (nla[NFTA_CHAIN_HANDLE] && name) {
-                       nla_strlcpy(nft_trans_chain_name(trans), name,
-                                   NFT_CHAIN_MAXNAMELEN);
+                       nft_trans_chain_name(trans) =
+                               nla_strdup(name, GFP_KERNEL);
+                       if (!nft_trans_chain_name(trans)) {
+                               kfree(trans);
+                               free_percpu(stats);
+                               return -ENOMEM;
+                       }
                }
                list_add_tail(&trans->list, &net->nft.commit_list);
                return 0;
@@ -1544,7 +1551,11 @@ static int nf_tables_newchain(struct net *net, struct sock *nlsk,
        INIT_LIST_HEAD(&chain->rules);
        chain->handle = nf_tables_alloc_handle(table);
        chain->table = table;
-       nla_strlcpy(chain->name, name, NFT_CHAIN_MAXNAMELEN);
+       chain->name = nla_strdup(name, GFP_KERNEL);
+       if (!chain->name) {
+               err = -ENOMEM;
+               goto err1;
+       }
 
        err = nf_tables_register_hooks(net, table, chain, afi->nops);
        if (err < 0)
@@ -1979,7 +1990,7 @@ err:
 
 struct nft_rule_dump_ctx {
        char *table;
-       char chain[NFT_CHAIN_MAXNAMELEN];
+       char *chain;
 };
 
 static int nf_tables_dump_rules(struct sk_buff *skb,
@@ -2047,6 +2058,7 @@ static int nf_tables_dump_rules_done(struct netlink_callback *cb)
 
        if (ctx) {
                kfree(ctx->table);
+               kfree(ctx->chain);
                kfree(ctx);
        }
        return 0;
@@ -2088,9 +2100,15 @@ static int nf_tables_getrule(struct net *net, struct sock *nlsk,
                                        return -ENOMEM;
                                }
                        }
-                       if (nla[NFTA_RULE_CHAIN])
-                               nla_strlcpy(ctx->chain, nla[NFTA_RULE_CHAIN],
-                                           sizeof(ctx->chain));
+                       if (nla[NFTA_RULE_CHAIN]) {
+                               ctx->chain = nla_strdup(nla[NFTA_RULE_CHAIN],
+                                                       GFP_KERNEL);
+                               if (!ctx->chain) {
+                                       kfree(ctx->table);
+                                       kfree(ctx);
+                                       return -ENOMEM;
+                               }
+                       }
                        c.data = ctx;
                }
 
@@ -4863,7 +4881,7 @@ static void nft_chain_commit_update(struct nft_trans *trans)
 {
        struct nft_base_chain *basechain;
 
-       if (nft_trans_chain_name(trans)[0])
+       if (nft_trans_chain_name(trans))
                strcpy(trans->ctx.chain->name, nft_trans_chain_name(trans));
 
        if (!nft_is_base_chain(trans->ctx.chain))
index 62787d985e9d346257f77fccd07de5b86a662409..e1dc527a493b8ba38b9fbea1a20d9a405613d50c 100644 (file)
@@ -162,6 +162,27 @@ static int nf_trace_fill_rule_info(struct sk_buff *nlskb,
                            NFTA_TRACE_PAD);
 }
 
+static bool nft_trace_have_verdict_chain(struct nft_traceinfo *info)
+{
+       switch (info->type) {
+       case NFT_TRACETYPE_RETURN:
+       case NFT_TRACETYPE_RULE:
+               break;
+       default:
+               return false;
+       }
+
+       switch (info->verdict->code) {
+       case NFT_JUMP:
+       case NFT_GOTO:
+               break;
+       default:
+               return false;
+       }
+
+       return true;
+}
+
 void nft_trace_notify(struct nft_traceinfo *info)
 {
        const struct nft_pktinfo *pkt = info->pkt;
@@ -176,12 +197,11 @@ void nft_trace_notify(struct nft_traceinfo *info)
 
        size = nlmsg_total_size(sizeof(struct nfgenmsg)) +
                nla_total_size(strlen(info->chain->table->name)) +
-               nla_total_size(NFT_CHAIN_MAXNAMELEN) +
+               nla_total_size(strlen(info->chain->name)) +
                nla_total_size_64bit(sizeof(__be64)) +  /* rule handle */
                nla_total_size(sizeof(__be32)) +        /* trace type */
                nla_total_size(0) +                     /* VERDICT, nested */
                        nla_total_size(sizeof(u32)) +   /* verdict code */
-                       nla_total_size(NFT_CHAIN_MAXNAMELEN) + /* jump target */
                nla_total_size(sizeof(u32)) +           /* id */
                nla_total_size(NFT_TRACETYPE_LL_HSIZE) +
                nla_total_size(NFT_TRACETYPE_NETWORK_HSIZE) +
@@ -194,6 +214,9 @@ void nft_trace_notify(struct nft_traceinfo *info)
                nla_total_size(sizeof(u32)) +           /* nfproto */
                nla_total_size(sizeof(u32));            /* policy */
 
+       if (nft_trace_have_verdict_chain(info))
+               size += nla_total_size(strlen(info->verdict->chain->name)); /* jump target */
+
        skb = nlmsg_new(size, GFP_ATOMIC);
        if (!skb)
                return;