diff mbox series

netfilter: Fix memleak in nf_nat_init

Message ID 20210109120121.15938-1-dinghao.liu@zju.edu.cn
State New
Headers show
Series netfilter: Fix memleak in nf_nat_init | expand

Commit Message

Dinghao Liu Jan. 9, 2021, 12:01 p.m. UTC 
When register_pernet_subsys() fails, nf_nat_bysource
should be freed just like when nf_ct_extend_register()
fails.

Fixes: 1cd472bf036ca ("netfilter: nf_nat: add nat hook register functions to nf_nat")
Signed-off-by: Dinghao Liu <dinghao.liu@zju.edu.cn>
---
 net/netfilter/nf_nat_core.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Pablo Neira Ayuso Jan. 10, 2021, 8:39 a.m. UTC | #1 
On Sat, Jan 09, 2021 at 08:01:21PM +0800, Dinghao Liu wrote:
> When register_pernet_subsys() fails, nf_nat_bysource

> should be freed just like when nf_ct_extend_register()

> fails.


Applied, thanks.
diff mbox series

Patch

diff --git a/net/netfilter/nf_nat_core.c b/net/netfilter/nf_nat_core.c
index ea923f8cf9c4..b7c3c902290f 100644
--- a/net/netfilter/nf_nat_core.c
+++ b/net/netfilter/nf_nat_core.c
@@ -1174,6 +1174,7 @@  static int __init nf_nat_init(void)
 	ret = register_pernet_subsys(&nat_net_ops);
 	if (ret < 0) {
 		nf_ct_extend_unregister(&nat_extend);
+		kvfree(nf_nat_bysource);
 		return ret;
 	}