slight bug in e1000

From: Ben Greear (
Date: Sun Aug 25 2002 - 12:57:48 EST

I just noticed this in e1000_ethtool.c (2.4.20-pre4):

static int
e1000_ethtool_geeprom(struct e1000_adapter *adapter,
                       struct ethtool_eeprom *eeprom, uint16_t *eeprom_buff)
        struct e1000_hw *hw = &adapter->hw;
        int i, max_len, first_word, last_word;

        if(eeprom->len == 0) return;

The return value should probably be zero or an error code. Definately
not undefined as it is now.


Ben Greear <>       <Ben_Greear AT>
President of Candela Technologies Inc

- To unsubscribe from this list: send the line "unsubscribe linux-net" in the body of a message to More majordomo info at

This archive was generated by hypermail 2b29 : Sat Aug 31 2002 - 22:00:01 EST