diff options
author | Eric W. Biederman <ebiederm@xmission.com> | 2007-09-27 01:10:06 -0400 |
---|---|---|
committer | David S. Miller <davem@sunset.davemloft.net> | 2007-10-10 19:52:48 -0400 |
commit | 0cc217e16cb8ca8ef2544363571fce94259900e0 (patch) | |
tree | 05d7ef10fd9971fffa3f3870a4c3e66077bb1dc2 /net/ipv4/devinet.c | |
parent | 5967789dbc8aafdba5813fa8e8cfce3c90516f83 (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.c | 4 |
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 = |