diff options
| author | Pablo Neira Ayuso | 2015-03-17 19:53:23 +0100 |
|---|---|---|
| committer | Pablo Neira Ayuso | 2015-03-18 11:57:31 +0100 |
| commit | ffdb210eb415501c289f6becafb54fe2f4535efa (patch) | |
| tree | 896a41492e8d6845cc7d061ca5f165b40cbba68c /net/netfilter/nf_tables_api.c | |
| parent | netfilter: Remove uses of seq_<foo> return values (diff) | |
| download | kernel-qcow2-linux-ffdb210eb415501c289f6becafb54fe2f4535efa.tar.gz kernel-qcow2-linux-ffdb210eb415501c289f6becafb54fe2f4535efa.tar.xz kernel-qcow2-linux-ffdb210eb415501c289f6becafb54fe2f4535efa.zip | |
netfilter: nf_tables: consolidate error path of nf_tables_newtable()
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
Diffstat (limited to 'net/netfilter/nf_tables_api.c')
| -rw-r--r-- | net/netfilter/nf_tables_api.c | 20 |
1 files changed, 11 insertions, 9 deletions
diff --git a/net/netfilter/nf_tables_api.c b/net/netfilter/nf_tables_api.c index ea51833c8f5a..a072d8769b9b 100644 --- a/net/netfilter/nf_tables_api.c +++ b/net/netfilter/nf_tables_api.c @@ -687,11 +687,10 @@ static int nf_tables_newtable(struct sock *nlsk, struct sk_buff *skb, if (!try_module_get(afi->owner)) return -EAFNOSUPPORT; + err = -ENOMEM; table = kzalloc(sizeof(*table), GFP_KERNEL); - if (table == NULL) { - module_put(afi->owner); - return -ENOMEM; - } + if (table == NULL) + goto err1; nla_strlcpy(table->name, name, NFT_TABLE_MAXNAMELEN); INIT_LIST_HEAD(&table->chains); @@ -700,13 +699,16 @@ static int nf_tables_newtable(struct sock *nlsk, struct sk_buff *skb, nft_ctx_init(&ctx, skb, nlh, afi, table, NULL, nla); err = nft_trans_table_add(&ctx, NFT_MSG_NEWTABLE); - if (err < 0) { - kfree(table); - module_put(afi->owner); - return err; - } + if (err < 0) + goto err2; + list_add_tail_rcu(&table->list, &afi->tables); return 0; +err2: + kfree(table); +err1: + module_put(afi->owner); + return err; } static int nft_flush_table(struct nft_ctx *ctx) |