diff options
author | Phil Sutter <phil@nwl.cc> | 2017-12-14 20:40:25 +0100 |
---|---|---|
committer | Pablo Neira Ayuso <pablo@netfilter.org> | 2017-12-15 16:23:09 +0100 |
commit | 57f85977ed72ee3d623bbc2391d503f8a7e72c5d (patch) | |
tree | 3f2bd858638bacea7f473eaf1b97388967119bd2 /src/trace.c | |
parent | 8bcf10b504c692deb3c98d395f42d34141f21e59 (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/trace.c')
-rw-r--r-- | src/trace.c | 3 |
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(); |