diff options
| author | Veaceslav Falico <vfalico@redhat.com> | 2014-01-10 11:59:45 +0100 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2014-01-14 07:22:27 +0100 |
| commit | 49b7624eda6867d2803bcc5bbf6f25936184304a (patch) | |
| tree | 8cf62ce9ae669d5587bbc7b9dc8ee504eecaed56 /net/xfrm/xfrm_policy.c | |
| parent | bonding: fix __get_first_agg RCU usage (diff) | |
| download | linux-49b7624eda6867d2803bcc5bbf6f25936184304a.tar.xz linux-49b7624eda6867d2803bcc5bbf6f25936184304a.zip | |
bonding: fix __get_active_agg() RCU logic
Currently, the implementation is meaningless - once again, we take the
slave structure and use it after we've exited RCU critical section.
Fix this by removing the rcu_read_lock() from __get_active_agg(), and
ensuring that all its callers are holding RCU.
Fixes: be79bd048 ("bonding: add RCU for bond_3ad_state_machine_handler()")
CC: dingtianhong@huawei.com
CC: Jay Vosburgh <fubar@us.ibm.com>
CC: Andy Gospodarek <andy@greyhouse.net>
Signed-off-by: Veaceslav Falico <vfalico@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/xfrm/xfrm_policy.c')
0 files changed, 0 insertions, 0 deletions