|
|
Log in / Subscribe / Register

xfrm: start adding netlink extack support

From:  Sabrina Dubroca <sd-AT-queasysnail.net>
To:  netdev-AT-vger.kernel.org
Subject:  [RFC PATCH ipsec-next 0/6] xfrm: start adding netlink extack support
Date:  Thu, 11 Nov 2021 13:02:41 +0100
Message-ID:  <cover.1636450303.git.sd@queasysnail.net>
Cc:  steffen.klassert-AT-secunet.com, Sabrina Dubroca <sd-AT-queasysnail.net>
Archive-link:  Article

XFRM states and policies are complex objects, and there are many
reasons why the kernel can reject userspace's request to create
one. This series makes it a bit clearer by providing extended ack
messages for policy creation.

A few other operations that reuse the same helper functions are also
getting partial extack support in this series. More patches will
follow to complete extack support, in particular for state creation.

Note: The policy->share attribute seems to be entirely ignored in the
kernel outside of checking its value in verify_newpolicy_info(). There
are some (very) old comments in copy_from_user_policy and
copy_to_user_policy suggesting that it should at least be copied
to/from userspace. I don't know what it was intended for.

Sabrina Dubroca (6):
  xfrm: propagate extack to all netlink doit handlers
  xfrm: add extack support to verify_newpolicy_info
  xfrm: add extack to verify_policy_dir
  xfrm: add extack to validate_tmpl
  xfrm: add extack to verify_policy_type
  xfrm: add extack to verify_sec_ctx_len

 net/xfrm/xfrm_user.c | 163 +++++++++++++++++++++++++++----------------
 1 file changed, 103 insertions(+), 60 deletions(-)

-- 
2.33.1



Copyright © 2021, Eklektix, Inc.
Comments and public postings are copyrighted by their creators.
Linux is a registered trademark of Linus Torvalds