diff options
author | <herbert@gondor.apana.org.au> | 2005-05-12 20:11:55 -0400 |
---|---|---|
committer | Jeff Garzik <jgarzik@pobox.com> | 2005-05-12 20:11:55 -0400 |
commit | b1fc5505e0dbcc3fd7c75bfe6bee39ec50080963 (patch) | |
tree | f8c0bb679dc8e72816e16820c3dbcccf924097eb /drivers/net/lance.c | |
parent | 88d7bd8cb9eb8d64bf7997600b0d64f7834047c5 (diff) |
[netdrvr] Fix register_netdev() races in older ISA net drivers
Diffstat (limited to 'drivers/net/lance.c')
-rw-r--r-- | drivers/net/lance.c | 15 |
1 files changed, 5 insertions, 10 deletions
diff --git a/drivers/net/lance.c b/drivers/net/lance.c index dec557fb6a99..ca90f0d1e4b0 100644 --- a/drivers/net/lance.c +++ b/drivers/net/lance.c | |||
@@ -356,11 +356,8 @@ int init_module(void) | |||
356 | dev->base_addr = io[this_dev]; | 356 | dev->base_addr = io[this_dev]; |
357 | dev->dma = dma[this_dev]; | 357 | dev->dma = dma[this_dev]; |
358 | if (do_lance_probe(dev) == 0) { | 358 | if (do_lance_probe(dev) == 0) { |
359 | if (register_netdev(dev) == 0) { | 359 | dev_lance[found++] = dev; |
360 | dev_lance[found++] = dev; | 360 | continue; |
361 | continue; | ||
362 | } | ||
363 | cleanup_card(dev); | ||
364 | } | 361 | } |
365 | free_netdev(dev); | 362 | free_netdev(dev); |
366 | break; | 363 | break; |
@@ -448,12 +445,7 @@ struct net_device * __init lance_probe(int unit) | |||
448 | err = do_lance_probe(dev); | 445 | err = do_lance_probe(dev); |
449 | if (err) | 446 | if (err) |
450 | goto out; | 447 | goto out; |
451 | err = register_netdev(dev); | ||
452 | if (err) | ||
453 | goto out1; | ||
454 | return dev; | 448 | return dev; |
455 | out1: | ||
456 | cleanup_card(dev); | ||
457 | out: | 449 | out: |
458 | free_netdev(dev); | 450 | free_netdev(dev); |
459 | return ERR_PTR(err); | 451 | return ERR_PTR(err); |
@@ -724,6 +716,9 @@ static int __init lance_probe1(struct net_device *dev, int ioaddr, int irq, int | |||
724 | dev->tx_timeout = lance_tx_timeout; | 716 | dev->tx_timeout = lance_tx_timeout; |
725 | dev->watchdog_timeo = TX_TIMEOUT; | 717 | dev->watchdog_timeo = TX_TIMEOUT; |
726 | 718 | ||
719 | err = register_netdev(dev); | ||
720 | if (err) | ||
721 | goto out_dma; | ||
727 | return 0; | 722 | return 0; |
728 | out_dma: | 723 | out_dma: |
729 | if (dev->dma != 4) | 724 | if (dev->dma != 4) |