do not log carriage return

Some commands (like ssh) end their output with carriage return (\r) and
new line (\n). Just cutting the new line before sending to syslog may
have undesired effects of unprintable characters. Reading cronie logs
with journalctl I can see:

Sep 06 16:50:01 linux CROND[152575]: [96B blob data]

Adding switch --all for unprintable characters it shows:

Sep 06 16:50:01 linux CROND[152575]: (root) CMDOUT (Warning: Permanently
added '10.52.33.217' (ED25519) to the list of known hosts.^M)

Just skip carriage return for syslog to prevent this.

skip directories when given as crontabs

crontab -l colors comment lines in a different color

fix compile instruction: configure is generated by autogen.sh

getdtablesize() can return very high values in containers

Avoid closing hundreds of millions descriptors or allocating
huge arrays by maxing the fd number to MAX_CLOSE_FD.
See rhbz#1723106

Power supply name not detected correctly.

Use a number range to detect ADP names rather than just 'ADP0'. On my laptop the name 'ADP1' is used.

Revert "Avoid creating pid files when crond doesn't fork"

This reverts commit 5b285b46b88dc63689c6a56542cb2ba81f861b66.
The PID file is useful to avoid running multiple crond instances
at once.

anacron: fix types in comparisons

This fixes warnings reported by -Wsign-compare compiler option.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>
Signed-off-by: Tomas Mraz <tmraz@fedoraproject.org>

mics: stop using plain integer as NULL pointer

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

cron: fix type in comparison

Use of size_t in this context should be also safer from bit shift
perspective, risk of overflowing it is smaller.

cronnext.c:80:16: warning: comparison between signed and unsigned integer
expressions [-Wsign-compare]
  for (f = 1; f < sizeof(flagname);  f = f << 1)
                ^

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

cron: fix variable shadowing

cronnext.c: In function 'matchuser':
cronnext.c:193:21: warning: declaration of 'user' shadows a global declaration [-Wshadow]
 int matchuser(char *user, char *list) {
                     ^~~~
In file included from funcs.h:37:0,
                 from cronnext.c:32:
structs.h:71:3: note: shadowed declaration is here
 } user;

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

cron: mark signal number arguments in signal handlers unused

The ATTRIBUTE_UNUSED is declared in cronie_common.h file.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

Update readme.md with recent releases

Move allowed() and related function to security.c

It makes more sense to have it there and it also uses audit and is
not called from cronnext.

build-sys: enable anacron by default

It is reasonable to think most users want to build anacron.  Notice that
this change will not effect any to packaging people who might have
--enable-anacron in their build setup.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

build-sys: use none-recursive build

None-recursive builds are quicker, and less problematic from make(1) object
tracking point of view.  See reference for full explanation.

References: http://miller.emu.id.au/pmiller/books/rmch/
Signed-off-by: Sami Kerola <kerolasa@iki.fi>

cronnext: optionally select jobs by substring

crond: Do not abort loading crontab prematurely in case of error

Release new version 1.5.4

crond: Fix regression from last release

Only first job from a crontab was being run.

Release new version 1.5.3

Fix warnings in anacron and crond code.

Fix CVE-2019-9704 and CVE-2019-9705

The users can cause DoS of the crond by loading huge crontab files.
We now allow maximum 1000 environment variables and 1000 crontab entries.
Also the comments and whitespace between the entries and variables
are now limited to 32768 characters.

Remove leftover debugging message.

Make crond run after autofs mounts

crontab: Make crontab without arguments fail.

Although this deviates from POSIX specification it is clearly more sane
behavior.

Use system-auth instead of password-auth for PAM authentication (#25)

Restart crond if it fails

Set up the systemd service file, so crond is automatically restarted after 30 seconds if it crashes.

load_env: parse the environment variables in-place

The environment variable parsing can be easily done in-place to
avoid allocating two large buffers on stack.
Fixes: #18

Update readme.md with the latest release and better link to Fedora

readme.md: Replace dead link to openSUSE ML with package link

Use the role from the crond context for system job contexts.

New SELinux policy added multiple roles for the system_u user on crond_t.
The default context returned from get_default_context_with_level() is now
unconfined_t instead of system_cronjob_t which is incorrect for system cron
jobs.
We use the role to limit the default context to system_cronjob_t.

Fix unsafe code

- drop usage of huge sparse array
- fix infinite loop in error case
- missing '\0' termination of mail command option
- missing check for fseek() failure

Fix possible memory leaks

Release new version 1.5.2

Silence may be used uninitialized warning.

crontab: do not try to replace the crontab with a directory

Using 'crontab .' removes the existing crontab as an unwanted
side-effect, there is 'crontab -r' for crontab removal.

Support the MAILFROM also in anacron.

Resolves github issue #17

Document that in range the first number must be less than second.

round time on the minute

print all jobs executed in interval

new options -l and -q

output in yaml format

print the job(s) that is going to be executed next

allow checking a time interval

print format for flags

cronnext can read additional crontabs from files

Use proper variable types.

Add myself as author.

Log startup even when started in non-forking mode.

Fix SELinux denial when running under Docker

When crond is running insider a Docker container hosted on
on a system with SELinux enabled (such as Fedora 26), the
following denial occurs each time crond executes a job:

2017-07-14T13:42:01.931319+00:00 f26docker auth.notice audit: AVC avc:  denied  { setexec } for  pid=16248 comm="crond" scontext=system_u:system_r:container_t:s0:c525,c757 tcontext=system_u:system_r:container_t:s0:c525,c757 tclass=process permissive=0

The denials get flagged as a problem by system management tools.

This patch fixes the problem by ensuring that SELinux is
enabled before trying to restore the default security context.

Software versions:
- cronie-1.5.1-5.fc26.x86_64
- selinux-policy-targeted-3.13.1-259.fc26.noarch
- docker-1.13.1-19.git27e468e.fc26.x86_64
- kernel-4.11.9-300.fc26.x86_64

See also
https://bugzilla.redhat.com/show_bug.cgi?id=1435516

build-sys: use silent build rule

Thsi makes make output less convoluted.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

cronnext: improve manual page

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

all: add version printing option -V

Also mention the option in usage() and manual pages.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

cronnext: use same style in usage() with other commands

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

various files: fix indentation

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

crond: never define unused cron_start_pam()

Move function completely within #ifdef just like all the code where this
function is used.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

Add Arch Linux to list of distributions

Fix misleading indentation.

Make cronnext build and work with additional features enabled.

Add MIME-Version header also to mails generated by anacron.

Add MIME-Version header to generated mails.

Fixes GitHub issue #5.

The output might not be mailed, make the message more precise.

Allow empty environment values in crontab.

Point releases to the GitHub cronie releases page

crontab.5: Add a mention about disabling logging

Mention possibility of disabling logging to syslog in a crontab
manual.

This change originally comes from cronie-1.4.7-disable_logging.patch
added to openSUSE by Vítězslav Čížek (vcizek@suse.com).

Avoid creating pid files when crond doesn't fork

When the cron daemon does not fork, as it is the case when using
systemd, pid files are useless. Avoid creating them in the first
place.

This change originally comes from cronie-nofork-nopid.patch
added to openSUSE by Cristian Rodríguez (crrodriguez@opensuse.org).

readme.md: remove CRs

Add readme.md for github

Make it clear that one of the reasons for getpwnam() failure is unknown user.

On some machines the power supply is named ADP0.

Query power supply status directly from kernel.

cronnext: Fix warnings

cronnext: Print usage and error on stderr.

Added useful utility cronnext to find out time of the next job run.

anacron: Check return value of fchdir.

Release new version 1.5.1

Remove unneeded obsolete SELinux include.

Remove unneeded obsolete SELinux includes.

Rename timeval time to tv so time function can be used.

Fix longstanding race condition on crontab modification.

If crontab is modified twice at the same second the crond reads it
which means the timestamp will not change, the latest content will
never be read.

Fix misleading comment in pam configuration file.

anacron: use mkstemp() instead of tempnam()

Although the use of tempnam() seemed to be safeguarded
better to not use it at all.

anacron: Drop potentially unsafe umask setting.

anacron: Do not try to mail to empty address.

Clarify the crontab(1) documentation.

Remove hardcoded "system_u" SELinux user

This gets rid of the hardcoded reference to the ref-policy system_u user
and instead relies on the user that cronie is currently running as.

Clarify return value.

Correct instructions for disabling Anacron.

Man page syntax fixes

Inherit PATH from the crond environment if -P option is used.

In crontab command use a temporary filename that is ignored by crond.

Add *.patch and *.orig to .gitignore

Add compile to .gitignore.

Release new version 1.5.0

- properly create dist tarball even with anacron and PAM disabled

Merge tag 'cronie1.4.12'

Pass the job environment also when executing the sendmail.

database.c: Use POSIX NAME_MAX instead of BSD MAXNAMLEN

fixes compilation with musl libc

Prevent adding duplicate orphans on reload.

Do not overwrite the last zero-byte.