Skip to content

Commit b14ac0d

Browse files
author
Jonathan Toppins
committed
bonding: get correct NA dest address
In commit 4d633d1 ("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: 4d633d1 ("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> (cherry picked from commit 1f154f3) Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=2112748 Signed-off-by: Jonathan Toppins <jtoppins@redhat.com>
1 parent 9c07525 commit b14ac0d

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

drivers/net/bonding/bond_main.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3222,7 +3222,7 @@ static int bond_na_rcv(const struct sk_buff *skb, struct bonding *bond,
32223222
goto out;
32233223

32243224
saddr = &combined->ip6.saddr;
3225-
daddr = &combined->ip6.saddr;
3225+
daddr = &combined->ip6.daddr;
32263226

32273227
slave_dbg(bond->dev, slave->dev, "%s: %s/%d av %d sv %d sip %pI6c tip %pI6c\n",
32283228
__func__, slave->dev->name, bond_slave_state(slave),

0 commit comments

Comments
 (0)