ipv6: exthdrs: Replace opencoded swap() implementation
authorZiyang Xuan <william.xuanziyang@huawei.com>
Mon, 7 Aug 2023 02:09:47 +0000 (10:09 +0800)
committerJakub Kicinski <kuba@kernel.org>
Tue, 8 Aug 2023 22:36:47 +0000 (15:36 -0700)
Get a coccinelle warning as follows:
net/ipv6/exthdrs.c:800:29-30: WARNING opportunity for swap()

Use swap() to replace opencoded implementation.

Signed-off-by: Ziyang Xuan <william.xuanziyang@huawei.com>
Reviewed-by: Pavan Chebbi <pavan.chebbi@broadcom.com>
Reviewed-by: Kuniyuki Iwashima <kuniyu@amazon.com>
Link: https://lore.kernel.org/r/20230807020947.1991716-1-william.xuanziyang@huawei.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
net/ipv6/exthdrs.c

index f4bfcca..4952ae7 100644 (file)
@@ -648,7 +648,6 @@ static int ipv6_rthdr_rcv(struct sk_buff *skb)
        struct inet6_dev *idev = __in6_dev_get(skb->dev);
        struct inet6_skb_parm *opt = IP6CB(skb);
        struct in6_addr *addr = NULL;
-       struct in6_addr daddr;
        int n, i;
        struct ipv6_rt_hdr *hdr;
        struct rt0_hdr *rthdr;
@@ -796,9 +795,7 @@ looped_back:
                return -1;
        }
 
-       daddr = *addr;
-       *addr = ipv6_hdr(skb)->daddr;
-       ipv6_hdr(skb)->daddr = daddr;
+       swap(*addr, ipv6_hdr(skb)->daddr);
 
        ip6_route_input(skb);
        if (skb_dst(skb)->error) {