Check for excessively small times in kern_clock_settime().
ClosedPublic
Actions

Authored by markj on May 3 2019, 4:05 PM.

Details

Reviewers

cem
imp
kib

Commits

rS347063: Disallow excessively small times of day in clock_settime(2).

Summary

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

Diff Detail

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 24087
Build 22955: arc lint + arc unit

Event Timeline

markj created this revision.May 3 2019, 4:05 PM

Harbormaster completed remote builds in B24087: Diff 57012.May 3 2019, 4:05 PM

markj added reviewers: cem, imp, kib.May 3 2019, 4:06 PM

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. :)

kib accepted this revision.May 3 2019, 6:04 PM

Closed by commit rS347063: Disallow excessively small times of day in clock_settime(2). (authored by markj). · Explain WhyMay 3 2019, 9:27 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

sys/

kern/

kern_time.c

4 lines

Diff 57012

View Options

Check for excessively small times in kern_clock_settime().ClosedPublicActions