netfilter: nf_tables: move dumper state allocation into ->start
authorFlorian Westphal <fw@strlen.de>
Mon, 23 Jul 2018 10:47:14 +0000 (12:47 +0200)
committerPablo Neira Ayuso <pablo@netfilter.org>
Mon, 23 Jul 2018 22:36:33 +0000 (00:36 +0200)
commit90fd131afc565159c9e0ea742f082b337e10f8c6
tree6245dc04a7b2cda296665138ccd0f5f0852fe7b8
parent6613b6173dee098997229caf1f3b961c49da75e6
netfilter: nf_tables: move dumper state allocation into ->start

Shaochun Chen points out we leak dumper filter state allocations
stored in dump_control->data in case there is an error before netlink sets
cb_running (after which ->done will be called at some point).

In order to fix this, add .start functions and do the allocations
there.

->done is going to clean up, and in case error occurs before
->start invocation no cleanups need to be done anymore.

Reported-by: shaochun chen <cscnull@gmail.com>
Signed-off-by: Florian Westphal <fw@strlen.de>
Acked-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
net/netfilter/nf_tables_api.c