This series addresses a common issues related to uninitialized memory access during EEPROM write operations in the e1000 and e1000e drivers.
In the e1000 driver, the return value of e1000_read_eeprom() was not checked, and both e1000 and e1000e drivers performed endianness conversion on the entire buffer, including uninitialized interior words. The changes are: - add missing error checks for e1000_read_eeprom() in e1000; - limit endianness conversion to the boundary words being read, avoiding a loop over entire (and partially uninitialized) buffer in both e1000 and e1000e. Patch 1: e1000: check return value of e1000_read_eeprom Patch 2: e1000: fix endianness conversion of uninitialized words Patch 3: e1000e: fix endianness conversion of uninitialized words Co-developed-by: Iskhakov Daniil <[email protected]> Signed-off-by: Iskhakov Daniil <[email protected]> Signed-off-by: Agalakov Daniil <[email protected]> --- .../net/ethernet/intel/e1000/e1000_ethtool.c | 18 +++++++++++++----- drivers/net/ethernet/intel/e1000e/ethtool.c | 19 ++++++++++++------- 2 files changed, 25 insertions(+), 12 deletions(-) -- 2.51.0
