From 57f85977ed72ee3d623bbc2391d503f8a7e72c5d Mon Sep 17 00:00:00 2001 From: Phil Sutter Date: Thu, 14 Dec 2017 20:40:25 +0100 Subject: trace: Check return value of mnl_attr_parse_nested() This is done everywhere else as well, so certainly not a bad thing here either. Signed-off-by: Phil Sutter Signed-off-by: Pablo Neira Ayuso --- src/trace.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'src') diff --git a/src/trace.c b/src/trace.c index bd05d3c..b016e72 100644 --- a/src/trace.c +++ b/src/trace.c @@ -301,7 +301,8 @@ static int nftnl_trace_parse_verdict(const struct nlattr *attr, { struct nlattr *tb[NFTA_VERDICT_MAX+1]; - mnl_attr_parse_nested(attr, nftnl_trace_parse_verdict_cb, tb); + if (mnl_attr_parse_nested(attr, nftnl_trace_parse_verdict_cb, tb) < 0) + return -1; if (!tb[NFTA_VERDICT_CODE]) abi_breakage(); -- cgit v1.2.3