Age | Commit message (Collapse) | Author |
|
options field was difficult to describe concisely;
- tweak wording accordingly
- sort flags
- condense STANDARDS to avoid text repitition, as suggested by deraadt
- document flags may be combined, now deraadt fixed the parsing
ok millert deraadt
|
|
-ns are now possible, as well as (useless) repetition like -nnn.
ok millert
|
|
We want to log when the job is added but will not be run, not every
time the run queue is processed.
|
|
field before command, rather than muddling up the command description to
explain them messily as some sort of exception
ok millert
|
|
This adds a new "-s" flag to the command field which indicates that
only a single instance of the job should run concurrenty.
OK beck@ job@ deraadt@
|
|
For example "0~30" will result in a random value between 0 and 30
inclusive. If either (or both) of the numbers on either side of
the '~' are omitted, the appropriate limit (low or high) for the
field will be used. OK deraadt@ beck@ jmc@
|
|
are actually locale-dependent on OpenBSD, and the programs should better
not be locale-dependent even when compiled on other systems.
millert@ points out that the month and weekday names in the fourth and
fifth columns of crontab(5) could in theory be made locale-dependent,
but we certainly don't want that, and currently, they are only compared
against static const char * arrays in entry.c containing English names.
Patch sent in by Jan Stary <hans at stare dot cz>.
OK millert@
|
|
OK Ingo Schwarze
|
|
|
|
GID_MAX-1 for good measure. OK deraadt@
|
|
|
|
value < 0. errno is only updated in this case. Change all (most?)
callers of syscalls to follow this better, and let's see if this strictness
helps us in the future.
|
|
|
|
ok deraadt@
|
|
|
|
|
|
|
|
OK jmc@ millert@
|
|
perform the fclose() in process_crontab(). Previously we were
closing the crontab fd twice--once in load_user() via fclose() and
once in process_crontab(). OK tb@
|
|
Not needed in theory, but could prevent accidental leaks. ok millert@
|
|
Fixes a failure at restart if a child process still has the socket
opened. Spotted by millert@, ok friehm@ millert@
|
|
ok jca@
|
|
|
|
a file when we scanned the at spool earlier.
|
|
by group crontab. The at(1) command now creates files owned by
group crontab, the crontab(1) command already does this.
Files in the crontab spool with parse errors are now ignored;
crontab(1) will not install a crontab file with parse errors.
The system crontab file (/etc/crontab) is not affected by this.
The required permissions on crontab files have been tightened.
Files in the cron spool must be mode 0600 (as created by crontab(1)).
The system crontab file may be readable/writable by the owner,
readable by group and readable by other. The system crontab must
be readable by the owner.
|
|
with openat(), fstatat() and unlinkat(). This is similar to how
we run cron jobs and eliminates the need for run_job() to find the
basename of the at file. OK deraadt@
|
|
group-writable, not after. Pointed out by Solar Designer.
|
|
Original patch from Wouter Clarie <wclarie at gmail com>,
tweaked in joint work with jca@,
OK millert@, OK jca@ on an earlier version.
|
|
|
|
stash a struct timespec, not just a time_t. Fixes a bug where cron
could skip re-reading the spool after two consecutive changes.
|
|
|
|
exec of shell + editor fails.
|
|
and/or syslog depending on whether stderr is hooked up at the time.
Also remove closelog() which is not needed since we are headed for exec.
OK guenther@
|
|
clearing it and then ORin in SA_RESTART.
|
|
"(username) WHAT (details)". Logs due to normal operation (e.g.
crontab operations or running commands) are logged at LOG_INFO like
before. Actual errors are logged at LOG_ERR, less important things
are logged at LOG_WARNING OR LOG_NOTICE. Also ignore SIGHUP now
that there is no log file to reopen.
|
|
|
|
chdir(2) to the cron dir and cron(8) now changes to / via daemon(3).
We no longer try to create/chmod the spool directories as they
should be set correctly at install time. The setegid(crontab)
has been moved to open_socket() so it is closer to the chmod(2)
call that needs it. OK deraadt@ tedu@
|
|
|
|
location if the new one doesn't exist for now. In order to allow
the fchown() to succeed, cron now sets its effective gid to crontab.
OK jca@ deraadt@
|
|
|
|
|
|
prod cron into checking the spool dir.
|
|
|
|
instead of calling fchown() after the fact. Fixes a pledge()
issue. OK semarie@
|
|
|
|
|
|
|
|
readability.
|
|
don't need to chdir to the queue dir. OK guenther@
|
|
some potential memory leaks in error paths. OK guenther@
|