瀏覽代碼

tunnels: missing rcu_assign_pointer()

xfrm4_tunnel_register() & xfrm6_tunnel_register() should
use rcu_assign_pointer() to make sure previous writes
(to handler->next) are committed to memory before chain
insertion.

deregister functions dont need a particular barrier.

Signed-off-by: Eric Dumazet <eric.dumazet@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Eric Dumazet 14 年之前
父節點
當前提交
49d61e2390
共有 2 個文件被更改,包括 2 次插入2 次删除
  1. 1 1
      net/ipv4/tunnel4.c
  2. 1 1
      net/ipv6/tunnel6.c

+ 1 - 1
net/ipv4/tunnel4.c

@@ -39,7 +39,7 @@ int xfrm4_tunnel_register(struct xfrm_tunnel *handler, unsigned short family)
 	}
 	}
 
 
 	handler->next = *pprev;
 	handler->next = *pprev;
-	*pprev = handler;
+	rcu_assign_pointer(*pprev, handler);
 
 
 	ret = 0;
 	ret = 0;
 
 

+ 1 - 1
net/ipv6/tunnel6.c

@@ -51,7 +51,7 @@ int xfrm6_tunnel_register(struct xfrm6_tunnel *handler, unsigned short family)
 	}
 	}
 
 
 	handler->next = *pprev;
 	handler->next = *pprev;
-	*pprev = handler;
+	rcu_assign_pointer(*pprev, handler);
 
 
 	ret = 0;
 	ret = 0;