summaryrefslogtreecommitdiffstats
path: root/src
diff options
context:
space:
mode:
authorPhil Sutter <phil@nwl.cc>2017-12-14 20:40:25 +0100
committerPablo Neira Ayuso <pablo@netfilter.org>2017-12-15 16:23:09 +0100
commit57f85977ed72ee3d623bbc2391d503f8a7e72c5d (patch)
tree3f2bd858638bacea7f473eaf1b97388967119bd2 /src
parent8bcf10b504c692deb3c98d395f42d34141f21e59 (diff)
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 <phil@nwl.cc> Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
Diffstat (limited to 'src')
-rw-r--r--src/trace.c3
1 files changed, 2 insertions, 1 deletions
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();