netdev - Re: [PATCH net-next v2 2/2] net: Add helper function to parse netlink msg of ip_tunnel

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: <432c0ab0943320affa092d93868973c7c9b060b8.camel@redhat.com>
Date:   Thu, 29 Sep 2022 15:16:41 +0200
From:   Paolo Abeni <pabeni@...hat.com>
To:     Liu Jian <liujian56@...wei.com>, davem@...emloft.net,
        yoshfuji@...ux-ipv6.org, dsahern@...nel.org, edumazet@...gle.com,
        kuba@...nel.org, netdev@...r.kernel.org
Subject: Re: [PATCH net-next v2 2/2] net: Add helper function to parse
 netlink msg of ip_tunnel_parm

On Wed, 2022-09-28 at 11:39 +0800, Liu Jian wrote:
> Add ip_tunnel_netlink_parms to parse netlink msg of ip_tunnel_parm.
> Reduces duplicate code, no actual functional changes.
> 
> Signed-off-by: Liu Jian <liujian56@...wei.com>
> ---
> v1->v2: Move the implementation of the helper function to ip_tunnel_core.c
>  include/net/ip_tunnels.h  |  3 +++
>  net/ipv4/ip_tunnel_core.c | 32 ++++++++++++++++++++++++++++++++
>  net/ipv4/ipip.c           | 24 +-----------------------
>  net/ipv6/sit.c            | 27 +--------------------------
>  4 files changed, 37 insertions(+), 49 deletions(-)
> 
> diff --git a/include/net/ip_tunnels.h b/include/net/ip_tunnels.h
> index 51da2957cf48..fca357679816 100644
> --- a/include/net/ip_tunnels.h
> +++ b/include/net/ip_tunnels.h
> @@ -305,6 +305,9 @@ void ip_tunnel_setup(struct net_device *dev, unsigned int net_id);
>  bool ip_tunnel_netlink_encap_parms(struct nlattr *data[],
>  				   struct ip_tunnel_encap *encap);
>  
> +void ip_tunnel_netlink_parms(struct nlattr *data[],
> +			     struct ip_tunnel_parm *parms);
> +
>  extern const struct header_ops ip_tunnel_header_ops;
>  __be16 ip_tunnel_parse_protocol(const struct sk_buff *skb);
>  
> diff --git a/net/ipv4/ip_tunnel_core.c b/net/ipv4/ip_tunnel_core.c
> index 526e6a52a973..1e1217e87885 100644
> --- a/net/ipv4/ip_tunnel_core.c
> +++ b/net/ipv4/ip_tunnel_core.c
> @@ -1114,3 +1114,35 @@ bool ip_tunnel_netlink_encap_parms(struct nlattr *data[],
>  	return ret;
>  }
>  EXPORT_SYMBOL(ip_tunnel_netlink_encap_parms);
> +
> +void ip_tunnel_netlink_parms(struct nlattr *data[],
> +			     struct ip_tunnel_parm *parms)
> +{
> +	if (data[IFLA_IPTUN_LINK])
> +		parms->link = nla_get_u32(data[IFLA_IPTUN_LINK]);
> +
> +	if (data[IFLA_IPTUN_LOCAL])
> +		parms->iph.saddr = nla_get_be32(data[IFLA_IPTUN_LOCAL]);
> +
> +	if (data[IFLA_IPTUN_REMOTE])
> +		parms->iph.daddr = nla_get_be32(data[IFLA_IPTUN_REMOTE]);
> +
> +	if (data[IFLA_IPTUN_TTL]) {
> +		parms->iph.ttl = nla_get_u8(data[IFLA_IPTUN_TTL]);
> +		if (parms->iph.ttl)
> +			parms->iph.frag_off = htons(IP_DF);
> +	}
> +
> +	if (data[IFLA_IPTUN_TOS])
> +		parms->iph.tos = nla_get_u8(data[IFLA_IPTUN_TOS]);
> +
> +	if (!data[IFLA_IPTUN_PMTUDISC] || nla_get_u8(data[IFLA_IPTUN_PMTUDISC]))
> +		parms->iph.frag_off = htons(IP_DF);
> +
> +	if (data[IFLA_IPTUN_FLAGS])
> +		parms->i_flags = nla_get_be16(data[IFLA_IPTUN_FLAGS]);
> +
> +	if (data[IFLA_IPTUN_PROTO])
> +		parms->iph.protocol = nla_get_u8(data[IFLA_IPTUN_PROTO]);
> +}
> +EXPORT_SYMBOL(ip_tunnel_netlink_parms);

The same here, I think it should be EXPORT_SYMBOL_GPL()

Thanks,

Paolo