netdev - Re: [PATCH net-next] netlink: make range pointers in policies const

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: <0d48d284-3138-4ca7-b622-20400fab7579@kernel.org>
Date: Wed, 25 Oct 2023 12:41:20 -0600
From: David Ahern <dsahern@...nel.org>
To: Jakub Kicinski <kuba@...nel.org>, davem@...emloft.net
Cc: netdev@...r.kernel.org, edumazet@...gle.com, pabeni@...hat.com,
 j.vosburgh@...il.com, andy@...yhouse.net, jhs@...atatu.com,
 xiyou.wangcong@...il.com, jiri@...nulli.us, vinicius.gomes@...el.com,
 johannes@...solutions.net, razor@...ckwall.org, idosch@...dia.com,
 linux-wireless@...r.kernel.org
Subject: Re: [PATCH net-next] netlink: make range pointers in policies const

On 10/25/23 10:22 AM, Jakub Kicinski wrote:
> struct nla_policy is usually constant itself, but unless
> we make the ranges inside constant we won't be able to
> make range structs const. The ranges are not modified
> by the core.
> 
> Signed-off-by: Jakub Kicinski <kuba@...nel.org>
> ---
> CC: j.vosburgh@...il.com
> CC: andy@...yhouse.net
> CC: dsahern@...nel.org
> CC: jhs@...atatu.com
> CC: xiyou.wangcong@...il.com
> CC: jiri@...nulli.us
> CC: vinicius.gomes@...el.com
> CC: johannes@...solutions.net
> CC: razor@...ckwall.org
> CC: idosch@...dia.com
> CC: linux-wireless@...r.kernel.org
> ---
>  drivers/net/bonding/bond_netlink.c | 2 +-
>  drivers/net/vxlan/vxlan_mdb.c      | 2 +-
>  include/net/netlink.h              | 4 ++--
>  net/ipv6/ioam6_iptunnel.c          | 2 +-
>  net/sched/sch_fq.c                 | 2 +-
>  net/sched/sch_fq_pie.c             | 2 +-
>  net/sched/sch_qfq.c                | 2 +-
>  net/sched/sch_taprio.c             | 2 +-
>  net/wireless/nl80211.c             | 2 +-
>  tools/net/ynl/ynl-gen-c.py         | 2 +-
>  10 files changed, 11 insertions(+), 11 deletions(-)
> 


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