net: qrtr: fix refcount saturation and potential UAF in qrtr_port_remove
- Published
- Last modified
- Where the record stands: published, rejected by its publisher, or superseded by another id.What do these mean? →
- published
- Whether CISA lists it in its Known Exploited Vulnerabilities catalog: proof that attackers are using it.What do these mean? →
- Not listed in CISA KEV
- Fix
- Fixed in 5.10.259, 5.15.210, 6.1.176, 6.6.143, 6.12.94, and 2 more
- Whether working attack code is public (Exploit-DB, Metasploit, Nuclei or GitHub). Public code makes attacks easier.What do these mean? →
- None on Exploit-DB, Metasploit or GitHub
- Exploitation: none
- Automatable: no
- Technical impact: total
Summary
In the Linux kernel, the following vulnerability has been resolved: net: qrtr: fix refcount saturation and potential UAF in qrtr_port_remove In qrtr_port_remove(), the socket reference count is decremented via __sock_put() before the port is removed from the qrtr_ports XArray and before the RCU grace period elapses. This breaks the fundamental RCU update paradigm. It exposes a race window where a concurrent RCU reader (such as qrtr_reset_ports() or qrtr_port_lookup()) can obtain a pointer to the socket from the XArray, and attempt to call sock_hold() on a socket whose reference count has already dropped to zero. This exact race condition was hit during syzkaller fuzzing, leading to the following refcount saturation warning and a potential Use-After-Free: refcount_t: saturated; leaking memory. WARNING: CPU: 3 PID: 1273 at lib/refcount.c:22 refcount_warn_saturate+0xae/0x1d0 Modules linked in: qrtr(+) bochs drm_shmem_helper ... Call Trace: <TASK> qrtr_reset_ports net/qrtr/af_qrtr.c:768 [inline] [qrtr] __qrtr_bind.isra.0+0x48b/0x570 net/qrtr/af_qrtr.c:805 [qrtr] qrtr_bind+0x17d/0x210 net/qrtr/af_qrtr.c:901 [qrtr] kernel_bind+0xe4/0x120 net/socket.c:3592 qrtr_ns_init+0x1a6/0x380 net/qrtr/ns.c:715 [qrtr] qrtr_proto_init+0x3b/0xff0 net/qrtr/af_qrtr.c:169 [qrtr] do_one_initcall+0xf5/0x5e0 init/main.c:1283 ... </TASK> Fix this by deferring the reference count decrement until after the xa_erase() and the synchronize_rcu() complete. (Note: The v1 of this patch incorrectly replaced __sock_put() with sock_put(). As Simon Horman pointed out, the callers of qrtr_port_remove() still hold a reference to the socket, so freeing the socket memory here would lead to a subsequent UAF in the caller. Thus, the __sock_put() is kept, but only repositioned to close the RCU race.)
Timeline
- CVE published
- CISA SSVC assessment
Exploitation: none · Automatable: no · Technical impact: total
- Record last updated
Affected versions
- Linux Linux
From bdabad3e363d825ddf9679dd431cca0b2c30f881 · fixed in 2aa4c12723fe432e623462a3be42a197a128722b; From bdabad3e363d825ddf9679dd431cca0b2c30f881 · fixed in 03bfa95e452e2b6ccd76a332060ae4feaf5ad84d; From bdabad3e363d825ddf9679dd431cca0b2c30f881 · fixed in 474293d90880622fde9d2430fb0165767090f7b3; From bdabad3e363d825ddf9679dd431cca0b2c30f881 · fixed in 2047c2aa0963bb2872fd722300a15bcb441a4c00; From bdabad3e363d825ddf9679dd431cca0b2c30f881 · fixed in 7de2d447072be3b1a76793f034432338fc9c494b; From bdabad3e363d825ddf9679dd431cca0b2c30f881 · fixed in ab269990ed58143a92a263be1bee626d82ac03da; From bdabad3e363d825ddf9679dd431cca0b2c30f881 · fixed in 3b20ec8f31e8a6a6782243f473b0abd3463621df; From bdabad3e363d825ddf9679dd431cca0b2c30f881 · fixed in a2171131ecda1ed61a594a1eb715e75fdad0fef5
- Linux Linux
4.7
- KernelLinux
From 4.7.0 · fixed in 5.10.259; From 5.11.0 · fixed in 5.15.210; From 5.16.0 · fixed in 6.1.176; From 6.2.0 · fixed in 6.6.143; From 6.7.0 · fixed in 6.12.94; From 6.13.0 · fixed in 6.18.36; From 6.19.0 · fixed in 7.0.13
References 8
git.kernel.org
- stable/c/2aa4c12723fe432e623462a3be42a197a128722b
- stable/c/03bfa95e452e2b6ccd76a332060ae4feaf5ad84d
- stable/c/474293d90880622fde9d2430fb0165767090f7b3
- stable/c/2047c2aa0963bb2872fd722300a15bcb441a4c00
- stable/c/7de2d447072be3b1a76793f034432338fc9c494b
- stable/c/ab269990ed58143a92a263be1bee626d82ac03da
- stable/c/3b20ec8f31e8a6a6782243f473b0abd3463621df
- stable/c/a2171131ecda1ed61a594a1eb715e75fdad0fef5