diff options
author | Herbert Xu <herbert@gondor.apana.org.au> | 2007-07-30 20:04:52 -0400 |
---|---|---|
committer | David S. Miller <davem@sunset.davemloft.net> | 2007-07-31 05:28:16 -0400 |
commit | b217d616a15fcfb3caf2a72c1a071c6d3f182f8d (patch) | |
tree | 301422d7c0fba653dfb5e32c80fedc2cede627e3 /net/ipv6/addrconf.c | |
parent | fcc5a03ac42564e9e255c1134dda47442289e466 (diff) |
[IPV4/IPV6]: Fail registration if inet device construction fails
Now that netdev notifications can fail, we can use this to signal
errors during registration for IPv4/IPv6. In particular, if we
fail to allocate memory for the inet device, we can fail the netdev
registration.
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ipv6/addrconf.c')
-rw-r--r-- | net/ipv6/addrconf.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/net/ipv6/addrconf.c b/net/ipv6/addrconf.c index 06012920912a..91ef3be5abad 100644 --- a/net/ipv6/addrconf.c +++ b/net/ipv6/addrconf.c | |||
@@ -2256,14 +2256,14 @@ static int addrconf_notify(struct notifier_block *this, unsigned long event, | |||
2256 | struct net_device *dev = (struct net_device *) data; | 2256 | struct net_device *dev = (struct net_device *) data; |
2257 | struct inet6_dev *idev = __in6_dev_get(dev); | 2257 | struct inet6_dev *idev = __in6_dev_get(dev); |
2258 | int run_pending = 0; | 2258 | int run_pending = 0; |
2259 | int err; | ||
2259 | 2260 | ||
2260 | switch(event) { | 2261 | switch(event) { |
2261 | case NETDEV_REGISTER: | 2262 | case NETDEV_REGISTER: |
2262 | if (!idev && dev->mtu >= IPV6_MIN_MTU) { | 2263 | if (!idev && dev->mtu >= IPV6_MIN_MTU) { |
2263 | idev = ipv6_add_dev(dev); | 2264 | idev = ipv6_add_dev(dev); |
2264 | if (!idev) | 2265 | if (!idev) |
2265 | printk(KERN_WARNING "IPv6: add_dev failed for %s\n", | 2266 | return notifier_from_errno(-ENOMEM); |
2266 | dev->name); | ||
2267 | } | 2267 | } |
2268 | break; | 2268 | break; |
2269 | case NETDEV_UP: | 2269 | case NETDEV_UP: |
@@ -2373,7 +2373,9 @@ static int addrconf_notify(struct notifier_block *this, unsigned long event, | |||
2373 | NULL); | 2373 | NULL); |
2374 | addrconf_sysctl_register(idev, &idev->cnf); | 2374 | addrconf_sysctl_register(idev, &idev->cnf); |
2375 | #endif | 2375 | #endif |
2376 | snmp6_register_dev(idev); | 2376 | err = snmp6_register_dev(idev); |
2377 | if (err) | ||
2378 | return notifier_from_errno(err); | ||
2377 | } | 2379 | } |
2378 | break; | 2380 | break; |
2379 | } | 2381 | } |