On Sat, 8 Apr 2017 14:41:59 +0300
<ido...@mellanox.com> wrote:

> +     err = br_changelink(dev, tb, data);
> +     if (err)
> +             goto unregister;
> +
> +     return 0;
> +
> +unregister:
> +     unregister_netdevice(dev);
> +     return err;
>  }

Why use a goto? just do:
        err = br_changelink(dev, tb, data);
        if (err)
                unregister_netdevice(dev)
        return err;
}


The goto looks ugly

Reply via email to