diff options
author | Denis V. Lunev <den@openvz.org> | 2008-01-15 02:05:55 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2008-01-28 18:02:12 -0500 |
commit | 9bd85e32644d4d3744117b0a196ad4382f8acf35 (patch) | |
tree | d25bb624d799e787f9de589f317001ee8b2357aa /net/ipv4/arp.c | |
parent | 06f0511df1b3b32fc8e0840514d4b207150f1fa7 (diff) |
[IPV4]: Remove extra argument from arp_ignore.
arp_ignore has two arguments: dev & in_dev. dev is used for
inet_confirm_addr calling only.
inet_confirm_addr, in turn, either gets in_dev from the device passed
or iterates over all network devices if the device passed is NULL. It
seems logical to directly pass in_dev into inet_confirm_addr.
Signed-off-by: Denis V. Lunev <den@openvz.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ipv4/arp.c')
-rw-r--r-- | net/ipv4/arp.c | 11 |
1 files changed, 5 insertions, 6 deletions
diff --git a/net/ipv4/arp.c b/net/ipv4/arp.c index 357e8987146b..6f0827b2b154 100644 --- a/net/ipv4/arp.c +++ b/net/ipv4/arp.c | |||
@@ -382,8 +382,7 @@ static void arp_solicit(struct neighbour *neigh, struct sk_buff *skb) | |||
382 | read_unlock_bh(&neigh->lock); | 382 | read_unlock_bh(&neigh->lock); |
383 | } | 383 | } |
384 | 384 | ||
385 | static int arp_ignore(struct in_device *in_dev, struct net_device *dev, | 385 | static int arp_ignore(struct in_device *in_dev, __be32 sip, __be32 tip) |
386 | __be32 sip, __be32 tip) | ||
387 | { | 386 | { |
388 | int scope; | 387 | int scope; |
389 | 388 | ||
@@ -403,7 +402,7 @@ static int arp_ignore(struct in_device *in_dev, struct net_device *dev, | |||
403 | case 3: /* Do not reply for scope host addresses */ | 402 | case 3: /* Do not reply for scope host addresses */ |
404 | sip = 0; | 403 | sip = 0; |
405 | scope = RT_SCOPE_LINK; | 404 | scope = RT_SCOPE_LINK; |
406 | dev = NULL; | 405 | in_dev = NULL; |
407 | break; | 406 | break; |
408 | case 4: /* Reserved */ | 407 | case 4: /* Reserved */ |
409 | case 5: | 408 | case 5: |
@@ -415,7 +414,7 @@ static int arp_ignore(struct in_device *in_dev, struct net_device *dev, | |||
415 | default: | 414 | default: |
416 | return 0; | 415 | return 0; |
417 | } | 416 | } |
418 | return !inet_confirm_addr(dev, sip, tip, scope); | 417 | return !inet_confirm_addr(in_dev, sip, tip, scope); |
419 | } | 418 | } |
420 | 419 | ||
421 | static int arp_filter(__be32 sip, __be32 tip, struct net_device *dev) | 420 | static int arp_filter(__be32 sip, __be32 tip, struct net_device *dev) |
@@ -807,7 +806,7 @@ static int arp_process(struct sk_buff *skb) | |||
807 | if (sip == 0) { | 806 | if (sip == 0) { |
808 | if (arp->ar_op == htons(ARPOP_REQUEST) && | 807 | if (arp->ar_op == htons(ARPOP_REQUEST) && |
809 | inet_addr_type(&init_net, tip) == RTN_LOCAL && | 808 | inet_addr_type(&init_net, tip) == RTN_LOCAL && |
810 | !arp_ignore(in_dev,dev,sip,tip)) | 809 | !arp_ignore(in_dev, sip, tip)) |
811 | arp_send(ARPOP_REPLY, ETH_P_ARP, sip, dev, tip, sha, | 810 | arp_send(ARPOP_REPLY, ETH_P_ARP, sip, dev, tip, sha, |
812 | dev->dev_addr, sha); | 811 | dev->dev_addr, sha); |
813 | goto out; | 812 | goto out; |
@@ -825,7 +824,7 @@ static int arp_process(struct sk_buff *skb) | |||
825 | int dont_send = 0; | 824 | int dont_send = 0; |
826 | 825 | ||
827 | if (!dont_send) | 826 | if (!dont_send) |
828 | dont_send |= arp_ignore(in_dev,dev,sip,tip); | 827 | dont_send |= arp_ignore(in_dev,sip,tip); |
829 | if (!dont_send && IN_DEV_ARPFILTER(in_dev)) | 828 | if (!dont_send && IN_DEV_ARPFILTER(in_dev)) |
830 | dont_send |= arp_filter(sip,tip,dev); | 829 | dont_send |= arp_filter(sip,tip,dev); |
831 | if (!dont_send) | 830 | if (!dont_send) |