On Sat, 8 Apr 2017 14:41:59 +0300 <[email protected]> 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
