diff options
| author | Zhang Shengju <zhangshengju@cmss.chinamobile.com> | 2016-02-09 10:37:46 +0000 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2016-02-11 09:57:00 -0500 |
| commit | 1e2a8868e0541b7d8870781b714d74ac7cad9a6a (patch) | |
| tree | 4bf88800643ec885813d06a6d15fc5c66c3ce80f /net/ipv4/sysctl_net_ipv4.c | |
| parent | 3e2a5e1539064329f5f6888274600841ad6f87bd (diff) | |
| download | op-kernel-dev-1e2a8868e0541b7d8870781b714d74ac7cad9a6a.zip op-kernel-dev-1e2a8868e0541b7d8870781b714d74ac7cad9a6a.tar.gz | |
bonding: use return instead of goto
Replace 'goto' with 'return' to remove unnecessary check at label:
err_undo_flags.
The reason is that 'err_undo_flags' do two things for the first slave device:
1.revert bond mac address if it is set by the slave device.
2.revert bond device type if it's not ARPHRD_ETHER.
It's not necessary for the following three places, they changed neither bond
mac address nor type. It's straightforward to return directly.
Signed-off-by: Zhang Shengju <zhangshengju@cmss.chinamobile.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ipv4/sysctl_net_ipv4.c')
0 files changed, 0 insertions, 0 deletions
