Page MenuHomeFreeBSD

Check for excessively small times in kern_clock_settime().
ClosedPublic

Authored by markj on May 3 2019, 4:05 PM.
Tags
None
Referenced Files
F131643743: D20151.diff
Fri, Oct 10, 12:04 AM
Unknown Object (File)
Aug 15 2025, 4:02 AM
Unknown Object (File)
Aug 4 2025, 8:44 PM
Unknown Object (File)
Aug 3 2025, 10:29 PM
Unknown Object (File)
Aug 1 2025, 11:03 PM
Unknown Object (File)
Jul 14 2025, 11:57 AM
Unknown Object (File)
Jul 10 2025, 10:43 AM
Unknown Object (File)
Jul 5 2025, 11:07 AM
Subscribers
None

Details

Summary

This addresses a panic hit by syzkaller if wall_cmos_clock is set.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

If this causes a panic, maybe it shouldn't be conditional on allow_insane_settime? After all, syzkaller may eventually discovered sysctls.

Otherwise, LGTM.

This revision is now accepted and ready to land.May 3 2019, 4:37 PM
In D20151#433908, @cem wrote:

If this causes a panic, maybe it shouldn't be conditional on allow_insane_settime? After all, syzkaller may eventually discovered sysctls.

The commit log message for r312702 hints that allow_insane_settime already prevents a privileged user from panicking the system.

syzkaller would probably discover debug.kdb.panic first. :)

I forget why we left that knob, then. :)

This revision was automatically updated to reflect the committed changes.