On Mon, Jun 8, 2026 at 11:55 AM Jamal Hadi Salim jhs@mojatatu.com wrote:
Thanks for reporting this.
You're welcome!
Unless i am looking at the wrong code version, in the current code i see m_eaction is always initialized before being used. m_eaction = READ_ONCE(m->tcfm_eaction); Probably a compiler false positive?
No, it is an actual bug -- you have:
int i, m_eaction; ... is_redirect = tcf_mirred_is_act_redirect(m_eaction); ... m_eaction = READ_ONCE(m->tcfm_eaction);
i.e. the assignment goes later.
Maybe you are looking at mainline? Please note that this is a 6.12.y -rc thread -- are you looking at the commit hash I mentioned?
Or maybe you are looking at `tcf_blockcast()` instead of `tcf_mirred_act()`?
I hope that helps.
Cheers, Miguel