diff options
author | Bruce Allan <bruce.w.allan@intel.com> | 2010-12-09 18:04:25 -0500 |
---|---|---|
committer | Jeff Kirsher <jeffrey.t.kirsher@intel.com> | 2010-12-11 01:13:49 -0500 |
commit | 61c758166701c4f156c0aba10260f5729b1a6f43 (patch) | |
tree | 73ae69878b55a8f493b7d037a7edde819f9a2251 /drivers/net/e1000e/ethtool.c | |
parent | e9262447233037e9336d5866628821156a17366f (diff) |
e1000e: static analysis tools complain of a possible null ptr p dereference
Adding this default case resolves the issue.
v2- Removed "break" in default case based on feedback
Signed-off-by: Bruce Allan <bruce.w.allan@intel.com>
Tested-by: Emil Tantilov <emil.s.tantilov@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
Diffstat (limited to 'drivers/net/e1000e/ethtool.c')
-rw-r--r-- | drivers/net/e1000e/ethtool.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/drivers/net/e1000e/ethtool.c b/drivers/net/e1000e/ethtool.c index c10dc694d733..39349d6dcd0b 100644 --- a/drivers/net/e1000e/ethtool.c +++ b/drivers/net/e1000e/ethtool.c | |||
@@ -1978,6 +1978,9 @@ static void e1000_get_ethtool_stats(struct net_device *netdev, | |||
1978 | p = (char *) adapter + | 1978 | p = (char *) adapter + |
1979 | e1000_gstrings_stats[i].stat_offset; | 1979 | e1000_gstrings_stats[i].stat_offset; |
1980 | break; | 1980 | break; |
1981 | default: | ||
1982 | data[i] = 0; | ||
1983 | continue; | ||
1981 | } | 1984 | } |
1982 | 1985 | ||
1983 | data[i] = (e1000_gstrings_stats[i].sizeof_stat == | 1986 | data[i] = (e1000_gstrings_stats[i].sizeof_stat == |