Revision 90fd131afc565159c9e0ea742f082b337e10f8c6 authored by Florian Westphal on 23 July 2018, 10:47:14 UTC, committed by Pablo Neira Ayuso on 23 July 2018, 22:36:33 UTC
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>
1 parent 6613b61
File | Mode | Size |
---|---|---|
Kconfig | -rw-r--r-- | 8.3 KB |
Kconfig.debug | -rw-r--r-- | 2.2 KB |
Makefile | -rw-r--r-- | 516 bytes |
rcu.h | -rw-r--r-- | 15.4 KB |
rcu_segcblist.c | -rw-r--r-- | 13.7 KB |
rcu_segcblist.h | -rw-r--r-- | 4.7 KB |
rcuperf.c | -rw-r--r-- | 18.9 KB |
rcutorture.c | -rw-r--r-- | 55.0 KB |
srcutiny.c | -rw-r--r-- | 5.8 KB |
srcutree.c | -rw-r--r-- | 42.6 KB |
sync.c | -rw-r--r-- | 7.1 KB |
tiny.c | -rw-r--r-- | 6.2 KB |
tree.c | -rw-r--r-- | 127.7 KB |
tree.h | -rw-r--r-- | 19.7 KB |
tree_exp.h | -rw-r--r-- | 24.3 KB |
tree_plugin.h | -rw-r--r-- | 80.2 KB |
update.c | -rw-r--r-- | 29.8 KB |
![swh spinner](/static/img/swh-spinner.gif)
Computing file changes ...