Commit f002d840 authored by Hideaki Yoshifuji's avatar Hideaki Yoshifuji

[IPV6]: Use cheaper ipv6_addr_any() where appropriate.

parent 13ef52ad
...@@ -709,7 +709,7 @@ static void ndisc_recv_ns(struct sk_buff *skb) ...@@ -709,7 +709,7 @@ static void ndisc_recv_ns(struct sk_buff *skb)
struct inet6_ifaddr *ifp; struct inet6_ifaddr *ifp;
struct inet6_dev *idev = NULL; struct inet6_dev *idev = NULL;
struct neighbour *neigh; struct neighbour *neigh;
int addr_type = ipv6_addr_type(saddr); int dad = ipv6_addr_any(saddr);
int inc; int inc;
if (ipv6_addr_is_multicast(&msg->target)) { if (ipv6_addr_is_multicast(&msg->target)) {
...@@ -722,7 +722,7 @@ static void ndisc_recv_ns(struct sk_buff *skb) ...@@ -722,7 +722,7 @@ static void ndisc_recv_ns(struct sk_buff *skb)
* RFC2461 7.1.1: * RFC2461 7.1.1:
* DAD has to be destined for solicited node multicast address. * DAD has to be destined for solicited node multicast address.
*/ */
if (addr_type == IPV6_ADDR_ANY && if (dad &&
!(daddr->s6_addr32[0] == htonl(0xff020000) && !(daddr->s6_addr32[0] == htonl(0xff020000) &&
daddr->s6_addr32[1] == htonl(0x00000000) && daddr->s6_addr32[1] == htonl(0x00000000) &&
daddr->s6_addr32[2] == htonl(0x00000001) && daddr->s6_addr32[2] == htonl(0x00000001) &&
...@@ -752,7 +752,7 @@ static void ndisc_recv_ns(struct sk_buff *skb) ...@@ -752,7 +752,7 @@ static void ndisc_recv_ns(struct sk_buff *skb)
* there MUST NOT be source link-layer address option * there MUST NOT be source link-layer address option
* in the message. * in the message.
*/ */
if (addr_type == IPV6_ADDR_ANY) { if (dad) {
if (net_ratelimit()) if (net_ratelimit())
printk(KERN_WARNING "ICMP6 NS: bad DAD packet (link-layer address option)\n"); printk(KERN_WARNING "ICMP6 NS: bad DAD packet (link-layer address option)\n");
return; return;
...@@ -768,10 +768,8 @@ static void ndisc_recv_ns(struct sk_buff *skb) ...@@ -768,10 +768,8 @@ static void ndisc_recv_ns(struct sk_buff *skb)
does DAD, otherwise we ignore solicitations does DAD, otherwise we ignore solicitations
until DAD timer expires. until DAD timer expires.
*/ */
if (addr_type != IPV6_ADDR_ANY) { if (!dad)
in6_ifa_put(ifp); goto out;
return;
}
if (dev->type == ARPHRD_IEEE802_TR) { if (dev->type == ARPHRD_IEEE802_TR) {
unsigned char *sadr = skb->mac.raw; unsigned char *sadr = skb->mac.raw;
if (((sadr[8] ^ dev->dev_addr[0]) & 0x7f) == 0 && if (((sadr[8] ^ dev->dev_addr[0]) & 0x7f) == 0 &&
...@@ -781,8 +779,7 @@ static void ndisc_recv_ns(struct sk_buff *skb) ...@@ -781,8 +779,7 @@ static void ndisc_recv_ns(struct sk_buff *skb)
sadr[12] == dev->dev_addr[4] && sadr[12] == dev->dev_addr[4] &&
sadr[13] == dev->dev_addr[5]) { sadr[13] == dev->dev_addr[5]) {
/* looped-back to us */ /* looped-back to us */
in6_ifa_put(ifp); goto out;
return;
} }
} }
addrconf_dad_failure(ifp); addrconf_dad_failure(ifp);
...@@ -820,7 +817,7 @@ static void ndisc_recv_ns(struct sk_buff *skb) ...@@ -820,7 +817,7 @@ static void ndisc_recv_ns(struct sk_buff *skb)
goto out; goto out;
} }
if (addr_type == IPV6_ADDR_ANY) { if (dad) {
struct in6_addr maddr; struct in6_addr maddr;
ipv6_addr_all_nodes(&maddr); ipv6_addr_all_nodes(&maddr);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment