On Tue, 14 Feb 2017 16:30:35 +0200, Or Gerlitz wrote: > The skip flags are not dumped to user-space, do that. > > Signed-off-by: Or Gerlitz <ogerl...@mellanox.com> > Acked-by: Jiri Pirko <j...@mellanox.com> > Acked-by: Yotam Gigi <yot...@mellanox.com> > --- > net/sched/cls_matchall.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/net/sched/cls_matchall.c b/net/sched/cls_matchall.c > index f2141cb..ce2f7d4 100644 > --- a/net/sched/cls_matchall.c > +++ b/net/sched/cls_matchall.c > @@ -244,6 +244,8 @@ static int mall_dump(struct net *net, struct tcf_proto > *tp, unsigned long fh, > nla_put_u32(skb, TCA_MATCHALL_CLASSID, head->res.classid)) > goto nla_put_failure; > > + nla_put_u32(skb, TCA_MATCHALL_FLAGS, head->flags); > +
Shouldn't the return status from nla_put_u32() be checked?
