diff options
author | Yuya Tajima <yuya.tajimaa@gmail.com> | 2023-05-15 15:34:27 +0000 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2023-05-17 09:05:47 +0100 |
commit | fa0583c202433c3d359a1b0579f52da16e25e4df (patch) | |
tree | 5cefd694152d86f670848865b92e2289f5fe474b /net/ipv6/exthdrs.c | |
parent | 81cf1ade0bb3a42023dd1694ed080c4adb7dafd3 (diff) | |
download | lwn-fa0583c202433c3d359a1b0579f52da16e25e4df.tar.gz lwn-fa0583c202433c3d359a1b0579f52da16e25e4df.zip |
seg6: Cleanup duplicates of skb_dst_drop calls
In processing IPv6 segment routing header (SRH), several functions call
skb_dst_drop before ip6_route_input. However, ip6_route_input calls
skb_dst_drop within it, so there is no need to call skb_dst_drop in advance.
Signed-off-by: Yuya Tajima <yuya.tajimaa@gmail.com>
Reviewed-by: Simon Horman <simon.horman@corigine.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ipv6/exthdrs.c')
-rw-r--r-- | net/ipv6/exthdrs.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/net/ipv6/exthdrs.c b/net/ipv6/exthdrs.c index a8d961d3a477..04c14fc4b14d 100644 --- a/net/ipv6/exthdrs.c +++ b/net/ipv6/exthdrs.c @@ -458,8 +458,6 @@ looped_back: ipv6_hdr(skb)->daddr = *addr; - skb_dst_drop(skb); - ip6_route_input(skb); if (skb_dst(skb)->error) { @@ -834,7 +832,6 @@ looped_back: *addr = ipv6_hdr(skb)->daddr; ipv6_hdr(skb)->daddr = daddr; - skb_dst_drop(skb); ip6_route_input(skb); if (skb_dst(skb)->error) { skb_push(skb, skb->data - skb_network_header(skb)); |