netdev - Reply: [net] bonding: use return instead of goto

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [thread-next>] [day] [month] [year] [list]

Message-ID: <000301d15fb6$77bf08f0$673d1ad0$@cmss.chinamobile.com>
Date:	Fri, 5 Feb 2016 09:42:24 +0800
From:	张胜举 <zhangshengju@...s.chinamobile.com>
To:	"'Jarod Wilson'" <jarod@...hat.com>
Cc:	<j.vosburgh@...il.com>, <netdev@...r.kernel.org>
Subject: Reply: [net] bonding: use return instead of goto

> On Wed, Feb 03, 2016 at 06:15:22AM +0000, Zhang Shengju wrote:
> > Replace 'goto' with 'return' to remove unnecessary check at label:
> > err_undo_flags.
> 
> I think you're going to have to explain how you came to the conclusion
that
> the check isn't necessary.
> 
> --
> Jarod Wilson
> jarod@...hat.com
Hi Jarod,

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.

I think it's not necessary for the three places, they changed neither  bond
mac address nor type. 
it's straightforward to return directly.

Thanks,
Shengju