netdev - [PATCH net-next 0/6] netlink: support reporting missing attributes

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [thread-next>] [day] [month] [year] [list]

Date:   Tue, 23 Aug 2022 21:50:18 -0700
From:   Jakub Kicinski <kuba@...nel.org>
To:     davem@...emloft.net
Cc:     netdev@...r.kernel.org, edumazet@...gle.com, pabeni@...hat.com,
        mkubecek@...e.cz, johannes@...solutions.net,
        Jakub Kicinski <kuba@...nel.org>
Subject: [PATCH net-next 0/6] netlink: support reporting missing attributes

This series adds support for reporting missing attributes
in a structured way. We can't point at things which don't
exist so we point at the containing nest or preceding
fixed header and report which attr type we expected to see.

Example of (YAML-based) user space reporting ethtool header
missing:

 Kernel error: missing attribute: .header

I was tempted to integrate the check with the policy
but it seems tricky without doing a full scan, and there
may be a ton of attrs in the policy. So leaving that
for later.

Jakub Kicinski (6):
  netlink: add support for ext_ack missing attributes
  netlink: add helper for extack attr presence checking
  genetlink: add helper for checking required attrs and use it in
    devlink
  devlink: use missing attribute ext_ack
  ethtool: strset: report missing ETHTOOL_A_STRINGSET_ID via ext_ack
  ethtool: report missing header via ext_ack in the default handler

 Documentation/userspace-api/netlink/intro.rst |  7 +++-
 include/linux/netlink.h                       | 24 +++++++++++
 include/net/genetlink.h                       | 14 +++++++
 include/uapi/linux/netlink.h                  |  4 ++
 net/core/devlink.c                            | 41 +++++++++----------
 net/ethtool/netlink.c                         |  3 ++
 net/ethtool/strset.c                          |  2 +-
 net/netlink/af_netlink.c                      | 12 ++++++
 net/netlink/genetlink.c                       |  2 +-
 9 files changed, 84 insertions(+), 25 deletions(-)

-- 
2.37.2