diff options
| author | Herbert Xu <[email protected]> | 2017-02-11 11:26:46 +0000 |
|---|---|---|
| committer | David S. Miller <[email protected]> | 2017-02-17 17:28:35 +0000 |
| commit | 40f9f439706073b4b0a654b3b99e18296b7990b3 (patch) | |
| tree | baa437731befae16bef34595d72e2b64c2e4a97b /net/tipc/net.c | |
| parent | gfs2: Use rhashtable walk interface in glock_hash_walk (diff) | |
| download | kernel-40f9f439706073b4b0a654b3b99e18296b7990b3.tar.gz kernel-40f9f439706073b4b0a654b3b99e18296b7990b3.zip | |
tipc: Fix tipc_sk_reinit race conditions
There are two problems with the function tipc_sk_reinit. Firstly
it's doing a manual walk over an rhashtable. This is broken as
an rhashtable can be resized and if you manually walk over it
during a resize then you may miss entries.
Secondly it's missing memory barriers as previously the code used
spinlocks which provide the barriers implicitly.
This patch fixes both problems.
Fixes: 07f6c4bc048a ("tipc: convert tipc reference table to...")
Signed-off-by: Herbert Xu <[email protected]>
Acked-by: Ying Xue <[email protected]>
Signed-off-by: David S. Miller <[email protected]>
Diffstat (limited to 'net/tipc/net.c')
| -rw-r--r-- | net/tipc/net.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/net/tipc/net.c b/net/tipc/net.c index 28bf4feeb81c..ab8a2d5d1e32 100644 --- a/net/tipc/net.c +++ b/net/tipc/net.c @@ -110,6 +110,10 @@ int tipc_net_start(struct net *net, u32 addr) char addr_string[16]; tn->own_addr = addr; + + /* Ensure that the new address is visible before we reinit. */ + smp_mb(); + tipc_named_reinit(net); tipc_sk_reinit(net); |