diff options
author | Danny Kukawka <danny.kukawka@bisect.de> | 2012-02-15 01:45:40 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2012-02-15 15:34:17 -0500 |
commit | 7ce5d222190cb3ce3ae88bafde7c4fa52a5103e0 (patch) | |
tree | 2b284365b7a2e11d4ba286a589bd35bc39918805 /net/bridge/br_device.c | |
parent | f2cedb63df14342ad40a8b5b324fc5d94a60b665 (diff) |
net: use eth_hw_addr_random() and reset addr_assign_type
Use eth_hw_addr_random() instead of calling random_ether_addr()
to set addr_assign_type correctly to NET_ADDR_RANDOM.
Reset the state to NET_ADDR_PERM as soon as the MAC get
changed via .ndo_set_mac_address.
v2: adapt to renamed eth_hw_addr_random()
Signed-off-by: Danny Kukawka <danny.kukawka@bisect.de>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/bridge/br_device.c')
-rw-r--r-- | net/bridge/br_device.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/net/bridge/br_device.c b/net/bridge/br_device.c index 71773b014e0c..a157bf827d87 100644 --- a/net/bridge/br_device.c +++ b/net/bridge/br_device.c | |||
@@ -171,6 +171,7 @@ static int br_set_mac_address(struct net_device *dev, void *p) | |||
171 | 171 | ||
172 | spin_lock_bh(&br->lock); | 172 | spin_lock_bh(&br->lock); |
173 | if (compare_ether_addr(dev->dev_addr, addr->sa_data)) { | 173 | if (compare_ether_addr(dev->dev_addr, addr->sa_data)) { |
174 | dev->addr_assign_type &= ~NET_ADDR_RANDOM; | ||
174 | memcpy(dev->dev_addr, addr->sa_data, ETH_ALEN); | 175 | memcpy(dev->dev_addr, addr->sa_data, ETH_ALEN); |
175 | br_fdb_change_mac_address(br, addr->sa_data); | 176 | br_fdb_change_mac_address(br, addr->sa_data); |
176 | br_stp_change_bridge_id(br, addr->sa_data); | 177 | br_stp_change_bridge_id(br, addr->sa_data); |
@@ -334,7 +335,7 @@ void br_dev_setup(struct net_device *dev) | |||
334 | { | 335 | { |
335 | struct net_bridge *br = netdev_priv(dev); | 336 | struct net_bridge *br = netdev_priv(dev); |
336 | 337 | ||
337 | random_ether_addr(dev->dev_addr); | 338 | eth_hw_addr_random(dev); |
338 | ether_setup(dev); | 339 | ether_setup(dev); |
339 | 340 | ||
340 | dev->netdev_ops = &br_netdev_ops; | 341 | dev->netdev_ops = &br_netdev_ops; |