diff options
author | David Ahern <dsa@cumulusnetworks.com> | 2016-11-07 15:03:09 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2016-11-09 18:49:39 -0500 |
commit | 9d1a6c4ea43e48c7880c85971c17939b56832d8a (patch) | |
tree | db094ff8fb661c8925a895b1eaae48afcf082cf9 /net/ipv4/icmp.c | |
parent | fd6f24d75ab07fd122c2910d343caa703f712873 (diff) |
net: icmp_route_lookup should use rt dev to determine L3 domain
icmp_send is called in response to some event. The skb may not have
the device set (skb->dev is NULL), but it is expected to have an rt.
Update icmp_route_lookup to use the rt on the skb to determine L3
domain.
Fixes: 613d09b30f8b ("net: Use VRF device index for lookups on TX")
Signed-off-by: David Ahern <dsa@cumulusnetworks.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ipv4/icmp.c')
-rw-r--r-- | net/ipv4/icmp.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/net/ipv4/icmp.c b/net/ipv4/icmp.c index 38abe70e595f..48734ee6293f 100644 --- a/net/ipv4/icmp.c +++ b/net/ipv4/icmp.c | |||
@@ -477,7 +477,7 @@ static struct rtable *icmp_route_lookup(struct net *net, | |||
477 | fl4->flowi4_proto = IPPROTO_ICMP; | 477 | fl4->flowi4_proto = IPPROTO_ICMP; |
478 | fl4->fl4_icmp_type = type; | 478 | fl4->fl4_icmp_type = type; |
479 | fl4->fl4_icmp_code = code; | 479 | fl4->fl4_icmp_code = code; |
480 | fl4->flowi4_oif = l3mdev_master_ifindex(skb_in->dev); | 480 | fl4->flowi4_oif = l3mdev_master_ifindex(skb_dst(skb_in)->dev); |
481 | 481 | ||
482 | security_skb_classify_flow(skb_in, flowi4_to_flowi(fl4)); | 482 | security_skb_classify_flow(skb_in, flowi4_to_flowi(fl4)); |
483 | rt = __ip_route_output_key_hash(net, fl4, | 483 | rt = __ip_route_output_key_hash(net, fl4, |
@@ -502,7 +502,7 @@ static struct rtable *icmp_route_lookup(struct net *net, | |||
502 | if (err) | 502 | if (err) |
503 | goto relookup_failed; | 503 | goto relookup_failed; |
504 | 504 | ||
505 | if (inet_addr_type_dev_table(net, skb_in->dev, | 505 | if (inet_addr_type_dev_table(net, skb_dst(skb_in)->dev, |
506 | fl4_dec.saddr) == RTN_LOCAL) { | 506 | fl4_dec.saddr) == RTN_LOCAL) { |
507 | rt2 = __ip_route_output_key(net, &fl4_dec); | 507 | rt2 = __ip_route_output_key(net, &fl4_dec); |
508 | if (IS_ERR(rt2)) | 508 | if (IS_ERR(rt2)) |