This is done everywhere else as well, so certainly not a bad thing here either. Signed-off-by: Phil Sutter <phil@xxxxxx> --- src/trace.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/trace.c b/src/trace.c index bd05d3c58d2fa..b016e723a7ded 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(); -- 2.13.1 -- To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html