diff options
| author | Todd C. Miller <millert@cvs.openbsd.org> | 2007-11-17 16:09:30 +0000 |
|---|---|---|
| committer | Todd C. Miller <millert@cvs.openbsd.org> | 2007-11-17 16:09:30 +0000 |
| commit | aac9512fc5b3f9778521d63eb8530e533bc65447 (patch) | |
| tree | f408fa0c9c5919a54aa108e6475e5503f3f23ce4 /usr.sbin/cron/crontab.c | |
| parent | 9b4dacab5d1ff741fa40e94687d11526501458a5 (diff) | |
Deal with timespec vs. timeval precision issues. Fixes a problem
where the "no changes" case was not properly detected due to timespec
supporting nanosec resolution but timeval only supporting microsec
resolution. OK deraadt@
Diffstat (limited to 'usr.sbin/cron/crontab.c')
| -rw-r--r-- | usr.sbin/cron/crontab.c | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/usr.sbin/cron/crontab.c b/usr.sbin/cron/crontab.c index f89b324b739..b2c97e84b82 100644 --- a/usr.sbin/cron/crontab.c +++ b/usr.sbin/cron/crontab.c @@ -1,4 +1,4 @@ -/* $OpenBSD: crontab.c,v 1.54 2007/10/17 20:02:33 deraadt Exp $ */ +/* $OpenBSD: crontab.c,v 1.55 2007/11/17 16:09:29 millert Exp $ */ /* Copyright 1988,1990,1993,1994 by Paul Vixie * All rights reserved @@ -21,7 +21,7 @@ * OF OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE. */ -static char const rcsid[] = "$OpenBSD: crontab.c,v 1.54 2007/10/17 20:02:33 deraadt Exp $"; +static char const rcsid[] = "$OpenBSD: crontab.c,v 1.55 2007/11/17 16:09:29 millert Exp $"; /* crontab - install and manage per-user crontab files * vix 02may87 [RCS has the rest of the log] @@ -311,9 +311,13 @@ edit_cmd(void) { perror("fstat"); goto fatal; } - memcpy(&mtimespec, &statbuf.st_mtimespec, sizeof(mtimespec)); + /* + * Note that timespec has higher precision than timeval so we + * store mtimespec using timeval precision so we can compare later. + */ TIMESPEC_TO_TIMEVAL(&tv[0], &statbuf.st_atimespec); TIMESPEC_TO_TIMEVAL(&tv[1], &statbuf.st_mtimespec); + TIMEVAL_TO_TIMESPEC(&tv[1], &mtimespec); /* Turn off signals. */ (void)signal(SIGHUP, SIG_IGN); |