patch 'net/e1000/base: fix NVM loop bounds and pointer access' has been queued to stable release 25.11.3
Kevin Traynor
ktraynor at redhat.com
Thu Jul 23 19:15:26 CEST 2026
Hi,
FYI, your patch has been queued to stable release 25.11.3
Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet.
It will be pushed if I get no objections before 07/27/26. So please
shout if anyone has objections.
Also note that after the patch there's a diff of the upstream commit vs the
patch applied to the branch. This will indicate if there was any rebasing
needed to apply to the stable branch. If there were code changes for rebasing
(ie: not only metadata diffs), please double check that the rebase was
correctly done.
Queued patches are on a temporary branch at:
https://github.com/kevintraynor/dpdk-stable
This queued commit can be viewed at:
https://github.com/kevintraynor/dpdk-stable/commit/c6936a6137f3ff1e8815395306d3741f62d5b3d3
Thanks.
Kevin
---
>From c6936a6137f3ff1e8815395306d3741f62d5b3d3 Mon Sep 17 00:00:00 2001
From: Menachem Fogel <menachem.fogel at intel.com>
Date: Wed, 20 May 2026 12:52:44 +0000
Subject: [PATCH] net/e1000/base: fix NVM loop bounds and pointer access
[ upstream commit 39fba42d04baf045452b224f44dcd9aea38c5b60 ]
Improve the NVM checksum routines by ensuring loop bounds are compared
at the correct integer width. Use array indexing instead of explicit
pointer arithmetic.
Fixes: af75078fece3 ("first public release")
Signed-off-by: Menachem Fogel <menachem.fogel at intel.com>
Signed-off-by: Dima Ruinskiy <dima.ruinskiy at intel.com>
Signed-off-by: Ciara Loftus <ciara.loftus at intel.com>
Acked-by: Bruce Richardson <bruce.richardson at intel.com>
---
drivers/net/intel/e1000/base/e1000_82575.c | 4 ++--
drivers/net/intel/e1000/base/e1000_manage.c | 4 ++--
2 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/net/intel/e1000/base/e1000_82575.c b/drivers/net/intel/e1000/base/e1000_82575.c
index c263f42ca1..8ae2b77d5f 100644
--- a/drivers/net/intel/e1000/base/e1000_82575.c
+++ b/drivers/net/intel/e1000/base/e1000_82575.c
@@ -2396,5 +2396,5 @@ s32 e1000_validate_nvm_checksum_with_offset(struct e1000_hw *hw, u16 offset)
DEBUGFUNC("e1000_validate_nvm_checksum_with_offset");
- for (i = offset; i < ((NVM_CHECKSUM_REG + offset) + 1); i++) {
+ for (i = offset; i < (u16)((NVM_CHECKSUM_REG + offset) + 1); i++) {
ret_val = hw->nvm.ops.read(hw, i, 1, &nvm_data);
if (ret_val) {
@@ -2433,5 +2433,5 @@ s32 e1000_update_nvm_checksum_with_offset(struct e1000_hw *hw, u16 offset)
DEBUGFUNC("e1000_update_nvm_checksum_with_offset");
- for (i = offset; i < (NVM_CHECKSUM_REG + offset); i++) {
+ for (i = offset; i < (u16)(NVM_CHECKSUM_REG + offset); i++) {
ret_val = hw->nvm.ops.read(hw, i, 1, &nvm_data);
if (ret_val) {
diff --git a/drivers/net/intel/e1000/base/e1000_manage.c b/drivers/net/intel/e1000/base/e1000_manage.c
index 4b81028302..71df2890f3 100644
--- a/drivers/net/intel/e1000/base/e1000_manage.c
+++ b/drivers/net/intel/e1000/base/e1000_manage.c
@@ -130,5 +130,5 @@ bool e1000_enable_tx_pkt_filtering_generic(struct e1000_hw *hw)
offset = E1000_MNG_DHCP_COOKIE_OFFSET >> 2;
for (i = 0; i < len; i++)
- *(buffer + i) = E1000_READ_REG_ARRAY_DWORD(hw, E1000_HOST_IF,
+ buffer[i] = E1000_READ_REG_ARRAY_DWORD(hw, E1000_HOST_IF,
offset + i);
hdr_csum = hdr->checksum;
@@ -174,5 +174,5 @@ s32 e1000_mng_write_cmd_header_generic(struct e1000_hw *hw,
for (i = 0; i < length; i++) {
E1000_WRITE_REG_ARRAY_DWORD(hw, E1000_HOST_IF, i,
- *((u32 *) hdr + i));
+ ((u32 *)hdr)[i]);
E1000_WRITE_FLUSH(hw);
}
--
2.55.0
---
Diff of the applied patch vs upstream commit (please double-check if non-empty:
---
--- - 2026-07-23 17:58:00.478536478 +0100
+++ 0063-net-e1000-base-fix-NVM-loop-bounds-and-pointer-acces.patch 2026-07-23 17:57:58.681918557 +0100
@@ -1 +1 @@
-From 39fba42d04baf045452b224f44dcd9aea38c5b60 Mon Sep 17 00:00:00 2001
+From c6936a6137f3ff1e8815395306d3741f62d5b3d3 Mon Sep 17 00:00:00 2001
@@ -5,0 +6,2 @@
+[ upstream commit 39fba42d04baf045452b224f44dcd9aea38c5b60 ]
+
@@ -11 +12,0 @@
-Cc: stable at dpdk.org
More information about the stable
mailing list