compliancy: do not mix declarations and code

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

clean up: reduce variable passing when they are not used

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

assingments: remove initialization if value is never used

The next operation with these variables will overwrite the values set at
init.

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

compliancy: use memset() rather than bzero()

Reference: http://pubs.opengroup.org/onlinepubs/9699919799/xrat/V4_xsh_chap03.html
Signed-off-by: Sami Kerola <kerolasa@iki.fi>

build-sys: make clean needs to remove cron-paths.h

The file is generated at build time.  This change is needed to make
distcheck work without an error.

[...]
ERROR: files left in build directory after distclean:
./src/cron-paths.h

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

build-sys: add autotools helper script

The script will help building when source code is checked out from git.

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

Do not log that syslog will be used instead of sendmail if explicitly requested.

Enable system crontab by default and use AC_ARG_ENABLE instead of AC_ARG_WITH.

Minor release 1.4.9.

Documentation improvements based on Michael Lass suggestions.

Add time-sync.target. If synchronization is used, crond will start after sync by ntpd/chrony. rhbz#872516

The sysadmin may want to arrange for the PATH to be correct in the
environment from which cron is launched.  This new option tells
cron to not overwrite this path for itself or for the child cron
jobs.

Signed-off-by: Marcela Mašláňová <mmaslano@redhat.com>

The enable-relro erased previously set LDFLAGS.

Fix typo in man page.

rhbz#836292
Thanks to: Dave Allan

Preferred hour of start of the job. If the hour is missed job will be skipped.

Range of hours for daily jobs must have lower-higher hour.

Crontab has wrong permission.

Based on rhbz#676081

syscrontab, the system crontab, is not mandatory anymore.

Cronie can be build without it. The patch was based on complaints of distribution without /etc/crontab file.

https://fedorahosted.org/cronie/ticket/10

Sendmail doesn't have to be installed.

The info message should make sense.

https://fedorahosted.org/cronie/ticket/15

Previously, crond exited in the signal handler for SIGINT and SIGTERM.
Thus, the Inotify close code in main() was never reached.  This commit
introduces a got_sigintterm variable that is set when SIGINT or SIGTERM
are received and leads to a clean exit of the main loop.

* src/cron.c: Rename quit() to sigintterm_handler().  Add got_sigintterm
variable and set it on SIGINT and SIGTERM.  Shorten sleep and exit main
loop on got_sigintterm.

Signed-off-by: Marcela Mašláňová <mmaslano@redhat.com>

9th bit is overwritten.

Hour bit fields have 24 bits (0..23, 3 bytes). bit_nclear() however in this case is called to clear bits 0..24! Thus, it overwrites the first bit of the following struct member (dom). This is not problematic for normal use as dom is set after hour as are all other fields.

The last field, dow, uses 8 bits so 9 are cleared. This could be the MIN_STAR bit of flags, but as flags is an int, there is probably a padding byte in between; at least I couldn't trigger it to be unset. So this could probably only cause problems on 8-bit systems.

https://fedorahosted.org/cronie/ticket/11
Signed-off-by: Marcela Mašláňová <mmaslano@redhat.com>

Remove unused variables.

Signed-off-by: Marcela Mašláňová <mmaslano@redhat.com>

Fix use of "const char *" in call to putenv().

Signed-off-by: Marcela Mašláňová <mmaslano@redhat.com>

The mail header for automatic reply will now contain another field:
"Precedence: bulk", which is not compulsory according to RFC3834.

Based on: Harald Reindl request in rhbz#811186

setkeycreatecon is not needed. Users should inherited key context if it was already set for them.

Signed-off-by: Marcela Mašláňová <mmaslano@redhat.com>

systemd unit file was added into contrib

This unit file is used in Fedora. The systemd-user-sessions.service guarantees
everything, what is needed for log in of users, is already running
(ypbind, sssd, ldap, ...).

adjust documentation title to reference "crontab" instead of "anacrontab"

Signed-off-by: Marcela Mašláňová <mmaslano@redhat.com>

Adjust the documentation to better describe the current situation with crontab and anacrontab.

Add name of file, where can be set PAM configuration.

Service restart needlessly reports failure.

Signed-off-by: Marcela Mašláňová <mmaslano@redhat.com>

Raw context should be used. Especially when mcstransd in older
releases is running.

No need to have full path in the ProgramName.

Merge branch 'master' of ssh://git.fedorahosted.org/git/cronie

Fix wrong condition in cron_set_unwatched(). Do not disable inotify when crontabs are missing.

Revert "Cronie disables inotify support when the /etc/crontab file does not exist at startup."

This reverts commit acdf4ae8456888ed78201906ef528f4c28f54582 as it introduces
fd leaks.

Minor release 1.4.8

Cron writes job output to syslog incorrectly. When cron is invoked in a way to print job output to syslog, it does print only the first character of the output. Author: Vitezslav Cizek

Signed-off-by: Marcela Mašláňová <mmaslano@redhat.com>

Check orphaned crontabs for adoption.

Unify logging in case of SyslogOutput with the rest of crond.

Change of email adress of cluster support author.

The charset of anacron's mail is always ANSI_X3.4-1968. There are no setlocale is anacron's source.

Signed-off-by: Marcela Mašláňová <mmaslano@redhat.com>

Cronie disables inotify support when the /etc/crontab file does not exist at startup.

Existence of crontab and directories wasn't controlled before creating
inotify watches.

Clean configure.

New release 1.4.7.

Split relro and pie into two options.

Add missing va_end() call.

Remove unused variable.

Fix incorrect pointer in sizeof.

Fixed leaking of env and members of entry in replace_cmd().

Fix memory leaks in load_user.

The last bit to set is always LAST_ - FIRST_.

Check malloc return for NULL.

Do not try to compile dead code.

Rename set_cron_cwd() to check_spool_dir() as we do not do any chdir
anymore.

Add missing check for stat() return value.

Safeguard for system time changes. Otherwise anacron might wait with the job
running for a too long time.

Fix the non-default timezone support. Do not run jobs multiple times
on DST change.

Set mailfrom to the user account, better PAM logging.

Set only groups in the process handling PAM calls. Keep uids at 0
so the process is not killable by the user.

Whitespace cleanups.

Fix SIGPIPE handling in do_command() and popen.
Ensure that PAM session is always closed.

Comment not freeing mailfrom - found during code review.

Free security contexts.

mkstemp needs only 6 X's

fdin could be tested before initialization.

PIE and RELRO flags to be set

We want all long running daemons to have PIE and RELRO flags set. This is a
missing security mechanism. Fixing this generally involves adding -fPIE -DPIE
to the CFLAGS, and -pie -Wl,-z,relro -Wl,-z,now  to the LDFLAGS.
Expected results:
PIE and partial RELRO at a minimum.

Add translation for remaining hardcoded contexts and av bits.

Revert "Cache the security class and bit values and translate the remaining hardcoded values."
Caching cannot be done as the values can change on SELinux policy update.

This reverts commit b15f72976965d2ae1a1273558bf45be7de077b79.

Revert "Missing exit if cron_init_security() fails."
Caching cannot be done as the values can change on SELinux policy update.

This reverts commit ac70de36ec6c403c28291689701bd2567c565107.

Missing exit if cron_init_security() fails.

Cache the security class and bit values and translate the remaining hardcoded values.

Clean up unnecessary assignment.

This patch causes cronie to ask kernel for constant definition
rather then using hard coded

Also add info to syslog message to help diagnose problems.

Signed-off-by: Marcela Mašláňová <mmaslano@redhat.com>

Do not lock jobs that fall out of allowed range - rhbz#661966

Man - another typo

Re-add missing option into man and fix some typos.

Thanks to Colin Dean.

New release 1.4.6

Rewrite of man pages & correction.

Signed-off-by: Marcela Mašláňová <mmaslano@redhat.com>

Remove useless man page.

Thanks to Colin Dean.

Apply previous patch correctly.

Check clustering before un/watch function

Signed-off-by: Marcela Mašláňová <mmaslano@redhat.com>

Merge branch 'cluster'

Check clustering before un/watch function

man page typo: change to correct option

Remove cluster support from inotify_database

check_inotify_database is called only when inotify is on, which
is not with cluster support.

The crontab command uses "-c" and "-n" instead of "-h".

Signed-off-by: Marcela Mašláňová <mmaslano@redhat.com>

Correct ifdef HAVE_FCHOWN

Based on https://fedorahosted.org/cronie/ticket/7
Thanks to crrodriguez

is_local set to zero

cronie on cluster

On issue we have is that although we've made our services very
resilient, by employing HA failover, load balancing and round robin DNS,
the one service that's difficult to do anything with is cron, because it
has traditionally been tied to a single machine.

For instance, we have a cluster of 4 Fedora servers which our end users
use as timeshare systems, using round robin DNS, and if one of the
servers is down it doesn't really matter too much. We don't even backup
the servers, relying on NFS home directories and rebuilding from scratch
using kickstart + cfengine if a server fails and can't be restarted.

However, the users have scattered crontab files around the 4 servers, so
that if the particular server a cron job is meant to run on dies, the
job doesn't run any more, and the crontab file may be permanently lost.

What we needed was a facility to allow crontabs in /var/spool/cron on
these 4 servers be NFS mounted from a single directory on our NetApp
filers (so giving us backups and snapshots), with any user able to run
"crontab -e" from any of the servers to manage a single shared crontab,
and for us in the IT Service to be able to set just one of the 4 servers
to run user crontab jobs at any time. However, we needed to keep
/etc/crontab and the files in /etc/cron.d/ specific to each individual
server still, and keep crond running on all 4 servers.

Signed-off-by: Marcela Mašláňová <mmaslano@redhat.com>

Typo in log message

Based on Tom London report https://bugzilla.redhat.com/show_bug.cgi?id=626947

Close leaking file descriptor

anacron was leaking fd, which caused denial of jobs by selinux.

New minor release 1.4.5.

OK value printed twice

Fix based on 615107. There were too many OK's messages.

Help and usage fix

Option -h was added. Also details about options were added into usage.

man crond -i

Document missing option.

Syslog output will be used instead of mail

If sendmail isn't installed, syslog is used. This patch should
also solve problem with RPM requirements of sendmail
(and which mail should be installed by default).

Based on: https://bugzilla.redhat.com/show_bug.cgi?id=472710#c42

Remove the whole unused part.

Remove 'dont log' part which probably never worked.

Beautify error outputs.

Option "-" don't log jobs as root

If the uid of the owner is 0 (root), he can put a "-" as first
character of a crontab entry. This will prevent cron from writing
a syslog message about this command getting executed.

Signed-off-by: Marcela Mašláňová <mmaslano@redhat.com>