ipv4: Restore flowi4_oif update before call to xfrm_lookup_route
authorDavid Ahern <dsahern@kernel.org>
Fri, 9 Oct 2020 18:01:01 +0000 (11:01 -0700)
committerJakub Kicinski <kuba@kernel.org>
Sat, 10 Oct 2020 18:38:59 +0000 (11:38 -0700)
Tobias reported regressions in IPsec tests following the patch
referenced by the Fixes tag below. The root cause is dropping the
reset of the flowi4_oif after the fib_lookup. Apparently it is
needed for xfrm cases, so restore the oif update to ip_route_output_flow
right before the call to xfrm_lookup_route.

Fixes: 2fbc6e89b2f1 ("ipv4: Update exception handling for multipath routes via same device")
Reported-by: Tobias Brunner <tobias@strongswan.org>
Signed-off-by: David Ahern <dsahern@kernel.org>
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
net/ipv4/route.c

index 58642b29a499d7a7f4679ffc1d8718974279a535..9bd30fd4de4b4ebaf018ca4db487e97ea0010551 100644 (file)
@@ -2769,10 +2769,12 @@ struct rtable *ip_route_output_flow(struct net *net, struct flowi4 *flp4,
        if (IS_ERR(rt))
                return rt;
 
-       if (flp4->flowi4_proto)
+       if (flp4->flowi4_proto) {
+               flp4->flowi4_oif = rt->dst.dev->ifindex;
                rt = (struct rtable *)xfrm_lookup_route(net, &rt->dst,
                                                        flowi4_to_flowi(flp4),
                                                        sk, 0);
+       }
 
        return rt;
 }