Commit c4fceb46 authored by Eric Dumazet's avatar Eric Dumazet Committed by Jakub Kicinski
Browse files

raw: remove unused variables from raw6_icmp_error() 



saddr and daddr are set but not used.

Fixes: ba44f818 ("raw: use more conventional iterators")
Reported-by: default avatarkernel test robot <lkp@intel.com>
Signed-off-by: default avatarEric Dumazet <edumazet@google.com>
Acked-by: default avatarJonathan Lemon <jonathan.lemon@gmail.com>
Link: https://lore.kernel.org/r/20220622032303.159394-1-edumazet@google.com


Signed-off-by: default avatarJakub Kicinski <kuba@kernel.org>
parent 12cf1b89

net/ipv6/raw.c

+0 −3
Original line number Diff line number Diff line
@@ -332,7 +332,6 @@ static void rawv6_err(struct sock *sk, struct sk_buff *skb, 
void raw6_icmp_error(struct sk_buff *skb, int nexthdr,

		u8 type, u8 code, int inner_offset, __be32 info)

{

	const struct in6_addr *saddr, *daddr;

	struct net *net = dev_net(skb->dev);

	struct hlist_nulls_head *hlist;

	struct hlist_nulls_node *hnode;
@@ -345,8 +344,6 @@ void raw6_icmp_error(struct sk_buff *skb, int nexthdr, 
	sk_nulls_for_each(sk, hnode, hlist) {

		/* Note: ipv6_hdr(skb) != skb->data */

		const struct ipv6hdr *ip6h = (const struct ipv6hdr *)skb->data;

		saddr = &ip6h->saddr;

		daddr = &ip6h->daddr;



		if (!raw_v6_match(net, sk, nexthdr, &ip6h->saddr, &ip6h->daddr,

				  inet6_iif(skb), inet6_iif(skb)))