Re: [PATCH net] net/sched: act_api: use mutex in tcf_idr_check_alloc
From: Jakub Kicinski
Date: Thu May 28 2026 - 21:14:27 EST
On Tue, 26 May 2026 15:08:47 -0700 Kyle Zeng wrote:
> Currently, the NEWTFILTER path uses RCU to guard action idr accesses while
> the DELTFILTER path uses mutex to guard action accesses. This
> inconsistency leads to a race condition scenario, which can lead to
> erroneous operations on refcount, eventually leading to use-after-free
> situation.
> In this patch, we revert the introduction of RCU back to mutex in the
> NEWFILTER path, which is consistent with the DELFILTER path, avoiding
> the race condition.
The commit message is quite inadequate here. Looks like a
run-of-the-mill UAF so you should explain the flow / race that leads
to it properly.
Doing some extra digging with Jamal off-list we can't find the reason
why normal RCU protection wouldn't work here so maybe hold off reposting
until you hear from Jamal.