diff options
| author | David S. Miller <davem@davemloft.net> | 2025-03-19 09:57:37 +0000 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2025-03-19 09:57:37 +0000 |
| commit | d9c743b6990b1ee3925f3ced1844cafa955bacb0 (patch) | |
| tree | 730c91319266d5994452fcbf31590f3661acf348 /net | |
| parent | 9a81fc3480bf5dbe2bf80e278c440770f6ba2692 (diff) | |
| parent | 3178d2b048365fe2c078cd53f85f2abf1487733b (diff) | |
Merge branch 'xa_alloc_cyclic-checks'
Michal Swiatkowski says:
====================
fix xa_alloc_cyclic() return checks
Pierre Riteau <pierre@stackhpc.com> found suspicious handling an error
from xa_alloc_cyclic() in scheduler code [1]. The same is done in few
other places.
v1 --> v2: [2]
* add fixes tags
* fix also the same usage in dpll and phy
[1] https://lore.kernel.org/netdev/20250213223610.320278-1-pierre@stackhpc.com/
[2] https://lore.kernel.org/netdev/20250214132453.4108-1-michal.swiatkowski@linux.intel.com/
====================
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net')
| -rw-r--r-- | net/devlink/core.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/devlink/core.c b/net/devlink/core.c index f49cd83f1955..7203c39532fc 100644 --- a/net/devlink/core.c +++ b/net/devlink/core.c @@ -117,7 +117,7 @@ static struct devlink_rel *devlink_rel_alloc(void) err = xa_alloc_cyclic(&devlink_rels, &rel->index, rel, xa_limit_32b, &next, GFP_KERNEL); - if (err) { + if (err < 0) { kfree(rel); return ERR_PTR(err); } |