diff options
| author |   Dan Carpenter <error27@gmail.com> | 2010-07-28 22:27:29 +0000 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2010-07-30 23:27:58 -0700 |
| commit | de140b0d511ad86a5dd0888c9095ae030ada2e86 (patch) | |
| tree | 29a2fb4db71c3ffc9ad78a9ce8e3f67d5e572600 /drivers/net/dnet.c | |
| parent | bonding: prevent sysfs from allowing arp monitoring with alb/tlb (diff) | |
| download | linux-dev-de140b0d511ad86a5dd0888c9095ae030ada2e86.tar.xz linux-dev-de140b0d511ad86a5dd0888c9095ae030ada2e86.zip | |
dnet: fixup error handling in initialization
There were two problems here. We returned success if dnet_mii_init()
failed and there was a release_mem_region() missing.
Signed-off-by: Dan Carpenter <error27@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/dnet.c')
| -rw-r--r-- | drivers/net/dnet.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/drivers/net/dnet.c b/drivers/net/dnet.c index 4ea7141f525d..7c075756611a 100644 --- a/drivers/net/dnet.c +++ b/drivers/net/dnet.c @@ -854,7 +854,7 @@ static int __devinit dnet_probe(struct platform_device *pdev) dev = alloc_etherdev(sizeof(*bp)); if (!dev) { dev_err(&pdev->dev, "etherdev alloc failed, aborting.\n"); - goto err_out; + goto err_out_release_mem; } /* TODO: Actually, we have some interesting features... */ @@ -911,7 +911,8 @@ static int __devinit dnet_probe(struct platform_device *pdev) if (err) dev_warn(&pdev->dev, "Cannot register PHY board fixup.\n"); - if (dnet_mii_init(bp) != 0) + err = dnet_mii_init(bp); + if (err) goto err_out_unregister_netdev; dev_info(&pdev->dev, "Dave DNET at 0x%p (0x%08x) irq %d %pM\n", @@ -936,6 +937,8 @@ err_out_iounmap: iounmap(bp->regs); err_out_free_dev: free_netdev(dev); +err_out_release_mem: + release_mem_region(mem_base, mem_size); err_out: return err; } |