Description
In the Linux kernel, the following vulnerability has been resolved: netrom: Fix a memory leak in nr_heartbeat_expiry() syzbot reported a memory leak in nr_create() [0]. Commit 409db27e3a2e ("netrom: Fix use-after-free of a listening socket.") added sock_hold() to the nr_heartbeat_expiry() function, where a) a socket has a SOCK_DESTROY flag or b) a listening socket has a SOCK_DEAD flag. But in the case "a," when the SOCK_DESTROY flag is set, the file descriptor has already been closed and the nr_release() function has been called. So it makes no sense to hold the reference count because no one will call another nr_destroy_socket() and put it as in the case "b." nr_connect nr_establish_data_link nr_start_heartbeat nr_release switch (nr->state) case NR_STATE_3 nr->state = NR_STATE_2 sock_set_flag(sk, SOCK_DESTROY); nr_rx_frame nr_process_rx_frame switch (nr->state) case NR_STATE_2 nr_state2_machine() nr_disconnect() nr_sk(sk)->state = NR_STATE_0 sock_set_flag(sk, SOCK_DEAD) nr_heartbeat_expiry switch (nr->state) case NR_STATE_0 if (sock_flag(sk, SOCK_DESTROY) || (sk->sk_state == TCP_LISTEN && sock_flag(sk, SOCK_DEAD))) sock_hold() // ( !!! ) nr_destroy_socket() To fix the memory leak, let's call sock_hold() only for a listening socket. Found by InfoTeCS on behalf of Linux Verification Center (linuxtesting.org) with Syzkaller. [0]: https://syzkaller.appspot.com/bug?extid=d327a1f3b12e1e206c16
Affected products
Version statuses are supplied by the assigning organization. Consult the original record for detailed ranges and platform conditions.
Linux Linux
Default version status: unaffected
a31caf5779ace8fa98b0d454133808e082ee7a1b to < d616876256b38ecf9a1a1c7d674192c5346bc69caffectedfe9b9e621cebe6b7e83f7e954c70f8bb430520e5 to < e07a9c2a850cdebf625e7a1b8171bd23a8554313affected7de16d75b20ab13b75a7291f449a1b00090edfea to < 5391f9db2cab5ef1cb411be1ab7dbec728078fbaaffectedd2d3ab1b1de3302de2c85769121fd4f890e47ceb to < 280cf1173726a7059b628c610c71050d5c0b6937affected51e394c6f81adbfe7c34d15f58b3d4d44f144acf to < a02fd5d775cf9787ee7698c797e20f2fa13d2e2baffected409db27e3a2eb5e8ef7226ca33be33361b3ed1c9 to < b6ebe4fed73eedeb73f4540f8edc4871945474c8affected409db27e3a2eb5e8ef7226ca33be33361b3ed1c9 to < d377f5a28332954b19e373d36823e59830ab1712affected409db27e3a2eb5e8ef7226ca33be33361b3ed1c9 to < 0b9130247f3b6a1122478471ff0e014ea96bb735affectede666990abb2e42dd4ba979b4706280a3664cfae7affected4.19.272 to < 4.19.317affected5.4.231 to < 5.4.279affected5.10.166 to < 5.10.221affected5.15.91 to < 5.15.162affected6.1.9 to < 6.1.96affected4.14.305 to < 4.15affected
Linux Linux
Default version status: affected
6.2affected0 to < 6.2unaffected4.19.317 to ≤ 4.19.*unaffected5.4.279 to ≤ 5.4.*unaffected5.10.221 to ≤ 5.10.*unaffected5.15.162 to ≤ 5.15.*unaffected6.1.96 to ≤ 6.1.*unaffected6.6.36 to ≤ 6.6.*unaffected6.9.7 to ≤ 6.9.*unaffected6.10 to ≤ *unaffected
References
- https://git.kernel.org/stable/c/d616876256b38ecf9a1a1c7d674192c5346bc69c ↗
- https://git.kernel.org/stable/c/e07a9c2a850cdebf625e7a1b8171bd23a8554313 ↗
- https://git.kernel.org/stable/c/5391f9db2cab5ef1cb411be1ab7dbec728078fba ↗
- https://git.kernel.org/stable/c/280cf1173726a7059b628c610c71050d5c0b6937 ↗
- https://git.kernel.org/stable/c/a02fd5d775cf9787ee7698c797e20f2fa13d2e2b ↗
- https://git.kernel.org/stable/c/b6ebe4fed73eedeb73f4540f8edc4871945474c8 ↗
- https://git.kernel.org/stable/c/d377f5a28332954b19e373d36823e59830ab1712 ↗
- https://git.kernel.org/stable/c/0b9130247f3b6a1122478471ff0e014ea96bb735 ↗
- https://lists.debian.org/debian-lts-announce/2025/01/msg00001.html ↗
- https://cert-portal.siemens.com/productcert/html/ssa-265688.html ↗
- https://cert-portal.siemens.com/productcert/html/ssa-613116.html ↗
- https://cert-portal.siemens.com/productcert/html/ssa-355557.html ↗