]> git.ipfire.org Git - thirdparty/linux.git/commit
inet: frags: avoid theoretical race in ip_frag_reinit()
authorJakub Kicinski <kuba@kernel.org>
Sun, 7 Dec 2025 01:09:39 +0000 (17:09 -0800)
committerJakub Kicinski <kuba@kernel.org>
Wed, 10 Dec 2025 09:15:27 +0000 (01:15 -0800)
commit8ef522c8a59a048117f7e05eb5213043c02f986f
tree6cb9298eb952b574f3c38705f7aac8ddb423edaf
parent2f6e056e95ff5020260ccfd85391a6474d87e4b5
inet: frags: avoid theoretical race in ip_frag_reinit()

In ip_frag_reinit() we want to move the frag timeout timer into
the future. If the timer fires in the meantime we inadvertently
scheduled it again, and since the timer assumes a ref on frag_queue
we need to acquire one to balance things out.

This is technically racy, we should have acquired the reference
_before_ we touch the timer, it may fire again before we take the ref.
Avoid this entire dance by using mod_timer_pending() which only modifies
the timer if its pending (and which exists since Linux v2.6.30)

Note that this was the only place we ever took a ref on frag_queue
since Eric's conversion to RCU. So we could potentially replace
the whole refcnt field with an atomic flag and a bit more RCU.

Fixes: 1da177e4c3f4 ("Linux-2.6.12-rc2")
Reviewed-by: Eric Dumazet <edumazet@google.com>
Link: https://patch.msgid.link/20251207010942.1672972-2-kuba@kernel.org
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
net/ipv4/inet_fragment.c
net/ipv4/ip_fragment.c