diff options
author | Tung Nguyen <tung.q.nguyen@dektech.com.au> | 2018-10-12 22:46:55 +0200 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2018-10-15 22:33:27 -0700 |
commit | d3092b2efca1cd1d492d0b08499a2066c5ca8cec (patch) | |
tree | 9840478f060772c97d0379312224889e5953e94e /net/ipv6/mcast.c | |
parent | 7ec8dc96e1cb45693f28f1287802ef6f2888dae0 (diff) |
tipc: fix unsafe rcu locking when accessing publication list
The binding table's 'cluster_scope' list is rcu protected to handle
races between threads changing the list and those traversing the list at
the same moment. We have now found that the function named_distribute()
uses the regular list_for_each() macro to traverse the said list.
Likewise, the function tipc_named_withdraw() is removing items from the
same list using the regular list_del() call. When these two functions
execute in parallel we see occasional crashes.
This commit fixes this by adding the missing _rcu() suffixes.
Signed-off-by: Tung Nguyen <tung.q.nguyen@dektech.com.au>
Signed-off-by: Jon Maloy <jon.maloy@ericsson.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ipv6/mcast.c')
0 files changed, 0 insertions, 0 deletions