From mboxrd@z Thu Jan 1 00:00:00 1970 From: Colin King Subject: [PATCH] amd-xgbe: fix unsigned comparison against less than zero Date: Tue, 15 Nov 2016 12:18:42 +0000 Message-ID: <20161115121842.5774-1-colin.king@canonical.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8bit Cc: linux-kernel@vger.kernel.org To: Tom Lendacky , netdev@vger.kernel.org Return-path: Sender: linux-kernel-owner@vger.kernel.org List-Id: netdev.vger.kernel.org From: Colin Ian King Comparing unsigned int ret to less than zero for an error status check is never true. Fix this by making ret a signed int. Reduce scope of ret too. Found with static analysis by CoverityScan, CID 1377750 Signed-off-by: Colin Ian King --- drivers/net/ethernet/amd/xgbe/xgbe-phy-v2.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/net/ethernet/amd/xgbe/xgbe-phy-v2.c b/drivers/net/ethernet/amd/xgbe/xgbe-phy-v2.c index 4ba4332..168507e 100644 --- a/drivers/net/ethernet/amd/xgbe/xgbe-phy-v2.c +++ b/drivers/net/ethernet/amd/xgbe/xgbe-phy-v2.c @@ -2346,7 +2346,7 @@ static bool xgbe_phy_valid_speed(struct xgbe_prv_data *pdata, int speed) static int xgbe_phy_link_status(struct xgbe_prv_data *pdata, int *an_restart) { struct xgbe_phy_data *phy_data = pdata->phy_data; - unsigned int ret, reg; + unsigned int reg; *an_restart = 0; @@ -2365,7 +2365,8 @@ static int xgbe_phy_link_status(struct xgbe_prv_data *pdata, int *an_restart) if (phy_data->phydev) { /* Check external PHY */ - ret = phy_read_status(phy_data->phydev); + int ret = phy_read_status(phy_data->phydev); + if (ret < 0) return 0; -- 2.10.2