Re: [PATCH v2] bonding: fix race condition in bonding_store_slaves_active
From: David Miller <davem@davemloft.net>
Date: 2012-11-29 18:14:11
From: David Miller <davem@davemloft.net>
Date: 2012-11-29 18:14:11
From: Jay Vosburgh <redacted> Date: Thu, 29 Nov 2012 09:37:56 -0800
Nikolay Aleksandrov [off-list ref] wrote:quoted
Race between bonding_store_slaves_active() and slave manipulation functions. The bond_for_each_slave use in bonding_store_slaves_active() is not protected by any synchronization mechanism. NULL pointer dereference is easy to reach. Fixed by acquiring the bond->lock for the slave walk. v2: Make description text < 75 columns Signed-off-by: Nikolay Aleksandrov <redacted>Signed-off-by: Jay Vosburgh <redacted>
Applied.