Page MenuHomeFreeBSD

proc: Disallow re-enabling of process itimers during exit
ClosedPublic

Authored by markj on Mar 27 2025, 10:32 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Nov 20, 5:45 PM
Unknown Object (File)
Thu, Nov 20, 5:44 PM
Unknown Object (File)
Thu, Nov 20, 5:43 PM
Unknown Object (File)
Thu, Nov 20, 5:43 PM
Unknown Object (File)
Thu, Nov 20, 5:31 PM
Unknown Object (File)
Tue, Nov 18, 8:09 PM
Unknown Object (File)
Tue, Nov 4, 4:56 PM
Unknown Object (File)
Sun, Nov 2, 3:32 AM
Subscribers

Details

Summary

During process exit, it's possible for the exiting thread to send a
signal to its process, via killjobc(). This happens after the itimer is
drained. If itimers are stopped, i.e., P2_ITSTOPPED is set, then
itimer_proc_continue() will resume the callout after it has been
drained.

Fix the problem by simply clearing P2_ITSTOPPED as part of the drain.
Then, a signal received after that point will not re-enable the callout.

Reported by: syzkaller

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

I think this is fine.
But also I suggest to add the check for P_WEXIT/P_KILLED before re-arming the itimer callouts.

This revision is now accepted and ready to land.Mar 27 2025, 4:59 PM

Check for P_WEXIT in realitexpire_reset_callout() too.

This revision now requires review to proceed.Mar 28 2025, 8:54 AM
This revision is now accepted and ready to land.Mar 28 2025, 3:22 PM