From: "Jose M. Guisado Gomez" <gui...@riseup.net>

When userdata was introduced for tables and objects its allocation was
only freed inside the error path of the new{table, object} functions.

Free user data inside corresponding destroy functions for tables and
objects.

Fixes: b131c96496b3 ("netfilter: nf_tables: add userdata support for 
nft_object")
Fixes: 7a81575b806e ("netfilter: nf_tables: add userdata attributes to 
nft_table")
Signed-off-by: Jose M. Guisado Gomez <gui...@riseup.net>
Signed-off-by: Pablo Neira Ayuso <pa...@netfilter.org>
---
 net/netfilter/nf_tables_api.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/net/netfilter/nf_tables_api.c b/net/netfilter/nf_tables_api.c
index 84c0c1aaae99..b3c3c3fc1969 100644
--- a/net/netfilter/nf_tables_api.c
+++ b/net/netfilter/nf_tables_api.c
@@ -1211,6 +1211,7 @@ static void nf_tables_table_destroy(struct nft_ctx *ctx)
 
        rhltable_destroy(&ctx->table->chains_ht);
        kfree(ctx->table->name);
+       kfree(ctx->table->udata);
        kfree(ctx->table);
 }
 
@@ -6231,6 +6232,7 @@ static void nft_obj_destroy(const struct nft_ctx *ctx, 
struct nft_object *obj)
 
        module_put(obj->ops->type->owner);
        kfree(obj->key.name);
+       kfree(obj->udata);
        kfree(obj);
 }
 
-- 
2.20.1

Reply via email to