Skip to content

Commit

Permalink
e1000e: fix error checks
Browse files Browse the repository at this point in the history
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>
  • Loading branch information
Adrian Bunk authored and Jeff Garzik committed Oct 17, 2007
1 parent 121244a commit e265522
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions drivers/net/e1000e/ethtool.c
Original file line number Diff line number Diff line change
Expand Up @@ -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);
Expand Down

0 comments on commit e265522

Please sign in to comment.