]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
ipv4: Use inet_sk_init_flowi4() in ip4_datagram_release_cb().
authorGuillaume Nault <gnault@redhat.com>
Mon, 16 Dec 2024 17:21:46 +0000 (18:21 +0100)
committerJakub Kicinski <kuba@kernel.org>
Fri, 20 Dec 2024 21:50:09 +0000 (13:50 -0800)
Use inet_sk_init_flowi4() to automatically initialise the flowi4
structure in ip4_datagram_release_cb() instead of passing parameters
manually to ip_route_output_ports().

Signed-off-by: Guillaume Nault <gnault@redhat.com>
Link: https://patch.msgid.link/9c326b8d9e919478f7952b21473d31da07eba2dd.1734357769.git.gnault@redhat.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
net/ipv4/datagram.c

index 4aca1f05edd3146403c53e319e53221ff78ab1b2..4b5bc6eb52e750ec6570d5c3e2e1d62bf50b3edb 100644 (file)
@@ -102,8 +102,6 @@ EXPORT_SYMBOL(ip4_datagram_connect);
 void ip4_datagram_release_cb(struct sock *sk)
 {
        const struct inet_sock *inet = inet_sk(sk);
-       const struct ip_options_rcu *inet_opt;
-       __be32 daddr = inet->inet_daddr;
        struct dst_entry *dst;
        struct flowi4 fl4;
        struct rtable *rt;
@@ -115,14 +113,9 @@ void ip4_datagram_release_cb(struct sock *sk)
                rcu_read_unlock();
                return;
        }
-       inet_opt = rcu_dereference(inet->inet_opt);
-       if (inet_opt && inet_opt->opt.srr)
-               daddr = inet_opt->opt.faddr;
-       rt = ip_route_output_ports(sock_net(sk), &fl4, sk, daddr,
-                                  inet->inet_saddr, inet->inet_dport,
-                                  inet->inet_sport, sk->sk_protocol,
-                                  ip_sock_rt_tos(sk), sk->sk_bound_dev_if);
 
+       inet_sk_init_flowi4(inet, &fl4);
+       rt = ip_route_output_flow(sock_net(sk), &fl4, sk);
        dst = !IS_ERR(rt) ? &rt->dst : NULL;
        sk_dst_set(sk, dst);