Re: rtnl_lock() imbalance

[Joakim Tjernlund <joakim.tjernlund@xxxxxxxxxxxx>]

Joakim Tjernlund/Transmode wrote on 2010/11/08 16:19:35:
>
> Trying to hunt down an rtnl_lock bug, br_add_bridge looks funny, it
> takes the rtnl_lock but doesn't release it in case of error. Bug?

ehh, missed that last goto out;
So no imbalance, sorry for the noise

>
> int br_add_bridge(struct net *net, const char *name)
> {
>  struct net_device *dev;
>  int ret;
>
>  dev = new_bridge_dev(net, name);
>  if (!dev)
>   return -ENOMEM;
>
>  rtnl_lock();
>  if (strchr(dev->name, '%')) {
>   ret = dev_alloc_name(dev, dev->name);
>   if (ret < 0)
>    goto out_free;
>  }
>
>  SET_NETDEV_DEVTYPE(dev, &br_type);
>
>  ret = register_netdevice(dev);
>  if (ret)
>   goto out_free;
>
>  ret = br_sysfs_addbr(dev);
>  if (ret)
>   unregister_netdevice(dev);
>  out:
>  rtnl_unlock();
>  return ret;
>
> out_free:
>  free_netdev(dev);
>  goto out;
> }

_______________________________________________
Bridge mailing list
Bridge@xxxxxxxxxxxxxxxxxxxxxxxxxx
https://lists.linux-foundation.org/mailman/listinfo/bridge