summaryrefslogtreecommitdiff
path: root/include/linux
diff options
context:
space:
mode:
authorAgalakov Daniil <ade@amicon.ru>2026-06-09 14:35:53 -0700
committerJakub Kicinski <kuba@kernel.org>2026-06-13 16:44:06 -0700
commit4cc8566ae0d1609d888d90bf4e49b4f6ee62c1cd (patch)
tree7a3cd4ee9c389480647bc25eba8f951bd2c68d55 /include/linux
parent4d9508d97b7bf81f71ef840607bfa2eabee1442b (diff)
e1000: limit endianness conversion to boundary words
[Why] In e1000_set_eeprom(), the eeprom_buff is allocated to hold a range of words. However, only the boundary words (the first and the last) are populated from the EEPROM if the write request is not word-aligned. The words in the middle of the buffer remain uninitialized because they are intended to be completely overwritten by the new data via memcpy(). The previous implementation had a loop that performed le16_to_cpus() on the entire buffer. This resulted in endianness conversion being performed on uninitialized memory for all interior words. Fix this by converting the endianness only for the boundary words immediately after they are successfully read from the EEPROM. Found by Linux Verification Center (linuxtesting.org) with SVACE. Co-developed-by: Iskhakov Daniil <dish@amicon.ru> Signed-off-by: Iskhakov Daniil <dish@amicon.ru> Signed-off-by: Agalakov Daniil <ade@amicon.ru> Signed-off-by: Tony Nguyen <anthony.l.nguyen@intel.com> Link: https://patch.msgid.link/20260609213559.178657-13-anthony.l.nguyen@intel.com Signed-off-by: Jakub Kicinski <kuba@kernel.org>
Diffstat (limited to 'include/linux')
0 files changed, 0 insertions, 0 deletions