netfilter: nf_tables: add nft_chain_add()
This patch adds a helper function to add the chain to the hashtable and the chain list. Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
This commit is contained in:
parent
67c49de4ad
commit
04b7db4144
|
@ -1914,6 +1914,20 @@ static int nft_basechain_init(struct nft_base_chain *basechain, u8 family,
|
||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
static int nft_chain_add(struct nft_table *table, struct nft_chain *chain)
|
||||||
|
{
|
||||||
|
int err;
|
||||||
|
|
||||||
|
err = rhltable_insert_key(&table->chains_ht, chain->name,
|
||||||
|
&chain->rhlhead, nft_chain_ht_params);
|
||||||
|
if (err)
|
||||||
|
return err;
|
||||||
|
|
||||||
|
list_add_tail_rcu(&chain->list, &table->chains);
|
||||||
|
|
||||||
|
return 0;
|
||||||
|
}
|
||||||
|
|
||||||
static int nf_tables_addchain(struct nft_ctx *ctx, u8 family, u8 genmask,
|
static int nf_tables_addchain(struct nft_ctx *ctx, u8 family, u8 genmask,
|
||||||
u8 policy, u32 flags)
|
u8 policy, u32 flags)
|
||||||
{
|
{
|
||||||
|
@ -1991,16 +2005,9 @@ static int nf_tables_addchain(struct nft_ctx *ctx, u8 family, u8 genmask,
|
||||||
if (err < 0)
|
if (err < 0)
|
||||||
goto err1;
|
goto err1;
|
||||||
|
|
||||||
err = rhltable_insert_key(&table->chains_ht, chain->name,
|
|
||||||
&chain->rhlhead, nft_chain_ht_params);
|
|
||||||
if (err)
|
|
||||||
goto err2;
|
|
||||||
|
|
||||||
trans = nft_trans_chain_add(ctx, NFT_MSG_NEWCHAIN);
|
trans = nft_trans_chain_add(ctx, NFT_MSG_NEWCHAIN);
|
||||||
if (IS_ERR(trans)) {
|
if (IS_ERR(trans)) {
|
||||||
err = PTR_ERR(trans);
|
err = PTR_ERR(trans);
|
||||||
rhltable_remove(&table->chains_ht, &chain->rhlhead,
|
|
||||||
nft_chain_ht_params);
|
|
||||||
goto err2;
|
goto err2;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -2008,8 +2015,13 @@ static int nf_tables_addchain(struct nft_ctx *ctx, u8 family, u8 genmask,
|
||||||
if (nft_is_base_chain(chain))
|
if (nft_is_base_chain(chain))
|
||||||
nft_trans_chain_policy(trans) = policy;
|
nft_trans_chain_policy(trans) = policy;
|
||||||
|
|
||||||
|
err = nft_chain_add(table, chain);
|
||||||
|
if (err < 0) {
|
||||||
|
nft_trans_destroy(trans);
|
||||||
|
goto err2;
|
||||||
|
}
|
||||||
|
|
||||||
table->use++;
|
table->use++;
|
||||||
list_add_tail_rcu(&chain->list, &table->chains);
|
|
||||||
|
|
||||||
return 0;
|
return 0;
|
||||||
err2:
|
err2:
|
||||||
|
|
Loading…
Reference in New Issue