netdev - Re: [PATCH v2 net-next 0/8] inet: better const qualifier awareness

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-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <56f4f269-db99-9c97-cc42-c6969c49b3da@kernel.org>
Date:   Thu, 16 Mar 2023 12:35:13 -0600
From:   David Ahern <dsahern@...nel.org>
To:     Eric Dumazet <edumazet@...gle.com>,
        "David S . Miller" <davem@...emloft.net>,
        Jakub Kicinski <kuba@...nel.org>,
        Paolo Abeni <pabeni@...hat.com>
Cc:     netdev@...r.kernel.org, Simon Horman <simon.horman@...igine.com>,
        eric.dumazet@...il.com
Subject: Re: [PATCH v2 net-next 0/8] inet: better const qualifier awareness

On 3/16/23 9:31 AM, Eric Dumazet wrote:
> inet_sk() can be changed to propagate const qualifier,
> thanks to container_of_const()
> 
> Following patches in this series add more const qualifiers.
> 
> Other helpers like tcp_sk(), udp_sk(), raw_sk(), ... will be handled
> in following series.
> 
> Eric Dumazet (8):
>   inet: preserve const qualifier in inet_sk()
>   ipv4: constify ip_mc_sf_allow() socket argument
>   udp: constify __udp_is_mcast_sock() socket argument
>   ipv6: constify inet6_mc_check()
>   udp6: constify __udp_v6_is_mcast_sock() socket argument
>   ipv6: raw: constify raw_v6_match() socket argument
>   ipv4: raw: constify raw_v4_match() socket argument
>   inet_diag: constify raw_lookup() socket argument
> 
>  include/linux/igmp.h        | 2 +-
>  include/net/addrconf.h      | 2 +-
>  include/net/inet_sock.h     | 5 +----
>  include/net/raw.h           | 2 +-
>  include/net/rawv6.h         | 2 +-
>  include/trace/events/sock.h | 4 ++--
>  include/trace/events/tcp.h  | 2 +-
>  net/ipv4/igmp.c             | 4 ++--
>  net/ipv4/ip_output.c        | 5 +++--
>  net/ipv4/raw.c              | 4 ++--
>  net/ipv4/raw_diag.c         | 2 +-
>  net/ipv4/udp.c              | 4 ++--
>  net/ipv6/mcast.c            | 8 ++++----
>  net/ipv6/ping.c             | 2 +-
>  net/ipv6/raw.c              | 2 +-
>  net/ipv6/udp.c              | 6 +++---
>  net/mptcp/sockopt.c         | 2 +-
>  security/lsm_audit.c        | 4 ++--
>  18 files changed, 30 insertions(+), 32 deletions(-)
> 

For the set:

Reviewed-by: David Ahern <dsahern@...nel.org>