CVEvulnerability
CVE-2024-27005
7.8 HIGH
Official page ↗
- Status
- Modified
- Published
- 2024-05-01
- Last modified
- 2026-08-04
- Assigner
- 416baaa9-dc9f-4396-8d5f-8c081fb06d67
- CVSS 3.1 (416baaa9-dc9f-4396-8d5f-8c081fb06d67)
- 7.8 HIGH — CVSS:3.1/AV:L/AC:L/PR:L/UI:N/S:U/C:H/I:H/A:H
- CVSS 3.1 (134c704f-9b21-4f2e-91b3-4a467353bcc0)
- 6.3 MEDIUM — CVSS:3.1/AV:L/AC:H/PR:L/UI:N/S:U/C:H/I:N/A:H
- Weaknesses
- CWE-362, CWE-667
Description
In the Linux kernel, the following vulnerability has been resolved:
interconnect: Don't access req_list while it's being manipulated
The icc_lock mutex was split into separate icc_lock and icc_bw_lock
mutexes in [1] to avoid lockdep splats. However, this didn't adequately
protect access to icc_node::req_list.
The icc_set_bw() function will eventually iterate over req_list while
only holding icc_bw_lock, but req_list can be modified while only
holding icc_lock. This causes races between icc_set_bw(), of_icc_get(),
and icc_put().
Example A:
CPU0 CPU1
---- ----
icc_set_bw(path_a)
mutex_lock(&icc_bw_lock);
icc_put(path_b)
mutex_lock(&icc_lock);
aggregate_requests()
hlist_for_each_entry(r, ...
hlist_del(...
<r = invalid pointer>
Example B:
CPU0 CPU1
---- ----
icc_set_bw(path_a)
mutex_lock(&icc_bw_lock);
path_b = of_icc_get()
of_icc_get_by_index()
mutex_lock(&icc_lock);
path_find()
path_init()
aggregate_requests()
hlist_for_each_entry(r, ...
hlist_add_head(...
<r = invalid pointer>
Fix this by ensuring icc_bw_lock is always held before manipulating
icc_node::req_list. The additional places icc_bw_lock is held don't
perform any memory allocations, so we should still be safe from the
original lockdep splats that motivated the separate locks.
[1] commit af42269c3523 ("interconnect: Fix locking for runpm vs reclaim")
Affected configurations
• linux linux kernel (>= 5.15.133 < 5.16)
• linux linux kernel (>= 6.1.55 < 6.2)
• linux linux kernel (>= 6.5.5 < 6.6.29)
• linux linux kernel (>= 6.7 < 6.8.8)
• linux linux kernel 6.9
Related
References
- https://git.kernel.org/stable/c/19ec82b3cad1abef2a929262b8c1528f4e0c192d git.kernel.org
- https://git.kernel.org/stable/c/4c65507121ea8e0b47fae6d2049c8688390d46b6 Patch
- https://git.kernel.org/stable/c/d0d04efa2e367921654b5106cc5c05e3757c2b42 Patch
- https://git.kernel.org/stable/c/de1bf25b6d771abdb52d43546cf57ad775fb68a1 Patch
- https://git.kernel.org/stable/c/fe549d8e976300d0dd75bd904eb216bed8b145e0 git.kernel.org
- https://git.kernel.org/stable/c/4c65507121ea8e0b47fae6d2049c8688390d46b6 Patch
- https://git.kernel.org/stable/c/d0d04efa2e367921654b5106cc5c05e3757c2b42 Patch
- https://git.kernel.org/stable/c/de1bf25b6d771abdb52d43546cf57ad775fb68a1 Patch
- https://lists.fedoraproject.org/archives/list/[email protected]/message/4EZ6PJW7VOZ224TD7N4JZNU6KV32ZJ53/ lists.fedoraproject.org
- https://lists.fedoraproject.org/archives/list/[email protected]/message/DAMSOZXJEPUOXW33WZYWCVAY7Z5S7OOY/ lists.fedoraproject.org
- https://lists.fedoraproject.org/archives/list/[email protected]/message/GCBZZEC7L7KTWWAS2NLJK6SO3IZIL4WW/ lists.fedoraproject.org