crontab: make usage easier to read

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

New release 1.4.10.

Condtionalize pam functions.

Drop duplicate -lselinux -laudit.

Fix missing includes for selinux and other types.

Fix some issues introduced in
c58b1d9 Documentation improvements based on Michael Lass suggestions.

* give -m an own line in manpage
* don't explain -P twice
* correct description of -P to what it actually does

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

Fix const usage in crontab.

docs: anacron fix out dated information about open file limits

For quite a while open files limit has defaulted to 1024, but rather than
informing what might be the number user should inspect value from own
time.

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

docs: fix reference to other manual

There is no crontabs(4).  For user it makes most sense to look
crontab(1).

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

docs: fix various groff issues

* Add spaces after dots (English spacing).
* Remove use of inline emphasis where possible.
* Use mail and url macros.
* Wrap long and join short lines.
* Remove end of line white spaces.

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

build-sys: make ./configure to seach vi path

Using "/usr/ucb/vi" as defaut fallback EDITOR is not going to work on
most of distributions where this package is installed.  That said it
might work somewhere, so searching the vi editor at configure time is the
sensible thing to do, if user does not want to define the default.

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

debuging: make Debug macro look like function

This will make indent(1) program not to think it is a clause.

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

declarations: fix shadow declarations

user.c:68:7: warning: declaration of 'save_errno' shadows a previous local [-Wshadow]
user.c:64:14: warning: shadowed declaration is here [-Wshadow]

do_command.c:559:9: warning: declaration of 'pid' shadows a previous local [-Wshadow]
do_command.c:91:8: warning: shadowed declaration is here [-Wshadow]

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

includes: make inclusions clean

This change makes files to include what they need, instead of having
generic inclusion headers which will include everything to everywhere.
Also the local headers havee ifndef & define protection, which makes them
to be save to include in any file, without one having to think in which
order the inclusions happen at compile time.

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

Fix up also the const in comment.

variables: use const where ever possible

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

build-sys: reduce selinux and pam compiling scope when possible

This will fix following warnings, and makes compiled program to contain
less unnecessary references.

crontab.c:60:14: warning: 'selinux_context' defined but not used [-Wunused-variable]
security.c:188:12: warning: 'cron_open_pam_session' defined but not used [-Wunused-function]
security.c:250:12: warning: 'cron_authorize_context' defined but not used [-Wunused-function]
security.c:283:12: warning: 'cron_authorize_range' defined but not used [-Wunused-function]

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

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