diff mbox

[net-next] ipv4: loopback device: ignore value changes after device is upped

Message ID 1389106545-7940-1-git-send-email-jiri@resnulli.us
State Accepted, archived
Delegated to: David Miller
Headers show

Commit Message

Jiri Pirko Jan. 7, 2014, 2:55 p.m. UTC
When lo is brought up, new ifa is created. Then, devconf and neigh values
bitfield should be set so later changes of default values would not
affect lo values.

Note that the same behaviour is in ipv6. Also note that this is likely
not an issue in many distros (for example Fedora 19) because userspace
sets address to lo manually before bringing it up.

Signed-off-by: Jiri Pirko <jiri@resnulli.us>
---
 net/ipv4/devinet.c | 2 ++
 1 file changed, 2 insertions(+)

Comments

David Miller Jan. 7, 2014, 8:55 p.m. UTC | #1
From: Jiri Pirko <jiri@resnulli.us>
Date: Tue,  7 Jan 2014 15:55:45 +0100

> When lo is brought up, new ifa is created. Then, devconf and neigh values
> bitfield should be set so later changes of default values would not
> affect lo values.
> 
> Note that the same behaviour is in ipv6. Also note that this is likely
> not an issue in many distros (for example Fedora 19) because userspace
> sets address to lo manually before bringing it up.
> 
> Signed-off-by: Jiri Pirko <jiri@resnulli.us>

Looks good, applied.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/net/ipv4/devinet.c b/net/ipv4/devinet.c
index 0feebd5..9809f7b 100644
--- a/net/ipv4/devinet.c
+++ b/net/ipv4/devinet.c
@@ -1385,6 +1385,8 @@  static int inetdev_event(struct notifier_block *this, unsigned long event,
 				memcpy(ifa->ifa_label, dev->name, IFNAMSIZ);
 				set_ifa_lifetime(ifa, INFINITY_LIFE_TIME,
 						 INFINITY_LIFE_TIME);
+				ipv4_devconf_setall(in_dev);
+				neigh_parms_data_state_setall(in_dev->arp_parms);
 				inet_insert_ifa(ifa);
 			}
 		}