Commit a21090cf authored by Stephen Hemminger's avatar Stephen Hemminger Committed by David S. Miller

ipv4: arp_notify address list bug

This fixes a bug with arp_notify.

If arp_notify is enabled, kernel will crash if address is changed
and no IP address is assigned.
  http://bugzilla.kernel.org/show_bug.cgi?id=14330Reported-by: default avatarHannes Frederic Sowa <hannes@stressinduktion.org>
Signed-off-by: default avatarStephen Hemminger <shemminger@vyatta.com>
Signed-off-by: default avatarEric Dumazet <eric.dumazet@gmail.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 1cec9727
...@@ -1077,12 +1077,16 @@ static int inetdev_event(struct notifier_block *this, unsigned long event, ...@@ -1077,12 +1077,16 @@ static int inetdev_event(struct notifier_block *this, unsigned long event,
ip_mc_up(in_dev); ip_mc_up(in_dev);
/* fall through */ /* fall through */
case NETDEV_CHANGEADDR: case NETDEV_CHANGEADDR:
if (IN_DEV_ARP_NOTIFY(in_dev)) /* Send gratuitous ARP to notify of link change */
arp_send(ARPOP_REQUEST, ETH_P_ARP, if (IN_DEV_ARP_NOTIFY(in_dev)) {
in_dev->ifa_list->ifa_address, struct in_ifaddr *ifa = in_dev->ifa_list;
dev,
in_dev->ifa_list->ifa_address, if (ifa)
NULL, dev->dev_addr, NULL); arp_send(ARPOP_REQUEST, ETH_P_ARP,
ifa->ifa_address, dev,
ifa->ifa_address, NULL,
dev->dev_addr, NULL);
}
break; break;
case NETDEV_DOWN: case NETDEV_DOWN:
ip_mc_down(in_dev); ip_mc_down(in_dev);
......
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