summaryrefslogtreecommitdiffstats
path: root/net/bridge/br_if.c
diff options
context:
space:
mode:
authorEric Dumazet <eric.dumazet@gmail.com>2011-08-22 06:05:59 +0000
committerDavid S. Miller <davem@davemloft.net>2011-08-22 16:49:56 -0700
commit11f3a6bdc2528d1ce2af50202dbf7138fdee1b34 (patch)
tree69ee1b19bd869600b0f0898986dd5e4b34d299c6 /net/bridge/br_if.c
parent832d80aceaab995c347d62abf7c241fc3e2eb0ce (diff)
downloadop-kernel-dev-11f3a6bdc2528d1ce2af50202dbf7138fdee1b34.zip
op-kernel-dev-11f3a6bdc2528d1ce2af50202dbf7138fdee1b34.tar.gz
bridge: fix a possible net_device leak
Jan Beulich reported a possible net_device leak in bridge code after commit bb900b27a2f4 (bridge: allow creating bridge devices with netlink) Reported-by: Jan Beulich <JBeulich@novell.com> Signed-off-by: Eric Dumazet <eric.dumazet@gmail.com> Acked-by: Stephen Hemminger <shemminger@vyatta.com> Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/bridge/br_if.c')
-rw-r--r--net/bridge/br_if.c6
1 files changed, 5 insertions, 1 deletions
diff --git a/net/bridge/br_if.c b/net/bridge/br_if.c
index 2cdf007..e738154 100644
--- a/net/bridge/br_if.c
+++ b/net/bridge/br_if.c
@@ -231,6 +231,7 @@ static struct net_bridge_port *new_nbp(struct net_bridge *br,
int br_add_bridge(struct net *net, const char *name)
{
struct net_device *dev;
+ int res;
dev = alloc_netdev(sizeof(struct net_bridge), name,
br_dev_setup);
@@ -240,7 +241,10 @@ int br_add_bridge(struct net *net, const char *name)
dev_net_set(dev, net);
- return register_netdev(dev);
+ res = register_netdev(dev);
+ if (res)
+ free_netdev(dev);
+ return res;
}
int br_del_bridge(struct net *net, const char *name)
OpenPOWER on IntegriCloud