netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 0/6 net-next] net: Introduce struct sockaddr_unspec
@ 2025-07-23 23:19 Kees Cook
  2025-07-23 23:19 ` [PATCH 1/6 net-next] net: uapi: Add __kernel_sockaddr_unspec for sockaddr of unknown length Kees Cook
                   ` (5 more replies)
  0 siblings, 6 replies; 13+ messages in thread
From: Kees Cook @ 2025-07-23 23:19 UTC (permalink / raw)
  To: Jakub Kicinski
  Cc: Kees Cook, David S. Miller, Eric Dumazet, Paolo Abeni,
	Simon Horman, Kuniyuki Iwashima, Willem de Bruijn, netdev

Hi,

The historically fixed-size struct sockaddr is part of UAPI and embedded
in many existing structures. The kernel uses struct sockaddr extensively
within the kernel to represent arbitrarily sized sockaddr structures,
which caused problems with the compiler's ability to determine object
sizes correctly. The "temporary" solution was to make sockaddr explicitly
use a flexible array, but this causes problems for embedding struct
sockaddr in structures, where once again the compiler has to guess about
the size of such objects.

Switching to sockaddr_storage internally everywhere wastes a lot of memory,
so we are left with needing two changes:
- introduction of an explicitly arbitrarily sized sockaddr struct
- switch struct sockaddr back to being fixed size

Doing the latter step requires all "arbitrarily sized" uses of struct
sockaddr to be replaced with the new struct from the first step.

So, introduce the new struct and start the conversions.

Thanks!

-Kees


Kees Cook (6):
  net: uapi: Add __kernel_sockaddr_unspec for sockaddr of unknown length
  net/l2tp: Add missing sa_family validation in
    pppol2tp_sockaddr_get_info
  net: Convert proto_ops bind() callbacks to use sockaddr_unspec
  net: Convert proto_ops connect() callbacks to use sockaddr_unspec
  net: Remove struct sockaddr from net.h
  net: Convert proto callbacks from sockaddr to sockaddr_unspec

 include/linux/bpf-cgroup.h                |  6 +++---
 include/linux/net.h                       | 11 ++++++-----
 include/net/inet_common.h                 | 13 ++++++-------
 include/net/ip.h                          |  4 ++--
 include/net/ipv6.h                        | 10 +++++-----
 include/net/ipv6_stubs.h                  |  2 +-
 include/net/ping.h                        |  2 +-
 include/net/sctp/sctp.h                   |  2 +-
 include/net/sock.h                        | 14 +++++++-------
 include/net/tcp.h                         |  2 +-
 include/net/udp.h                         |  2 +-
 include/net/vsock_addr.h                  |  2 +-
 include/uapi/linux/socket.h               | 15 +++++++++++++++
 net/rds/rds.h                             |  2 +-
 net/smc/smc.h                             |  4 ++--
 crypto/af_alg.c                           |  2 +-
 drivers/block/drbd/drbd_receiver.c        |  6 +++---
 drivers/infiniband/hw/erdma/erdma_cm.c    |  6 +++---
 drivers/infiniband/sw/siw/siw_cm.c        |  8 ++++----
 drivers/isdn/mISDN/l1oip_core.c           |  2 +-
 drivers/isdn/mISDN/socket.c               |  4 ++--
 drivers/net/ppp/pppoe.c                   |  2 +-
 drivers/net/ppp/pptp.c                    |  4 ++--
 drivers/net/wireless/ath/ath10k/qmi.c     |  2 +-
 drivers/net/wireless/ath/ath11k/qmi.c     |  2 +-
 drivers/net/wireless/ath/ath12k/qmi.c     |  2 +-
 drivers/nvme/host/tcp.c                   |  4 ++--
 drivers/nvme/target/tcp.c                 |  2 +-
 drivers/slimbus/qcom-ngd-ctrl.c           |  2 +-
 drivers/target/iscsi/iscsi_target_login.c |  2 +-
 drivers/xen/pvcalls-back.c                |  4 ++--
 fs/afs/rxrpc.c                            |  6 +++---
 fs/coredump.c                             |  2 +-
 fs/dlm/lowcomms.c                         |  8 ++++----
 fs/ocfs2/cluster/tcp.c                    |  6 +++---
 fs/smb/client/connect.c                   |  4 ++--
 fs/smb/server/transport_tcp.c             |  4 ++--
 net/9p/trans_fd.c                         |  6 +++---
 net/appletalk/ddp.c                       |  4 ++--
 net/atm/pvc.c                             |  4 ++--
 net/atm/svc.c                             |  4 ++--
 net/ax25/af_ax25.c                        |  4 ++--
 net/bluetooth/hci_sock.c                  |  2 +-
 net/bluetooth/iso.c                       |  6 +++---
 net/bluetooth/l2cap_sock.c                |  4 ++--
 net/bluetooth/rfcomm/core.c               |  6 +++---
 net/bluetooth/rfcomm/sock.c               |  4 ++--
 net/bluetooth/sco.c                       |  4 ++--
 net/caif/caif_socket.c                    |  2 +-
 net/can/bcm.c                             |  2 +-
 net/can/isotp.c                           |  2 +-
 net/can/j1939/socket.c                    |  4 ++--
 net/can/raw.c                             |  2 +-
 net/ceph/messenger.c                      |  2 +-
 net/core/filter.c                         |  4 ++--
 net/core/sock.c                           |  6 +++---
 net/ieee802154/socket.c                   | 12 ++++++------
 net/ipv4/af_inet.c                        | 12 ++++++------
 net/ipv4/datagram.c                       |  4 ++--
 net/ipv4/ping.c                           |  8 ++++----
 net/ipv4/raw.c                            |  3 ++-
 net/ipv4/tcp.c                            |  2 +-
 net/ipv4/tcp_ipv4.c                       |  4 ++--
 net/ipv4/udp.c                            |  6 ++++--
 net/ipv4/udp_tunnel_core.c                |  4 ++--
 net/ipv6/af_inet6.c                       |  6 +++---
 net/ipv6/datagram.c                       |  8 ++++----
 net/ipv6/ip6_udp_tunnel.c                 |  4 ++--
 net/ipv6/ping.c                           |  2 +-
 net/ipv6/raw.c                            |  3 ++-
 net/ipv6/tcp_ipv6.c                       |  6 +++---
 net/ipv6/udp.c                            |  5 +++--
 net/l2tp/l2tp_core.c                      |  8 ++++----
 net/l2tp/l2tp_ip.c                        |  6 ++++--
 net/l2tp/l2tp_ip6.c                       |  5 +++--
 net/l2tp/l2tp_ppp.c                       |  9 ++++++++-
 net/llc/af_llc.c                          |  4 ++--
 net/mctp/af_mctp.c                        |  2 +-
 net/mctp/test/route-test.c                |  2 +-
 net/mptcp/pm_kernel.c                     |  4 ++--
 net/mptcp/protocol.c                      |  5 +++--
 net/mptcp/subflow.c                       |  4 ++--
 net/netfilter/ipvs/ip_vs_sync.c           |  6 +++---
 net/netlink/af_netlink.c                  |  4 ++--
 net/netrom/af_netrom.c                    |  4 ++--
 net/nfc/llcp_sock.c                       |  6 +++---
 net/nfc/rawsock.c                         |  2 +-
 net/packet/af_packet.c                    |  4 ++--
 net/phonet/pep.c                          |  3 ++-
 net/phonet/socket.c                       |  6 +++---
 net/qrtr/af_qrtr.c                        |  4 ++--
 net/qrtr/ns.c                             |  2 +-
 net/rds/af_rds.c                          |  2 +-
 net/rds/bind.c                            |  2 +-
 net/rds/tcp_connect.c                     |  4 ++--
 net/rds/tcp_listen.c                      |  2 +-
 net/rose/af_rose.c                        |  4 ++--
 net/rxrpc/af_rxrpc.c                      |  4 ++--
 net/rxrpc/rxperf.c                        |  2 +-
 net/sctp/socket.c                         | 13 +++++++------
 net/smc/af_smc.c                          |  6 +++---
 net/socket.c                              | 12 ++++++------
 net/sunrpc/clnt.c                         |  6 +++---
 net/sunrpc/svcsock.c                      |  2 +-
 net/sunrpc/xprtsock.c                     |  6 +++---
 net/tipc/socket.c                         |  6 +++---
 net/unix/af_unix.c                        | 12 ++++++------
 net/vmw_vsock/af_vsock.c                  |  6 +++---
 net/vmw_vsock/vsock_addr.c                |  2 +-
 net/x25/af_x25.c                          |  4 ++--
 net/xdp/xsk.c                             |  2 +-
 111 files changed, 278 insertions(+), 245 deletions(-)

-- 
2.34.1


^ permalink raw reply	[flat|nested] 13+ messages in thread

end of thread, other threads:[~2025-07-24 22:28 UTC | newest]

Thread overview: 13+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-07-23 23:19 [PATCH 0/6 net-next] net: Introduce struct sockaddr_unspec Kees Cook
2025-07-23 23:19 ` [PATCH 1/6 net-next] net: uapi: Add __kernel_sockaddr_unspec for sockaddr of unknown length Kees Cook
2025-07-24 21:40   ` Jakub Kicinski
2025-07-24 22:28     ` Kees Cook
2025-07-23 23:19 ` [PATCH 2/6 net-next] net/l2tp: Add missing sa_family validation in pppol2tp_sockaddr_get_info Kees Cook
2025-07-23 23:19 ` [PATCH 3/6 net-next] net: Convert proto_ops bind() callbacks to use sockaddr_unspec Kees Cook
2025-07-24 11:29   ` kernel test robot
2025-07-24 13:31     ` Kees Cook
2025-07-24 13:02   ` kernel test robot
2025-07-23 23:19 ` [PATCH 4/6 net-next] net: Convert proto_ops connect() " Kees Cook
2025-07-24 12:00   ` kernel test robot
2025-07-23 23:19 ` [PATCH 5/6 net-next] net: Remove struct sockaddr from net.h Kees Cook
2025-07-23 23:19 ` [PATCH 6/6 net-next] net: Convert proto callbacks from sockaddr to sockaddr_unspec Kees Cook

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).