sch_htb: make htb_deactivate() idempotent
- 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 6.1.139, 6.6.91, 6.12.29, 6.14.7
- 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
Summary
In the Linux kernel, the following vulnerability has been resolved: sch_htb: make htb_deactivate() idempotent Alan reported a NULL pointer dereference in htb_next_rb_node() after we made htb_qlen_notify() idempotent. It turns out in the following case it introduced some regression: htb_dequeue_tree(): |-> fq_codel_dequeue() |-> qdisc_tree_reduce_backlog() |-> htb_qlen_notify() |-> htb_deactivate() |-> htb_next_rb_node() |-> htb_deactivate() For htb_next_rb_node(), after calling the 1st htb_deactivate(), the clprio[prio]->ptr could be already set to NULL, which means htb_next_rb_node() is vulnerable here. For htb_deactivate(), although we checked qlen before calling it, in case of qlen==0 after qdisc_tree_reduce_backlog(), we may call it again which triggers the warning inside. To fix the issues here, we need to: 1) Make htb_deactivate() idempotent, that is, simply return if we already call it before. 2) Make htb_next_rb_node() safe against ptr==NULL. Many thanks to Alan for testing and for the reproducer.
Timeline
- CVE published
- Record last updated
Affected versions
- Linux Linux
Version range not reported
- Linux Linux
From 6.1.138 · fixed in 6.1.139; From 6.6.90 · fixed in 6.6.91; From 6.12.28 · fixed in 6.12.29; From 6.14.6 · fixed in 6.14.7
References 8
git.kernel.org
- stable/c/99ff8a20fd61315bf9ae627440a5ff07d22ee153
- stable/c/a9945f7cf1709adc5d2d31cb6cfc85627ce299a8
- stable/c/c2d25fddd867ce20a266806634eeeb5c30cb520c
- stable/c/c928dd4f6bf0c25c72b11824a1e9ac9bd37296a0
- stable/c/31ff70ad39485698cf779f2078132d80b57f6c07
- stable/c/98cd7ed92753090a714f0802d4434314526fe61d
- stable/c/c4792b9e38d2f61b07eac72f10909fa76130314b
- stable/c/3769478610135e82b262640252d90f6efb05be71