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.

Log the crond shutdown.

No need to call strcmp on already matched variables.

Use unbiased random number for RANDOM_DELAY.

- also fix possible division by 0

Call PAM only when it makes sense.

- do not check PAM in crontab when uid is 0
- do not call PAM at all in crond for system cron jobs that are
  run as uid 0

Fix broken test for match_rx() failure.

Properly check the existence of the user at the time the job is run.

And do not ignore jobs for users that were not existing at database
reload.

New release 1.4.12.

COPYING also contains COPYING.anacron, because COPYING should contain all used licenses.

Services must be running before starting crond, otherwise many jobs is not running properly for no obvious reason.

nss-user-lookup.target should help when people are authenticated through nss/sssd.

Based on: https://bugzilla.redhat.com/show_bug.cgi?id=1067236

More logging when getpwnam fails.

Based on: rhbz#1123984

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

BSD doesn't define environ in unistd.h. Also cronie didn't have option
for enabling BSD specifics.

Based on: naota@gentoo.org

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

Add missing brackets into anacron readtab.

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

Include missing header for signal handlers.

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

Missing reload definition in unit file was added.

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

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

Close all unwanted open file descriptors before executing commands and mail.

There might be file descriptors open by things like pam_systemd that
we do not want to keep open for the child process.

Original idea by: Jakub Jelen <jjelen@redhat.com>

crond installs a signal hander for SIGINT & SIGTERM which removes the pid file and exits. This handler is not reset for individual forked sub-processes which results in the condition that if the child receives SIGINT or SIGTERM the pid file is erroneously removed.

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

Anacron does not correctly check configuration parameters

for validity and can segfault on certain data types. The return code from the
match_rx calls in readtab.c are not being checked correctly. Bugfix let
to segfaults in downstreams on non intel architectures. All options in
anacrontab should be now working correctly.

Based on patch from: Martin Poole

Remove syslog.target according to Fedora and systemd guidelines.

https://fedorahosted.org/fpc/ticket/177#comment:2

Thanks to: Ville Skyttä

[PATCH] Fix anacron.8 UTF-8 encoding.

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

Spelling fixes.

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

strncmp() size fixes.

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

Anacron does not correctly check configuration parameters for validity and can segfault on certain data types. The return code from the match_rx calls in readtab.c are not being checked correctly.

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

crond -n must change pam setting.

Non-root user can't read configuration. Init file will fail with proper error message.

rhbz#1006869

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

Fix cronie-common.h file not being found correctly in out of tree build.

Signed-off-by: Saul Wold <sgw@linux.intel.com>

Pull PAM environment variables also from session modules.

- also fix memory leak of environment returned by pam_getenvlist()

KillMode in systemd unit file will enable to finish jobs executed by cron. Even if cron dies, jobs should continue.

Based on: https://bugzilla.redhat.com/show_bug.cgi?id=919290

Fcntl locking has different semantics than flock. Prefer flock.

Release 1.4.11

Log failures in load_user().

Add support for RANDOM_DELAY of jobs.

Fixed bug introduced by previous commit and simplified error handling.

Copy some variables from the crond process environment.

Also cleanup error handling in load_env().

setenv takes 3 arguments

Do not use putenv with string literal.

Make From field in mails from crond RFC compliant.

cron: clarify usage() print out

The -n option needs argument, which was missing from the usage().

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

anacron: fix shadow declarations

Change global variables to have more specific name.  This will make
warnings about shadowing to go away, and may result to a little more
readable code.

log.c:74:35: warning: declaration of 'args' shadows a global declaration [-Wshadow]
global.h:97:15: warning: shadowed declaration is here [-Wshadow]

Reference: http://web.archiveorange.com/archive/v/N6p0RpcfZsIBsaU8B2sE#72jDywyk8NZ3i9g
Signed-off-by: Sami Kerola <kerolasa@iki.fi>

smatch scan: fix various warningss found using smatch

entry.c:396 load_entry() info: redundant null check on e->pwd calling free()
entry.c:398 load_entry() info: redundant null check on e->cmd calling free()

/usr/include/bits/fcntl.h:48:10: warning: preprocessor token O_NOFOLLOW redefined
macros.h:136:9: this was the original definition

security.c:98:44: warning: non-ANSI function declaration of function 'cron_restore_default_security_context'
crontab.c:439:29: warning: non-ANSI function declaration of function 'tmp_path'

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

Mention systemctl command in man page of crond. Based on rhbz#891160.

contrib: remove bashisms

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

docs: corrections to FSF license files, and postal address

The COPYING and file is replaced by files from GNU web site.

http://www.gnu.org/licenses/gpl-2.0.txt

Postal addresses to FSF in other files are updated to match with the
address in license files.

References: http://lists.gnu.org/archive/html/freefont-announce/2005-04/msg00001.html
Signed-off-by: Sami Kerola <kerolasa@iki.fi>

init script: remove bashisms

References: https://sourceforge.net/projects/checkbaskisms/
Signed-off-by: Sami Kerola <kerolasa@iki.fi>

anacron: make function declarations to be ansi compliant

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

anacron: remove numeric exit and file descriptor values

Use symbolic values from standard header instead.

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

anacron: clean up usage() output

Inform in usage which options will need arguments, and add reference to
manual page.

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

anacron: set end job message string to be constant

runjob.c:308:7: warning: assignment discards 'const' qualifier from pointer target type [enabled by default]

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

anacron: add function attributes

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

compile: add function attributes

Some of the argument values are not in use, and few functions does not
return.  Informing which will make few warning messages disapper, and
compiler to generate better binary.

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

compile: fix signed and unsigned warnings

cron.c:100:16: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
cron.c:108:16: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
cron.c:125:16: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]

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

crond: make usage easier to read

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

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.