diff options
author | Sudip Mukherjee <sudipm.mukherjee@gmail.com> | 2015-09-03 08:00:30 +0200 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2015-09-06 02:32:53 +0200 |
commit | bd516bd1feeb3890502178a74228ec05271f2b6d (patch) | |
tree | 24e89e1951cff4c3005e89b3eba56bfd97c83165 /drivers/net/wan | |
parent | net/ipv6: Correct PIM6 mrt_lock handling (diff) | |
download | linux-bd516bd1feeb3890502178a74228ec05271f2b6d.tar.xz linux-bd516bd1feeb3890502178a74228ec05271f2b6d.zip |
net: wan: sbni: fix device usage count
dev_get_by_name() will increment the usage count if the matching device
is found. But we were not decrementing the count if we have got the
device and the device is non-active.
Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/wan')
-rw-r--r-- | drivers/net/wan/sbni.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/net/wan/sbni.c b/drivers/net/wan/sbni.c index 758c4ba1e97c..8fef8d83436d 100644 --- a/drivers/net/wan/sbni.c +++ b/drivers/net/wan/sbni.c @@ -1358,6 +1358,8 @@ sbni_ioctl( struct net_device *dev, struct ifreq *ifr, int cmd ) if( !slave_dev || !(slave_dev->flags & IFF_UP) ) { netdev_err(dev, "trying to enslave non-active device %s\n", slave_name); + if (slave_dev) + dev_put(slave_dev); return -EPERM; } |