genetlink: piggy back on resv_op to default to a reject policy
authorJakub Kicinski <kuba@kernel.org>
Fri, 21 Oct 2022 19:35:32 +0000 (12:35 -0700)
committerJakub Kicinski <kuba@kernel.org>
Tue, 25 Oct 2022 02:08:46 +0000 (19:08 -0700)
commit4fa86555d1cd338afc6e6308cc1ff890a014ec8c
treee707212a1ba96a3966ad51ba3f9e8b5ee01e63a5
parent9d9effca9d7d7cf6341182a7c5cabcbd6fa28063
genetlink: piggy back on resv_op to default to a reject policy

To keep backward compatibility we used to leave attribute parsing
to the family if no policy is specified. This becomes tedious as
we move to more strict validation. Families must define reject all
policies if they don't want any attributes accepted.

Piggy back on the resv_start_op field as the switchover point.
AFAICT only ethtool has added new commands since the resv_start_op
was defined, and it has per-op policies so this should be a no-op.

Nonetheless the patch should still go into v6.1 for consistency.

Link: https://lore.kernel.org/all/20221019125745.3f2e7659@kernel.org/
Link: https://lore.kernel.org/r/20221021193532.1511293-1-kuba@kernel.org
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
include/net/genetlink.h
net/netlink/genetlink.c