Nixpkgs security tracker

Login with GitHub
⚠️ You are using a production deployment that is still only suitable for demo purposes. Any work done in this might be wiped later without notice.

Suggestion detail

Dismissed
(max. allowed matches exceeded)
created 3 weeks ago Activity log
  • Created & dismissed (max. allowed matches exceeded) suggestion
fs/fcntl: fix SOFTIRQ-unsafe lock order in fasync signaling

In the Linux kernel, the following vulnerability has been resolved: fs/fcntl: fix SOFTIRQ-unsafe lock order in fasync signaling A SOFTIRQ-safe to SOFTIRQ-unsafe lock order deadlock can occur in send_sigio() and send_sigurg() when a process group receives a signal. When FASYNC is configured for a process group (PIDTYPE_PGID), both functions use read_lock(&tasklist_lock) to traverse the task list. However, they are frequently called from softirq context: - send_sigio() via input_inject_event -> kill_fasync - send_sigurg() via tcp_check_urg -> sk_send_sigurg (NET_RX_SOFTIRQ) The deadlock is caused by the rwlock writer fairness mechanism: 1. CPU 0 (process context) holds read_lock(&tasklist_lock) in do_wait(). 2. CPU 1 (process context) attempts write_lock(&tasklist_lock) in fork() or exit() and spins, which blocks all new readers. 3. CPU 0 is interrupted by a softirq (e.g., TCP URG packet reception). 4. The softirq calls send_sigurg() and attempts to acquire read_lock(&tasklist_lock), deadlocking because CPU 1 is waiting. Since PID hashing and do_each_pid_task() traversals are already RCU-protected, the read_lock on tasklist_lock is no longer strictly required for safe traversal. Fix this by replacing tasklist_lock with rcu_read_lock(), aligning the process group signaling path with the single-PID path. This also mitigates a potential remote denial of service vector via TCP URG packets. Lockdep splat: ===================================================== WARNING: SOFTIRQ-safe -> SOFTIRQ-unsafe lock order detected [...] Chain exists of: &dev->event_lock --> &f_owner->lock --> tasklist_lock Possible interrupt unsafe locking scenario: CPU0 CPU1 ---- ---- lock(tasklist_lock); local_irq_disable(); lock(&dev->event_lock); lock(&f_owner->lock); <Interrupt> lock(&dev->event_lock); *** DEADLOCK ***

Affected products

Linux
  • =<6.18.*
  • =<7.1.*
  • <7.0.13
  • <36c1b57b2ecf3c61ac93f5f07bd29b6f21e226ed
  • =<7.0.*
  • <32dbd5ce4be3a3ed7e00f8af18795cc84fc50a33
  • <897d6a7247739fb1528f98c575df4f2e5de7f994
  • <6.18.36
  • =<6.1.*
  • <1bee417678f1135e35b25a37734db46aa94258d2
  • <6.6.143
  • <5.10.259
  • <6.12.94
  • <54626335ea4174ab2d9a183b511d825f6765e47b
  • <b5fa9e32fb6718f70c986ee14dd5d01b4846f331
  • <bfcc8e8d8a495bb34cae9e620adfb75fb13a3954
  • <5.15.210
  • <20a93e397abe850c49b6fa0e8cc827b5f634a8f5
  • =<5.15.*
  • <7.1.1
  • <6.1.176
  • =<6.12.*
  • =<5.10.*
  • =<6.6.*