diff options
| author | Alexander Mikhalitsyn <[email protected]> | 2024-05-28 22:30:30 +0200 |
|---|---|---|
| committer | Jakub Kicinski <[email protected]> | 2024-05-29 18:43:42 -0700 |
| commit | b8c8abefc07b47f0dc9342530b7618237df96724 (patch) | |
| tree | b1d24a1a969d70a81d35587cc010fc2827cb3bb5 | |
| parent | 92f1655aa2b2294d0b49925f3b875a634bd3b59e (diff) | |
ipv4: correctly iterate over the target netns in inet_dump_ifaddr()
A recent change to inet_dump_ifaddr had the function incorrectly iterate
over net rather than tgt_net, resulting in the data coming for the
incorrect network namespace.
Fixes: cdb2f80f1c10 ("inet: use xa_array iterator to implement inet_dump_ifaddr()")
Reported-by: Stéphane Graber <[email protected]>
Closes: https://github.com/lxc/incus/issues/892
Bisected-by: Stéphane Graber <[email protected]>
Signed-off-by: Alexander Mikhalitsyn <[email protected]>
Tested-by: Stéphane Graber <[email protected]>
Acked-by: Christian Brauner <[email protected]>
Reviewed-by: Eric Dumazet <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
Signed-off-by: Jakub Kicinski <[email protected]>
| -rw-r--r-- | net/ipv4/devinet.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/ipv4/devinet.c b/net/ipv4/devinet.c index e827da128c5f..f3892ee9dfb3 100644 --- a/net/ipv4/devinet.c +++ b/net/ipv4/devinet.c @@ -1903,7 +1903,7 @@ static int inet_dump_ifaddr(struct sk_buff *skb, struct netlink_callback *cb) cb->seq = inet_base_seq(tgt_net); - for_each_netdev_dump(net, dev, ctx->ifindex) { + for_each_netdev_dump(tgt_net, dev, ctx->ifindex) { in_dev = __in_dev_get_rcu(dev); if (!in_dev) continue; |