diff options
| author | Colin Ian King <colin.king@canonical.com> | 2018-12-22 11:56:53 +0000 |
|---|---|---|
| committer | Kishon Vijay Abraham I <kishon@ti.com> | 2019-01-16 18:00:54 +0530 |
| commit | 1138a442a0c5e97594ec63e898098b0c9e781172 (patch) | |
| tree | 7db3d876d079b8491210f585c93de5948d368bb3 | |
| parent | 3bd6e94bec122a951d462c239b47954cf5f36e33 (diff) | |
| download | lwn-1138a442a0c5e97594ec63e898098b0c9e781172.tar.gz lwn-1138a442a0c5e97594ec63e898098b0c9e781172.zip | |
phy: ti: ensure priv is not null before dereferencing it
Currently priv is being dereferenced before priv is being null checked.
Fix this by moving the null check on priv before the dereference.
Detected by CoverityScan, CID#1476018 ("Dereference before null check")
Fixes: 92b58b34741f ("phy: ti: introduce phy-gmii-sel driver")
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: Kishon Vijay Abraham I <kishon@ti.com>
| -rw-r--r-- | drivers/phy/ti/phy-gmii-sel.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/phy/ti/phy-gmii-sel.c b/drivers/phy/ti/phy-gmii-sel.c index 77fdaa551977..a52c5bb35033 100644 --- a/drivers/phy/ti/phy-gmii-sel.c +++ b/drivers/phy/ti/phy-gmii-sel.c @@ -204,11 +204,11 @@ static struct phy *phy_gmii_sel_of_xlate(struct device *dev, if (args->args_count < 1) return ERR_PTR(-EINVAL); + if (!priv || !priv->if_phys) + return ERR_PTR(-ENODEV); if (priv->soc_data->features & BIT(PHY_GMII_SEL_RMII_IO_CLK_EN) && args->args_count < 2) return ERR_PTR(-EINVAL); - if (!priv || !priv->if_phys) - return ERR_PTR(-ENODEV); if (phy_id > priv->soc_data->num_ports) return ERR_PTR(-EINVAL); if (phy_id != priv->if_phys[phy_id - 1].id) |