diff options
author | 2007-10-15 14:02:21 -0700 | |
---|---|---|
committer | 2007-10-16 21:10:27 -0400 | |
commit | e265522ca7d7a0cfa24aa2c811d22a1764a90783 (patch) | |
tree | b9f5c851e691a03f402b78a38000dbc5b5b079e3 /drivers | |
parent | e1000e: Fix debug printk macro (diff) | |
download | linux-e265522ca7d7a0cfa24aa2c811d22a1764a90783.tar.gz linux-e265522ca7d7a0cfa24aa2c811d22a1764a90783.tar.bz2 linux-e265522ca7d7a0cfa24aa2c811d22a1764a90783.zip |
e1000e: fix error checks
Spotted by the Coverity checker.
Signed-off-by: Adrian Bunk <bunk@kernel.org>
Signed-off-by: Auke Kok <auke-jan.h.kok@intel.com>
Signed-off-by: Jeff Garzik <jeff@garzik.org>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/net/e1000e/ethtool.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/net/e1000e/ethtool.c b/drivers/net/e1000e/ethtool.c index b7a7e2ae5e13..ca06c354b53c 100644 --- a/drivers/net/e1000e/ethtool.c +++ b/drivers/net/e1000e/ethtool.c @@ -1451,11 +1451,11 @@ static int e1000_loopback_test(struct e1000_adapter *adapter, u64 *data) } *data = e1000_setup_desc_rings(adapter); - if (data) + if (*data) goto out; *data = e1000_setup_loopback_test(adapter); - if (data) + if (*data) goto err_loopback; *data = e1000_run_loopback_test(adapter); |