summaryrefslogtreecommitdiffstats
path: root/net
diff options
context:
space:
mode:
authorAl Viro <viro@zeniv.linux.org.uk>2007-09-16 16:43:04 -0700
committerDavid S. Miller <davem@davemloft.net>2007-09-16 16:43:04 -0700
commitd9f30ec0b0d129b9cbf2b041a6a3159aa24592f6 (patch)
tree325c2d1550ed02e3c652625bb6c9371589af13ff /net
parent2a38b775b77f99308a4e571c13d908df78ac5e57 (diff)
downloadop-kernel-dev-d9f30ec0b0d129b9cbf2b041a6a3159aa24592f6.zip
op-kernel-dev-d9f30ec0b0d129b9cbf2b041a6a3159aa24592f6.tar.gz
[VLAN]: Fix net_device leak.
In "[VLAN]: Move device registation to seperate function" (commit e89fe42cd03c8fd3686df82d8390a235717a66de), a pile of code got moved to register_vlan_dev(), including grabbing a reference to underlying device. However, original dev_hold() had been left behind, so we leak a reference to net_device now... Signed-off-by: Al Viro <viro@zeniv.linux.org.uk> Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net')
-rw-r--r--net/8021q/vlan.c2
1 files changed, 0 insertions, 2 deletions
diff --git a/net/8021q/vlan.c b/net/8021q/vlan.c
index 1583c5e..2a54691 100644
--- a/net/8021q/vlan.c
+++ b/net/8021q/vlan.c
@@ -562,8 +562,6 @@ static int register_vlan_device(struct net_device *real_dev,
if (err < 0)
goto out_free_newdev;
- /* Account for reference in struct vlan_dev_info */
- dev_hold(real_dev);
#ifdef VLAN_DEBUG
printk(VLAN_DBG "Allocated new device successfully, returning.\n");
#endif
OpenPOWER on IntegriCloud