diff options
| author | Michal Swiatkowski <michal.swiatkowski@linux.intel.com> | 2025-03-12 10:52:49 +0100 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2025-03-19 09:57:36 +0000 |
| commit | f3b97b7d4bf316c3991e5634c9f4847c2df35478 (patch) | |
| tree | f4d0a6c051a2a2ee4a241f32e033b58c5f07c5c6 | |
| parent | 9a81fc3480bf5dbe2bf80e278c440770f6ba2692 (diff) | |
devlink: fix xa_alloc_cyclic() error handling
In case of returning 1 from xa_alloc_cyclic() (wrapping) ERR_PTR(1) will
be returned, which will cause IS_ERR() to be false. Which can lead to
dereference not allocated pointer (rel).
Fix it by checking if err is lower than zero.
This wasn't found in real usecase, only noticed. Credit to Pierre.
Fixes: c137743bce02 ("devlink: introduce object and nested devlink relationship infra")
Signed-off-by: Michal Swiatkowski <michal.swiatkowski@linux.intel.com>
Reviewed-by: Andrew Lunn <andrew@lunn.ch>
Reviewed-by: Jiri Pirko <jiri@nvidia.com>
Signed-off-by: David S. Miller <davem@davemloft.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 f49cd83f1955f..7203c39532fcc 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); } |