Messages in this thread | | | From | Michal Kubecek <> | Subject | [PATCH net-next v2 0/3] netlink: strict attribute checking follow-up | Date | Thu, 2 May 2019 16:15:10 +0200 (CEST) |
| |
Three follow-up patches for recent strict netlink validation series.
Patch 1 fixes dump handling for genetlink families which validate and parse messages themselves (e.g. because they need different policies for diferent commands).
Patch 2 sets bad_attr in extack in one place where this was omitted.
Patch 3 adds new NL_VALIDATE_NESTED flags for strict validation to enable checking that NLA_F_NESTED value in received messages matches expectations and includes this flag in NL_VALIDATE_STRICT. This would change userspace visible behavior but the previous switching to NL_VALIDATE_STRICT for new code is still only in net-next at the moment.
v2: change error messages to mention NLA_F_NESTED explicitly
Michal Kubecek (3): genetlink: do not validate dump requests if there is no policy netlink: set bad attribute also on maxtype check netlink: add validation of NLA_F_NESTED flag
include/net/netlink.h | 11 ++++++++++- lib/nlattr.c | 18 +++++++++++++++++- net/netlink/genetlink.c | 24 ++++++++++++++---------- 3 files changed, 41 insertions(+), 12 deletions(-)
-- 2.21.0
| |