Commit 32c10bbf authored by Mahesh Bandewar's avatar Mahesh Bandewar Committed by David S. Miller

ipvlan: always use the current L2 addr of the master

If the underlying master ever changes its L2 (e.g. bonding device),
then make sure that the IPvlan slaves always emit packets with the
current L2 of the master instead of the stale mac addr which was
copied during the device creation. The problem can be seen with
following script -

  #!/bin/bash
  # Create a vEth pair
  ip link add dev veth0 type veth peer name veth1
  ip link set veth0 up
  ip link set veth1 up
  ip link show veth0
  ip link show veth1
  # Create an IPvlan device on one end of this vEth pair.
  ip link add link veth0 dev ipvl0 type ipvlan mode l2
  ip link show ipvl0
  # Change the mac-address of the vEth master.
  ip link set veth0 address 02:11:22:33:44:55

Fixes: 2ad7bf36 ("ipvlan: Initial check-in of the IPVLAN driver.")
Signed-off-by: default avatarMahesh Bandewar <maheshb@google.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 743b8bb6
...@@ -407,7 +407,7 @@ static int ipvlan_hard_header(struct sk_buff *skb, struct net_device *dev, ...@@ -407,7 +407,7 @@ static int ipvlan_hard_header(struct sk_buff *skb, struct net_device *dev,
* while the packets use the mac-addr on the physical device. * while the packets use the mac-addr on the physical device.
*/ */
return dev_hard_header(skb, phy_dev, type, daddr, return dev_hard_header(skb, phy_dev, type, daddr,
saddr ? : dev->dev_addr, len); saddr ? : phy_dev->dev_addr, len);
} }
static const struct header_ops ipvlan_header_ops = { static const struct header_ops ipvlan_header_ops = {
...@@ -730,6 +730,11 @@ static int ipvlan_device_event(struct notifier_block *unused, ...@@ -730,6 +730,11 @@ static int ipvlan_device_event(struct notifier_block *unused,
ipvlan_adjust_mtu(ipvlan, dev); ipvlan_adjust_mtu(ipvlan, dev);
break; break;
case NETDEV_CHANGEADDR:
list_for_each_entry(ipvlan, &port->ipvlans, pnode)
ether_addr_copy(ipvlan->dev->dev_addr, dev->dev_addr);
break;
case NETDEV_PRE_TYPE_CHANGE: case NETDEV_PRE_TYPE_CHANGE:
/* Forbid underlying device to change its type. */ /* Forbid underlying device to change its type. */
return NOTIFY_BAD; return NOTIFY_BAD;
......
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