]> git.itanic.dy.fi Git - linux-stable/commitdiff
bonding: get correct NA dest address
authorHangbin Liu <liuhangbin@gmail.com>
Tue, 6 Dec 2022 03:20:55 +0000 (11:20 +0800)
committerJakub Kicinski <kuba@kernel.org>
Wed, 7 Dec 2022 04:20:38 +0000 (20:20 -0800)
In commit 4d633d1b468b ("bonding: fix ICMPv6 header handling when receiving
IPv6 messages"), there is a copy/paste issue for NA daddr. I found that
in my testing and fixed it in my local branch. But I forgot to re-format
the patch and sent the wrong mail.

Fix it by reading the correct dest address.

Fixes: 4d633d1b468b ("bonding: fix ICMPv6 header handling when receiving IPv6 messages")
Signed-off-by: Hangbin Liu <liuhangbin@gmail.com>
Reviewed-by: Eric Dumazet <edumazet@google.com>
Reviewed-by: Jiri Pirko <jiri@nvidia.com>
Acked-by: Jonathan Toppins <jtoppins@redhat.com>
Link: https://lore.kernel.org/r/20221206032055.7517-1-liuhangbin@gmail.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
drivers/net/bonding/bond_main.c

index f298b9b3eb77e417f81fe06b3838b340ef56bf86..b9a882f182d296a095bbcdd786b262ff41ede181 100644 (file)
@@ -3247,7 +3247,7 @@ static int bond_na_rcv(const struct sk_buff *skb, struct bonding *bond,
                goto out;
 
        saddr = &combined->ip6.saddr;
-       daddr = &combined->ip6.saddr;
+       daddr = &combined->ip6.daddr;
 
        slave_dbg(bond->dev, slave->dev, "%s: %s/%d av %d sv %d sip %pI6c tip %pI6c\n",
                  __func__, slave->dev->name, bond_slave_state(slave),