diff options
| author |   cheloha <cheloha@openbsd.org> | 2020-10-26 16:17:00 +0000 |
|---|---|---|
| committer | cheloha <cheloha@openbsd.org> | 2020-10-26 16:17:00 +0000 |
| commit | e5476f2390163dc26aac81900a26e09a7310fa80 (patch) | |
| tree | 25424312b37d9d3c52bb6374fefee70ed37ca09d | |
| parent | The agentx API will first appear in 6.9. (diff) | |
| download | wireguard-openbsd-e5476f2390163dc26aac81900a26e09a7310fa80.tar.xz wireguard-openbsd-e5476f2390163dc26aac81900a26e09a7310fa80.zip | |
kevent(2): ktrace the timeout before validating it
As deraadt@ has pointed out, tracing timevals and timespecs before
validating them makes debugging easier.
| -rw-r--r-- | sys/kern/kern_event.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/sys/kern/kern_event.c b/sys/kern/kern_event.c index 7cb7dbf1cc9..d15d7ea3006 100644 --- a/sys/kern/kern_event.c +++ b/sys/kern/kern_event.c @@ -1,4 +1,4 @@ -/* $OpenBSD: kern_event.c,v 1.143 2020/10/11 07:11:59 mpi Exp $ */ +/* $OpenBSD: kern_event.c,v 1.144 2020/10/26 16:17:00 cheloha Exp $ */ /*- * Copyright (c) 1999,2000,2001 Jonathan Lemon <jlemon@FreeBSD.org> @@ -581,14 +581,14 @@ sys_kevent(struct proc *p, void *v, register_t *retval) error = copyin(SCARG(uap, timeout), &ts, sizeof(ts)); if (error) goto done; - if (ts.tv_sec < 0 || !timespecisvalid(&ts)) { - error = EINVAL; - goto done; - } #ifdef KTRACE if (KTRPOINT(p, KTR_STRUCT)) ktrreltimespec(p, &ts); #endif + if (ts.tv_sec < 0 || !timespecisvalid(&ts)) { + error = EINVAL; + goto done; + } tsp = &ts; } |