aboutsummaryrefslogtreecommitdiffstats
path: root/net/ipv4/devinet.c
diff options
context:
space:
mode:
authorEric W. Biederman <ebiederm@xmission.com>2007-09-27 01:10:06 -0400
committerDavid S. Miller <davem@sunset.davemloft.net>2007-10-10 19:52:48 -0400
commit0cc217e16cb8ca8ef2544363571fce94259900e0 (patch)
tree05d7ef10fd9971fffa3f3870a4c3e66077bb1dc2 /net/ipv4/devinet.c
parent5967789dbc8aafdba5813fa8e8cfce3c90516f83 (diff)
[IPV4]: When possible test for IFF_LOOPBACK and not dev == loopback_dev
Now that multiple loopback devices are becoming possible it makes the code a little cleaner and more maintainable to test if a deivice is th a loopback device by testing dev->flags & IFF_LOOPBACK instead of dev == loopback_dev. Signed-off-by: Eric W. Biederman <ebiederm@xmission.com> Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ipv4/devinet.c')
-rw-r--r--net/ipv4/devinet.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/net/ipv4/devinet.c b/net/ipv4/devinet.c
index e7f2b022316e..55d199e4ae21 100644
--- a/net/ipv4/devinet.c
+++ b/net/ipv4/devinet.c
@@ -1059,7 +1059,7 @@ static int inetdev_event(struct notifier_block *this, unsigned long event,
1059 in_dev = inetdev_init(dev); 1059 in_dev = inetdev_init(dev);
1060 if (!in_dev) 1060 if (!in_dev)
1061 return notifier_from_errno(-ENOMEM); 1061 return notifier_from_errno(-ENOMEM);
1062 if (dev == loopback_dev) { 1062 if (dev->flags & IFF_LOOPBACK) {
1063 IN_DEV_CONF_SET(in_dev, NOXFRM, 1); 1063 IN_DEV_CONF_SET(in_dev, NOXFRM, 1);
1064 IN_DEV_CONF_SET(in_dev, NOPOLICY, 1); 1064 IN_DEV_CONF_SET(in_dev, NOPOLICY, 1);
1065 } 1065 }
@@ -1075,7 +1075,7 @@ static int inetdev_event(struct notifier_block *this, unsigned long event,
1075 case NETDEV_UP: 1075 case NETDEV_UP:
1076 if (dev->mtu < 68) 1076 if (dev->mtu < 68)
1077 break; 1077 break;
1078 if (dev == loopback_dev) { 1078 if (dev->flags & IFF_LOOPBACK) {
1079 struct in_ifaddr *ifa; 1079 struct in_ifaddr *ifa;
1080 if ((ifa = inet_alloc_ifa()) != NULL) { 1080 if ((ifa = inet_alloc_ifa()) != NULL) {
1081 ifa->ifa_local = 1081 ifa->ifa_local =