From 27d8946db90b79762a36e66647bb8d8fc4c17ce9 Mon Sep 17 00:00:00 2001 From: Phil Sutter Date: Fri, 12 Oct 2018 17:50:15 +0200 Subject: json: Fix memleak in dup_stmt_json() The variable 'root' is always assigned to after initialization, so there is no point in initializing it upon declaration. Fixes: e70354f53e9f6 ("libnftables: Implement JSON output support") Signed-off-by: Phil Sutter Signed-off-by: Pablo Neira Ayuso --- src/json.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'src/json.c') diff --git a/src/json.c b/src/json.c index f5d97c62..1ab2d431 100644 --- a/src/json.c +++ b/src/json.c @@ -1131,7 +1131,7 @@ json_t *notrack_stmt_json(const struct stmt *stmt, struct output_ctx *octx) json_t *dup_stmt_json(const struct stmt *stmt, struct output_ctx *octx) { - json_t *root = json_object(); + json_t *root; if (stmt->dup.to) { root = json_pack("{s:o}", "addr", expr_print_json(stmt->dup.to, octx)); -- cgit v1.2.3