]> www.infradead.org Git - users/dwmw2/linux.git/commitdiff
inetpeer: remove create argument of inet_getpeer_v[46]()
authorEric Dumazet <edumazet@google.com>
Sun, 15 Dec 2024 17:56:26 +0000 (17:56 +0000)
committerJakub Kicinski <kuba@kernel.org>
Wed, 18 Dec 2024 03:37:00 +0000 (19:37 -0800)
All callers of inet_getpeer_v4() and inet_getpeer_v6()
want to create an inetpeer.

Signed-off-by: Eric Dumazet <edumazet@google.com>
Link: https://patch.msgid.link/20241215175629.1248773-2-edumazet@google.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
include/net/inetpeer.h
net/ipv4/icmp.c
net/ipv4/ip_fragment.c
net/ipv4/route.c
net/ipv6/icmp.c
net/ipv6/ip6_output.c
net/ipv6/ndisc.c

index 74ff688568a0c6559946a9ae763d5c9822f1d112..6f51f81d6cb19c623e9b347dbdbbd8d849848f6e 100644 (file)
@@ -101,25 +101,24 @@ struct inet_peer *inet_getpeer(struct inet_peer_base *base,
 
 static inline struct inet_peer *inet_getpeer_v4(struct inet_peer_base *base,
                                                __be32 v4daddr,
-                                               int vif, int create)
+                                               int vif)
 {
        struct inetpeer_addr daddr;
 
        daddr.a4.addr = v4daddr;
        daddr.a4.vif = vif;
        daddr.family = AF_INET;
-       return inet_getpeer(base, &daddr, create);
+       return inet_getpeer(base, &daddr, 1);
 }
 
 static inline struct inet_peer *inet_getpeer_v6(struct inet_peer_base *base,
-                                               const struct in6_addr *v6daddr,
-                                               int create)
+                                               const struct in6_addr *v6daddr)
 {
        struct inetpeer_addr daddr;
 
        daddr.a6 = *v6daddr;
        daddr.family = AF_INET6;
-       return inet_getpeer(base, &daddr, create);
+       return inet_getpeer(base, &daddr, 1);
 }
 
 static inline int inetpeer_addr_cmp(const struct inetpeer_addr *a,
index 963a89ae9c26e8b462de57e4af981c6c46061052..5eeb9f569a706cf2766d74bcf1a667c8930804f2 100644 (file)
@@ -322,7 +322,7 @@ static bool icmpv4_xrlim_allow(struct net *net, struct rtable *rt,
                goto out;
 
        vif = l3mdev_master_ifindex(dst->dev);
-       peer = inet_getpeer_v4(net->ipv4.peers, fl4->daddr, vif, 1);
+       peer = inet_getpeer_v4(net->ipv4.peers, fl4->daddr, vif);
        rc = inet_peer_xrlim_allow(peer,
                                   READ_ONCE(net->ipv4.sysctl_icmp_ratelimit));
        if (peer)
index 07036a2943c19f13f2d6d1d77cb8123867575b50..46e1171299f22ccf0b201eabbff5d3279a0703d8 100644 (file)
@@ -89,7 +89,7 @@ static void ip4_frag_init(struct inet_frag_queue *q, const void *a)
        q->key.v4 = *key;
        qp->ecn = 0;
        qp->peer = q->fqdir->max_dist ?
-               inet_getpeer_v4(net->ipv4.peers, key->saddr, key->vif, 1) :
+               inet_getpeer_v4(net->ipv4.peers, key->saddr, key->vif) :
                NULL;
 }
 
index 0fbec350961862f76b7eab332539472fed5a5286..297a9939c6e74beffc592dbdd7266281fe842440 100644 (file)
@@ -873,7 +873,7 @@ void ip_rt_send_redirect(struct sk_buff *skb)
        rcu_read_unlock();
 
        net = dev_net(rt->dst.dev);
-       peer = inet_getpeer_v4(net->ipv4.peers, ip_hdr(skb)->saddr, vif, 1);
+       peer = inet_getpeer_v4(net->ipv4.peers, ip_hdr(skb)->saddr, vif);
        if (!peer) {
                icmp_send(skb, ICMP_REDIRECT, ICMP_REDIR_HOST,
                          rt_nexthop(rt, ip_hdr(skb)->daddr));
@@ -976,7 +976,7 @@ static int ip_error(struct sk_buff *skb)
        }
 
        peer = inet_getpeer_v4(net->ipv4.peers, ip_hdr(skb)->saddr,
-                              l3mdev_master_ifindex(skb->dev), 1);
+                              l3mdev_master_ifindex(skb->dev));
 
        send = true;
        if (peer) {
index 071b0bc1179d81b18c340ce415cef21e02a30cd7..4593e3992c67b84e3a10f30be28762974094d21f 100644 (file)
@@ -222,7 +222,7 @@ static bool icmpv6_xrlim_allow(struct sock *sk, u8 type,
                if (rt->rt6i_dst.plen < 128)
                        tmo >>= ((128 - rt->rt6i_dst.plen)>>5);
 
-               peer = inet_getpeer_v6(net->ipv6.peers, &fl6->daddr, 1);
+               peer = inet_getpeer_v6(net->ipv6.peers, &fl6->daddr);
                res = inet_peer_xrlim_allow(peer, tmo);
                if (peer)
                        inet_putpeer(peer);
index 993106876604b525e41d0bacddbe6992ca51f7b3..cdcbb3b6c5da3abed0c94d0ade8118d6799a60bc 100644 (file)
@@ -613,7 +613,7 @@ int ip6_forward(struct sk_buff *skb)
                else
                        target = &hdr->daddr;
 
-               peer = inet_getpeer_v6(net->ipv6.peers, &hdr->daddr, 1);
+               peer = inet_getpeer_v6(net->ipv6.peers, &hdr->daddr);
 
                /* Limit redirects both by destination (here)
                   and by source (inside ndisc_send_redirect)
index aba94a34867379000e958538d880799c2d0c1476..f113554d13325453cd04ce4e5686d837943e96ff 100644 (file)
@@ -1731,7 +1731,7 @@ void ndisc_send_redirect(struct sk_buff *skb, const struct in6_addr *target)
                          "Redirect: destination is not a neighbour\n");
                goto release;
        }
-       peer = inet_getpeer_v6(net->ipv6.peers, &ipv6_hdr(skb)->saddr, 1);
+       peer = inet_getpeer_v6(net->ipv6.peers, &ipv6_hdr(skb)->saddr);
        ret = inet_peer_xrlim_allow(peer, 1*HZ);
        if (peer)
                inet_putpeer(peer);