diff options
| author | Eric Dumazet <eric.dumazet@gmail.com> | 2011-08-22 06:05:59 +0000 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2011-08-22 16:49:56 -0700 |
| commit | 11f3a6bdc2528d1ce2af50202dbf7138fdee1b34 (patch) | |
| tree | 69ee1b19bd869600b0f0898986dd5e4b34d299c6 /net/bridge | |
| parent | 832d80aceaab995c347d62abf7c241fc3e2eb0ce (diff) | |
| download | lwn-11f3a6bdc2528d1ce2af50202dbf7138fdee1b34.tar.gz lwn-11f3a6bdc2528d1ce2af50202dbf7138fdee1b34.zip | |
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')
| -rw-r--r-- | net/bridge/br_if.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/net/bridge/br_if.c b/net/bridge/br_if.c index 2cdf0070419f..e73815456adf 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) |