ipv4: rcu cleanup in ip_ra_control()
authorEric Dumazet <edumazet@google.com>
Tue, 9 Sep 2014 15:11:41 +0000 (08:11 -0700)
committerDavid S. Miller <davem@davemloft.net>
Wed, 10 Sep 2014 03:10:44 +0000 (20:10 -0700)
Remove one sparse warning :
net/ipv4/ip_sockglue.c:328:22: warning: incorrect type in assignment (different address spaces)
net/ipv4/ip_sockglue.c:328:22:    expected struct ip_ra_chain [noderef] <asn:4>*next
net/ipv4/ip_sockglue.c:328:22:    got struct ip_ra_chain *[assigned] ra

And replace one rcu_assign_ptr() by RCU_INIT_POINTER() where applicable.

Signed-off-by: Eric Dumazet <edumazet@google.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/ipv4/ip_sockglue.c

index 455e75bcb1677dd6bcf4b5c65bae18d607d95137..c373a9ad45556815d1d95f4faf71b8f8a3f61996 100644 (file)
@@ -303,7 +303,7 @@ int ip_ra_control(struct sock *sk, unsigned char on,
                        }
                        /* dont let ip_call_ra_chain() use sk again */
                        ra->sk = NULL;
-                       rcu_assign_pointer(*rap, ra->next);
+                       RCU_INIT_POINTER(*rap, ra->next);
                        spin_unlock_bh(&ip_ra_lock);
 
                        if (ra->destructor)
@@ -325,7 +325,7 @@ int ip_ra_control(struct sock *sk, unsigned char on,
        new_ra->sk = sk;
        new_ra->destructor = destructor;
 
-       new_ra->next = ra;
+       RCU_INIT_POINTER(new_ra->next, ra);
        rcu_assign_pointer(*rap, new_ra);
        sock_hold(sk);
        spin_unlock_bh(&ip_ra_lock);