]> git.baikalelectronics.ru Git - kernel.git/commit
Merge branch 'net-sched-act_ctinfo-fixes'
authorDavid S. Miller <davem@davemloft.net>
Mon, 17 Jun 2019 21:00:30 +0000 (14:00 -0700)
committerDavid S. Miller <davem@davemloft.net>
Mon, 17 Jun 2019 21:00:30 +0000 (14:00 -0700)
commitee6c8794c93ad676565e39d9f1249f37d0b84b14
tree797b9dfd9bbca1f6faa647611ad82e94412bcf62
parent4450e4c4daf08a8ca642e6d93b2782ffa2161181
parent6ea4f80335b8d9ce1cee06b1cd7ba4eb1f8289c2
Merge branch 'net-sched-act_ctinfo-fixes'

Kevin Darbyshire-Bryant says:

====================
net: sched: act_ctinfo: fixes

This is first attempt at sending a small series.  Order is important
because one bug (policy validation) prevents us from encountering the
more important 'OOPS' generating bug in action creation.  Fix the OOPS
first.

Confession time: Until very recently, development of this module has
been done on 'net-next' tree to 'clean compile' level with run-time
testing on backports to 4.14 & 4.19 kernels under openwrt.  It turns out
that sched: action: based code has been under more active change than I
realised.

During the back & forward porting during development & testing, the
critical ACT_P_CREATED return code got missed despite being in the 4.14
& 4.19 backports.  I have now gone through the init functions, using
act_csum as reference with a fine toothed comb and am happy they do the
same things.

This issue hadn't been caught till now due to another issue caused by
new strict nla_parse_nested function failing parsing validation before
action creation.

Thanks to Marcelo Leitner <marcelo.leitner@gmail.com> for flagging
extack deficiency (fixed in ff8111068b78 sched: act_ctinfo: use extack
error reporting) which led to 7f27bdc5aabf ("netlink: add validation of
NLA_F_NESTED flag") and 637a69d0ea08 ("netlink: make validation more
configurable for future strictness”) which led to the policy validation
fix, which then led to the action creation fix both contained in this
series.

If I ever get to a developer conference please feel free to
tar/feather/apply cone of shame.
====================

Signed-off-by: David S. Miller <davem@davemloft.net>