summaryrefslogtreecommitdiffstats
path: root/net
diff options
context:
space:
mode:
authorGao feng <gaofeng@cn.fujitsu.com>2012-12-19 23:41:43 +0000
committerDavid S. Miller <davem@davemloft.net>2012-12-21 13:17:07 -0800
commit9b1536c490d572f2ed4af0a15bd5d6cf349a9db4 (patch)
treede6ce47b0545fdee53eac2c3455117110a4ed397 /net
parent9650388b5c56578fdccc79c57a8c82fb92b8e7f1 (diff)
downloadop-kernel-dev-9b1536c490d572f2ed4af0a15bd5d6cf349a9db4.zip
op-kernel-dev-9b1536c490d572f2ed4af0a15bd5d6cf349a9db4.tar.gz
bridge: call br_netpoll_disable in br_add_if
When netdev_set_master faild in br_add_if, we should call br_netpoll_disable to do some cleanup jobs,such as free the memory of struct netpoll which allocated in br_netpoll_enable. Signed-off-by: Gao feng <gaofeng@cn.fujitsu.com> Acked-by: Cong Wang <amwang@redhat.com> Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net')
-rw-r--r--net/bridge/br_if.c8
1 files changed, 5 insertions, 3 deletions
diff --git a/net/bridge/br_if.c b/net/bridge/br_if.c
index 1c8fdc3..37fe693 100644
--- a/net/bridge/br_if.c
+++ b/net/bridge/br_if.c
@@ -366,11 +366,11 @@ int br_add_if(struct net_bridge *br, struct net_device *dev)
err = netdev_set_master(dev, br->dev);
if (err)
- goto err3;
+ goto err4;
err = netdev_rx_handler_register(dev, br_handle_frame, p);
if (err)
- goto err4;
+ goto err5;
dev->priv_flags |= IFF_BRIDGE_PORT;
@@ -402,8 +402,10 @@ int br_add_if(struct net_bridge *br, struct net_device *dev)
return 0;
-err4:
+err5:
netdev_set_master(dev, NULL);
+err4:
+ br_netpoll_disable(p);
err3:
sysfs_remove_link(br->ifobj, p->dev->name);
err2:
OpenPOWER on IntegriCloud