From dbaf6ea8f6a1a1e7f1d5abc2e4e2fef891c471b7 Mon Sep 17 00:00:00 2001 From: Phil Sutter Date: Thu, 14 Dec 2017 20:40:23 +0100 Subject: ruleset: Avoid reading garbage in nftnl_ruleset_cb() If nftnl_ruleset_json_parse() is called with arg == NULL, ctx.data is left uninitialized and will later be used in nftnl_ruleset_cb(). Avoid this by using a C99-style initializer for 'ctx' which sets all omitted fields to zero. Signed-off-by: Phil Sutter Signed-off-by: Pablo Neira Ayuso --- src/ruleset.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) (limited to 'src') diff --git a/src/ruleset.c b/src/ruleset.c index 3de9b87..cf86ca6 100644 --- a/src/ruleset.c +++ b/src/ruleset.c @@ -519,11 +519,11 @@ static int nftnl_ruleset_json_parse(const void *json, json_error_t error; int i, len; const char *key; - struct nftnl_parse_ctx ctx; - - ctx.cb = cb; - ctx.format = type; - ctx.flags = 0; + struct nftnl_parse_ctx ctx = { + .cb = cb, + .format = type, + .flags = 0, + }; ctx.set_list = nftnl_set_list_alloc(); if (ctx.set_list == NULL) -- cgit v1.2.3