aboutsummaryrefslogtreecommitdiffstats
path: root/net
diff options
context:
space:
mode:
authorMark McLoughlin <markmc@redhat.com>2008-01-04 03:56:25 -0500
committerDavid S. Miller <davem@davemloft.net>2008-01-04 06:55:34 -0500
commit44344b2a85f03326c7047a8c861b0c625c674839 (patch)
treefe578496a5c71a74500260a5f9bcced8d5299f22 /net
parent14be85f555fa53a3b80119569e4c2fe88aa9fd3e (diff)
[INET]: Fix netdev renaming and inet address labels
When re-naming an interface, the previous secondary address labels get lost e.g. $> brctl addbr foo $> ip addr add 192.168.0.1 dev foo $> ip addr add 192.168.0.2 dev foo label foo:00 $> ip addr show dev foo | grep inet inet 192.168.0.1/32 scope global foo inet 192.168.0.2/32 scope global foo:00 $> ip link set foo name bar $> ip addr show dev bar | grep inet inet 192.168.0.1/32 scope global bar inet 192.168.0.2/32 scope global bar:2 Turns out to be a simple thinko in inetdev_changename() - clearly we want to look at the address label, rather than the device name, for a suffix to retain. Signed-off-by: Mark McLoughlin <markmc@redhat.com> Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net')
-rw-r--r--net/ipv4/devinet.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/net/ipv4/devinet.c b/net/ipv4/devinet.c
index 3168c3de4919..b42f74617bac 100644
--- a/net/ipv4/devinet.c
+++ b/net/ipv4/devinet.c
@@ -1027,7 +1027,7 @@ static void inetdev_changename(struct net_device *dev, struct in_device *in_dev)
1027 memcpy(ifa->ifa_label, dev->name, IFNAMSIZ); 1027 memcpy(ifa->ifa_label, dev->name, IFNAMSIZ);
1028 if (named++ == 0) 1028 if (named++ == 0)
1029 continue; 1029 continue;
1030 dot = strchr(ifa->ifa_label, ':'); 1030 dot = strchr(old, ':');
1031 if (dot == NULL) { 1031 if (dot == NULL) {
1032 sprintf(old, ":%d", named); 1032 sprintf(old, ":%d", named);
1033 dot = old; 1033 dot = old;