Re: [PATCH net v2 2/2] bridge: netlink: register netdevice before executing changelink

[Stephen Hemminger <stephen@xxxxxxxxxxxxxxxxxx>]

On Sat, 8 Apr 2017 14:41:59 +0300
<idosch@xxxxxxxxxxxx> 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