net_sched: em_meta: add READ_ONCE() in var_sk_bound_if()
authorEric Dumazet <edumazet@google.com>
Fri, 13 May 2022 18:55:47 +0000 (11:55 -0700)
committerDavid S. Miller <davem@davemloft.net>
Mon, 16 May 2022 09:31:06 +0000 (10:31 +0100)
sk->sk_bound_dev_if can change under us, use READ_ONCE() annotation.

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

index 0a04468b73145546097788db0645ef9e5b459bca..49bae3d5006b0f83330b4cbe30344c0741743575 100644 (file)
@@ -311,12 +311,15 @@ META_COLLECTOR(int_sk_bound_if)
 
 META_COLLECTOR(var_sk_bound_if)
 {
+       int bound_dev_if;
+
        if (skip_nonlocal(skb)) {
                *err = -1;
                return;
        }
 
-       if (skb->sk->sk_bound_dev_if == 0) {
+       bound_dev_if = READ_ONCE(skb->sk->sk_bound_dev_if);
+       if (bound_dev_if == 0) {
                dst->value = (unsigned long) "any";
                dst->len = 3;
        } else {
@@ -324,7 +327,7 @@ META_COLLECTOR(var_sk_bound_if)
 
                rcu_read_lock();
                dev = dev_get_by_index_rcu(sock_net(skb->sk),
-                                          skb->sk->sk_bound_dev_if);
+                                          bound_dev_if);
                *err = var_dev(dev, dst);
                rcu_read_unlock();
        }