]> git.ipfire.org Git - thirdparty/kernel/stable.git/commitdiff
bnxt_en: correct offset handling for IPv6 destination address
authorAlok Tiwari <alok.a.tiwari@oracle.com>
Sat, 20 Sep 2025 12:11:17 +0000 (05:11 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 2 Oct 2025 11:35:45 +0000 (13:35 +0200)
[ Upstream commit 3d3aa9472c6dd0704e9961ed4769caac5b1c8d52 ]

In bnxt_tc_parse_pedit(), the code incorrectly writes IPv6
destination values to the source address field (saddr) when
processing pedit offsets within the destination address range.

This patch corrects the assignment to use daddr instead of saddr,
ensuring that pedit operations on IPv6 destination addresses are
applied correctly.

Fixes: 9b9eb518e338 ("bnxt_en: Add support for NAT(L3/L4 rewrite)")
Signed-off-by: Alok Tiwari <alok.a.tiwari@oracle.com>
Reviewed-by: Somnath Kotur <somnath.kotur@broadcom.com>
Link: https://patch.msgid.link/20250920121157.351921-1-alok.a.tiwari@oracle.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
drivers/net/ethernet/broadcom/bnxt/bnxt_tc.c

index 775d0b7521ca06ae5e893a4d32af337108495826..ef2bd84c55ff5009bbb06fdd98d60c49d682d1ee 100644 (file)
@@ -243,7 +243,7 @@ bnxt_tc_parse_pedit(struct bnxt *bp, struct bnxt_tc_actions *actions,
                           offset < offset_of_ip6_daddr + 16) {
                        actions->nat.src_xlate = false;
                        idx = (offset - offset_of_ip6_daddr) / 4;
-                       actions->nat.l3.ipv6.saddr.s6_addr32[idx] = htonl(val);
+                       actions->nat.l3.ipv6.daddr.s6_addr32[idx] = htonl(val);
                } else {
                        netdev_err(bp->dev,
                                   "%s: IPv6_hdr: Invalid pedit field\n",