diff options
| author | Dan Carpenter <error27@gmail.com> | 2011-06-11 18:55:22 -0700 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2011-06-11 18:55:22 -0700 |
| commit | 83fe32de63e60af34fa8dae83716cb13b8677abd (patch) | |
| tree | e4f7526b481ac06bdb7fb6712d30a84ac64aa7ab /net/core | |
| parent | f1c089e3192f1afdfa76226dc38ef81b08ac810d (diff) | |
| download | lwn-83fe32de63e60af34fa8dae83716cb13b8677abd.tar.gz lwn-83fe32de63e60af34fa8dae83716cb13b8677abd.zip | |
netpoll: call dev_put() on error in netpoll_setup()
There is a dev_put(ndev) missing on an error path. This was
introduced in 0c1ad04aecb "netpoll: prevent netpoll setup on slave
devices".
Signed-off-by: Dan Carpenter <error27@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/core')
| -rw-r--r-- | net/core/netpoll.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/net/core/netpoll.c b/net/core/netpoll.c index 42ea4b0e59f1..18d9cbda3a39 100644 --- a/net/core/netpoll.c +++ b/net/core/netpoll.c @@ -795,7 +795,8 @@ int netpoll_setup(struct netpoll *np) if (ndev->master) { printk(KERN_ERR "%s: %s is a slave device, aborting.\n", np->name, np->dev_name); - return -EBUSY; + err = -EBUSY; + goto put; } if (!netif_running(ndev)) { |