netdev - Re: [PATCH -net 0/2] bonding: fix arp

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-prev] [day] [month] [year] [list]

Message-ID: <522A36D2.1080306@redhat.com>
Date:	Fri, 06 Sep 2013 22:10:58 +0200
From:	Nikolay Aleksandrov <nikolay@...hat.com>
To:	netdev@...r.kernel.org
CC:	fubar@...ibm.com, davem@...emloft.net, andy@...yhouse.net,
	mleitner@...hat.com
Subject: Re: [PATCH -net 0/2] bonding: fix arp_validate desync state & race

On 09/06/2013 09:41 PM, Nikolay Aleksandrov wrote:
> Hello all,
> These two patches aim to fix the possible de-sync state which the bond
> can enter if we have arp_validate without arp_interval or the other way
> around. They also fix a race condition between arp_validate setting and
> mode changing.
> 
> Patch 01 - fixes the race condition between store_arp_validate and bond
> mode change by using rtnl for sync
> Patch 02 - fixes the possible de-sync state by setting/unsetting recv_probe
> if arp_interval is set/unset and also if arp_validate is set/unset
> 
> Best regards,
>  Nikolay Aleksandrov
> 
> 
> Nikolay Aleksandrov (2):
>   bonding: fix store_arp_validate race with mode change
>   bonding: fix bond_arp_rcv setting and arp validate desync state
> 
>  drivers/net/bonding/bond_main.c  |  4 ++--
>  drivers/net/bonding/bond_sysfs.c | 31 +++++++++++++++++++++++++------
>  drivers/net/bonding/bonding.h    |  1 +
>  3 files changed, 28 insertions(+), 8 deletions(-)
> 
Ugh, self-nack I just noticed the mode check in store_arp_validate passes
if new_value is 0.

I'll wait some time to see if there isn't anything else and re-post.

Cheers,
 Nik
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html