浏览代码

e1000e: Use sizeof struct rather than pointer in e1000_get_eeprom()

Don't use the sizeof the pointer to clear the result

Signed-off-by: Roel Kluin <roel.kluin@gmail.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Roel Kluin 15 年之前
父节点
当前提交
8528b016d6
共有 1 个文件被更改,包括 2 次插入1 次删除
  1. 2 1
      drivers/net/e1000e/ethtool.c

+ 2 - 1
drivers/net/e1000e/ethtool.c

@@ -535,7 +535,8 @@ static int e1000_get_eeprom(struct net_device *netdev,
 
 
 	if (ret_val) {
 	if (ret_val) {
 		/* a read error occurred, throw away the result */
 		/* a read error occurred, throw away the result */
-		memset(eeprom_buff, 0xff, sizeof(eeprom_buff));
+		memset(eeprom_buff, 0xff, sizeof(u16) *
+		       (last_word - first_word + 1));
 	} else {
 	} else {
 		/* Device's eeprom is always little-endian, word addressable */
 		/* Device's eeprom is always little-endian, word addressable */
 		for (i = 0; i < last_word - first_word + 1; i++)
 		for (i = 0; i < last_word - first_word + 1; i++)