=================================================================== RCS file: /cvsrepo/anoncvs/cvs/src/usr.bin/calendar/calendar.c,v retrieving revision 1.8 retrieving revision 1.17 diff -u -r1.8 -r1.17 --- src/usr.bin/calendar/calendar.c 1997/08/26 23:37:21 1.8 +++ src/usr.bin/calendar/calendar.c 2002/02/16 21:27:44 1.17 @@ -1,4 +1,4 @@ -/* $OpenBSD: calendar.c,v 1.8 1997/08/26 23:37:21 millert Exp $ */ +/* $OpenBSD: calendar.c,v 1.17 2002/02/16 21:27:44 millert Exp $ */ /* * Copyright (c) 1989, 1993, 1994 @@ -43,30 +43,45 @@ #if 0 static const char sccsid[] = "@(#)calendar.c 8.3 (Berkeley) 3/25/94"; #else -static char rcsid[] = "$OpenBSD: calendar.c,v 1.8 1997/08/26 23:37:21 millert Exp $"; +static char rcsid[] = "$OpenBSD: calendar.c,v 1.17 2002/02/16 21:27:44 millert Exp $"; #endif #endif /* not lint */ +#include +#include +#include #include #include #include +#include #include +#include #include #include #include #include +#include #include #include "pathnames.h" #include "calendar.h" +char *calendarFile = "calendar"; /* default calendar file */ +char *calendarHome = ".calendar"; /* HOME */ +char *calendarNoMail = "nomail"; /* don't sent mail if this file exists */ + struct passwd *pw; int doall = 0; time_t f_time = 0; +int bodun_always = 0; int f_dayAfter = 0; /* days after current date */ int f_dayBefore = 0; /* days before current date */ +struct specialev spev[NUMEV]; + +void childsig(int); + int main(argc, argv) int argc; @@ -75,23 +90,28 @@ int ch; char *caldir; - (void) setlocale(LC_ALL, ""); + (void)setlocale(LC_ALL, ""); - while ((ch = getopt(argc, argv, "-af:t:A:B:")) != -1) + while ((ch = getopt(argc, argv, "-abf:t:A:B:")) != -1) switch (ch) { case '-': /* backward contemptible */ case 'a': if (getuid()) - errx(1, strerror(EPERM)); + errx(1, "%s", strerror(EPERM)); doall = 1; break; + case 'b': + bodun_always++; + break; + case 'f': /* other calendar file */ calendarFile = optarg; break; case 't': /* other date, undocumented, for tests */ - f_time = Mktime (optarg); + if ((f_time = Mktime(optarg)) <= 0) + errx(1, "specified date is outside allowed range"); break; case 'A': /* days after current date */ @@ -115,17 +135,120 @@ if (f_time <= 0) (void)time(&f_time); - settime(f_time); + if (f_dayBefore) { + /* Move back in time and only look forwards */ + f_dayAfter += f_dayBefore; + f_time -= SECSPERDAY * f_dayBefore; + f_dayBefore = 0; + } + settime(&f_time); - if (doall) + if (doall) { + pid_t kid, deadkid; + int kidstat, kidreaped, runningkids; + int acstat; + struct stat sbuf; + time_t t; + unsigned int sleeptime; + + signal(SIGCHLD, childsig); + runningkids = 0; + t = time(NULL); while ((pw = getpwent()) != NULL) { - (void)setegid(pw->pw_gid); - (void)initgroups(pw->pw_name, pw->pw_gid); - (void)seteuid(pw->pw_uid); - if (!chdir(pw->pw_dir)) + acstat = 0; + /* Avoid unnecessary forks. The calendar file is only + * opened as the user later; if it can't be opened, + * it's no big deal. Also, get to correct directory. + * Note that in an NFS environment root may get EACCES + * on a chdir(), in which case we have to fork. As long as + * we can chdir() we can stat(), unless the user is + * modifying permissions while this is running. + */ + if (chdir(pw->pw_dir)) { + if (errno == EACCES) + acstat = 1; + else + continue; + } + if (stat(calendarFile, &sbuf) != 0) { + if (chdir(calendarHome)) { + if (errno == EACCES) + acstat = 1; + else + continue; + } + if (stat(calendarNoMail, &sbuf) == 0 || + stat(calendarFile, &sbuf) != 0) + continue; + } + sleeptime = USERTIMEOUT; + switch ((kid = fork())) { + case -1: /* error */ + warn("fork"); + continue; + case 0: /* child */ + (void)setlocale(LC_ALL, ""); + if (setusercontext(NULL, pw, pw->pw_uid, + LOGIN_SETALL ^ LOGIN_SETLOGIN)) + err(1, "unable to set user context (uid %d)", + (int)pw->pw_uid); + if (acstat) { + if (chdir(pw->pw_dir) || + stat(calendarFile, &sbuf) != 0 || + chdir(calendarHome) || + stat(calendarNoMail, &sbuf) == 0 || + stat(calendarFile, &sbuf) != 0) + exit(0); + } cal(); - (void)seteuid(0); + exit(0); + } + /* parent: wait a reasonable time, then kill child if + * necessary. + */ + runningkids++; + kidreaped = 0; + do { + sleeptime = sleep(sleeptime); + /* Note that there is the possibility, if the sleep + * stops early due to some other signal, of the child + * terminating and not getting detected during the next + * sleep. In that unlikely worst case, we just sleep + * too long for that user. + */ + for (;;) { + deadkid = waitpid(-1, &kidstat, WNOHANG); + if (deadkid <= 0) + break; + runningkids--; + if (deadkid == kid) { + kidreaped = 1; + sleeptime = 0; + } + } + } while (sleeptime); + + if (!kidreaped) { + /* It doesn't _really_ matter if the kill fails, e.g. + * if there's only a zombie now. + */ + (void)kill(kid, SIGTERM); + warnx("uid %d did not finish in time", (int)pw->pw_uid); + } + if (time(NULL) - t >= SECSPERDAY) + errx(2, "'calendar -a' took more than a day; stopped at uid %d", + (int)pw->pw_uid); } + for (;;) { + deadkid = waitpid(-1, &kidstat, WNOHANG); + if (deadkid <= 0) + break; + runningkids--; + } + if (runningkids) + warnx( +"%d child processes still running when 'calendar -a' finished", runningkids); + } else if ((caldir = getenv("CALENDAR_DIR")) != NULL) { if(!chdir(caldir)) cal(); @@ -140,6 +263,14 @@ usage() { (void)fprintf(stderr, - "usage: calendar [-a] [-A days] [-B days] [-f calendarfile] [-t [[[yy]yy][mm]]dd]\n"); + "usage: calendar [-a] [-A num] [-b] [-B num] [-t [[[cc]yy][mm]]dd] " + "[-f calendarfile]\n"); exit(1); +} + + +void +childsig(sig) + int sig; +{ }