diff options
| author | Michal Swiatkowski <michal.swiatkowski@linux.intel.com> | 2025-03-12 10:52:51 +0100 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2025-03-19 09:57:37 +0000 |
| commit | 3178d2b048365fe2c078cd53f85f2abf1487733b (patch) | |
| tree | 730c91319266d5994452fcbf31590f3661acf348 | |
| parent | 3614bf90130d60f191a5fe218d04f6251c678e13 (diff) | |
phy: fix xa_alloc_cyclic() error handling
xa_alloc_cyclic() can return 1, which isn't an error. To prevent
situation when the caller of this function will treat it as no error do
a check only for negative here.
Fixes: 384968786909 ("net: phy: Introduce ethernet link topology representation")
Signed-off-by: Michal Swiatkowski <michal.swiatkowski@linux.intel.com>
Reviewed-by: Maxime Chevallier <maxime.chevallier@bootlin.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
| -rw-r--r-- | drivers/net/phy/phy_link_topology.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/phy/phy_link_topology.c b/drivers/net/phy/phy_link_topology.c index 4a5d73002a1a..0e9e987f37dd 100644 --- a/drivers/net/phy/phy_link_topology.c +++ b/drivers/net/phy/phy_link_topology.c @@ -73,7 +73,7 @@ int phy_link_topo_add_phy(struct net_device *dev, xa_limit_32b, &topo->next_phy_index, GFP_KERNEL); - if (ret) + if (ret < 0) goto err; return 0; |