netdev - Re: [PATCH RFC net-next 08/20] net/ipv6: Defer initialization of dst to data path

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]

Date:   Mon, 26 Feb 2018 14:17:13 -0500 (EST)
From:   David Miller <davem@...emloft.net>
To:     dsahern@...il.com
Cc:     netdev@...r.kernel.org, idosch@...sch.org,
        roopa@...ulusnetworks.com, eric.dumazet@...il.com,
        weiwan@...gle.com, kafai@...com, yoshfuji@...ux-ipv6.org
Subject: Re: [PATCH RFC net-next 08/20] net/ipv6: Defer initialization of
 dst to data path

From: David Ahern <dsahern@...il.com>
Date: Sun, 25 Feb 2018 11:47:18 -0800

> +static void ip6_rt_init_dst(struct rt6_info *rt, struct rt6_info *ort)
> +{
 ...
> +	rt->dst.error = 0;
> +	rt->dst.output = ip6_output;
 ...
> @@ -930,14 +999,12 @@ static void rt6_set_from(struct rt6_info *rt, struct rt6_info *from)
>  
>  static void ip6_rt_copy_init(struct rt6_info *rt, struct rt6_info *ort)
>  {
> -	rt->dst.input = ort->dst.input;
> -	rt->dst.output = ort->dst.output;
> +	ip6_rt_init_dst(rt, ort);
> +
>  	rt->rt6i_dst = ort->rt6i_dst;
> -	rt->dst.error = ort->dst.error;
>  	rt->rt6i_idev = ort->rt6i_idev;
>  	if (rt->rt6i_idev)
>  		in6_dev_hold(rt->rt6i_idev);
> -	rt->dst.lastuse = jiffies;
>  	rt->rt6i_gateway = ort->fib6_nh.nh_gw;
>  	rt->rt6i_flags = ort->rt6i_flags;
>  	rt6_set_from(rt, ort);

This seems to change behavior.

In the old code, the dst error value is propagated from 'ort' into 'rt'.

Here you set it to zero and that's it.

Is it set somewhere else?

I don't think you can assume that all routes that go via this copy
path are not reject routes or other kinds that need the error code
set, if that is what you were thinking.

Thanks.