diff options
author | Dan Carpenter <error27@gmail.com> | 2011-06-11 21:55:22 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2011-06-11 21:55:22 -0400 |
commit | 83fe32de63e60af34fa8dae83716cb13b8677abd (patch) | |
tree | e4f7526b481ac06bdb7fb6712d30a84ac64aa7ab /net/core/netpoll.c | |
parent | f1c089e3192f1afdfa76226dc38ef81b08ac810d (diff) |
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/netpoll.c')
-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) | |||
795 | if (ndev->master) { | 795 | if (ndev->master) { |
796 | printk(KERN_ERR "%s: %s is a slave device, aborting.\n", | 796 | printk(KERN_ERR "%s: %s is a slave device, aborting.\n", |
797 | np->name, np->dev_name); | 797 | np->name, np->dev_name); |
798 | return -EBUSY; | 798 | err = -EBUSY; |
799 | goto put; | ||
799 | } | 800 | } |
800 | 801 | ||
801 | if (!netif_running(ndev)) { | 802 | if (!netif_running(ndev)) { |