move everything to new NOMMU helpers, except udhcp
[oweals/busybox.git] / miscutils / crond.c
index 3f60468de48e251aff73226c152ec586de1d3732..9d511f960f3a370a4767b2fabe7fdb6e9c5d7a05 100644 (file)
 #define VERSION "2.3.2"
 
 #include "busybox.h"
-#include <stdio.h>
-#include <stdlib.h>
-#include <stdarg.h>
-#include <string.h>
-#include <errno.h>
-#include <time.h>
-#include <dirent.h>
-#include <fcntl.h>
-#include <unistd.h>
-#include <syslog.h>
-#include <signal.h>
-#include <getopt.h>
-#include <sys/ioctl.h>
-#include <sys/wait.h>
-#include <sys/stat.h>
-#include <sys/resource.h>
+#include <sys/syslog.h>
 
 #define arysize(ary)    (sizeof(ary)/sizeof((ary)[0]))
 
@@ -80,10 +65,10 @@ typedef struct CronLine {
 #define DaemonUid 0
 
 #if ENABLE_DEBUG_CROND_OPTION
-static short DebugOpt;
+static unsigned DebugOpt;
 #endif
 
-static short LogLevel = 8;
+static unsigned LogLevel = 8;
 static const char *LogFile;
 static const char *CDir = CRONTABS;
 
@@ -135,7 +120,7 @@ static void crondlog(const char *ctl, ...)
                                close(logfd);
 #if ENABLE_DEBUG_CROND_OPTION
                        } else {
-                               bb_perror_msg("Can't open log file");
+                               bb_perror_msg("can't open log file");
 #endif
                        }
                }
@@ -146,81 +131,50 @@ static void crondlog(const char *ctl, ...)
        }
 }
 
+int crond_main(int ac, char **av);
 int crond_main(int ac, char **av)
 {
-       unsigned long opt;
+       unsigned opt;
        char *lopt, *Lopt, *copt;
+       USE_DEBUG_CROND_OPTION(char *dopt;)
 
-#if ENABLE_DEBUG_CROND_OPTION
-       char *dopt;
-
-       bb_opt_complementally = "f-b:b-f:S-L:L-S:d-l";
-#else
-       bb_opt_complementally = "f-b:b-f:S-L:L-S";
-#endif
-
+       opt_complementary = "f-b:b-f:S-L:L-S" USE_DEBUG_CROND_OPTION(":d-l");
        opterr = 0;                     /* disable getopt 'errors' message. */
-       opt = bb_getopt_ulflags(ac, av, "l:L:fbSc:"
-#if ENABLE_DEBUG_CROND_OPTION
-                                                       "d:"
-#endif
-                                                       , &lopt, &Lopt, &copt
-#if ENABLE_DEBUG_CROND_OPTION
-                                                       , &dopt
-#endif
-               );
-       if (opt & 1) {
-               LogLevel = atoi(lopt);
-       }
-       if (opt & 2) {
-               if (*Lopt != 0) {
+       opt = getopt32(ac, av, "l:L:fbSc:" USE_DEBUG_CROND_OPTION("d:"),
+                       &lopt, &Lopt, &copt USE_DEBUG_CROND_OPTION(, &dopt));
+       if (opt & 1) /* -l */
+               LogLevel = xatou(lopt);
+       if (opt & 2) /* -L */
+               if (*Lopt)
                        LogFile = Lopt;
-               }
-       }
-       if (opt & 32) {
-               if (*copt != 0) {
+       if (opt & 32) /* -c */
+               if (*copt)
                        CDir = copt;
-               }
-       }
 #if ENABLE_DEBUG_CROND_OPTION
-       if (opt & 64) {
-               DebugOpt = atoi(dopt);
+       if (opt & 64) { /* -d */
+               DebugOpt = xatou(dopt);
                LogLevel = 0;
        }
 #endif
 
-       /*
-        * change directory
-        */
-
-       bb_xchdir(CDir);
-       signal(SIGHUP, SIG_IGN);        /* hmm.. but, if kill -HUP original
-                                                                * version - his died. ;(
-                                                                */
-       /*
-        * close stdin and stdout, stderr.
+       /* close stdin and stdout, stderr.
         * close unused descriptors -  don't need.
         * optional detach from controlling terminal
         */
+       if (!(opt & 4))
+               bb_daemonize_or_rexec(DAEMON_CLOSE_EXTRA_FDS, av);
 
-       if (!(opt & 4)) {
-#ifdef BB_NOMMU
-               /* reexec for vfork() do continue parent */
-               vfork_daemon_rexec(1, 0, ac, av, "-f");
-#else
-               bb_xdaemon(1, 0);
-#endif
-       }
+       xchdir(CDir);
+       signal(SIGHUP, SIG_IGN); /* ? original crond dies on HUP... */
 
-       (void) startlogger();   /* need if syslog mode selected */
+       startlogger();  /* need if syslog mode selected */
 
        /*
         * main loop - synchronize to 1 second after the minute, minimum sleep
         *             of 1 second.
         */
-
        crondlog("\011%s " VERSION " dillon, started, log level %d\n",
-                        bb_applet_name, LogLevel);
+                        applet_name, LogLevel);
 
        SynchronizeDir();
 
@@ -279,7 +233,7 @@ int crond_main(int ac, char **av)
                        }
                }
        }
-       bb_fflush_stdout_and_exit(EXIT_SUCCESS); /* not reached */
+       return 0; /* not reached */
 }
 
 static int ChangeUser(const char *user)
@@ -293,7 +247,7 @@ static int ChangeUser(const char *user)
        pas = getpwnam(user);
        if (pas == 0) {
                crondlog("\011failed to get uid for %s", user);
-               return (-1);
+               return -1;
        }
        setenv("USER", pas->pw_name, 1);
        setenv("HOME", pas->pw_dir, 1);
@@ -305,22 +259,22 @@ static int ChangeUser(const char *user)
        err_msg = change_identity_e2str(pas);
        if (err_msg) {
                crondlog("\011%s for user %s", err_msg, user);
-               return (-1);
+               return -1;
        }
        if (chdir(pas->pw_dir) < 0) {
                crondlog("\011chdir failed: %s: %m", pas->pw_dir);
                if (chdir(TMPDIR) < 0) {
                        crondlog("\011chdir failed: %s: %m", TMPDIR);
-                       return (-1);
+                       return -1;
                }
        }
-       return (pas->pw_uid);
+       return pas->pw_uid;
 }
 
 static void startlogger(void)
 {
        if (LogFile == 0) {
-               openlog(bb_applet_name, LOG_CONS | LOG_PID, LOG_CRON);
+               openlog(applet_name, LOG_CONS | LOG_PID, LOG_CRON);
        }
 #if ENABLE_DEBUG_CROND_OPTION
        else {                          /* test logfile */
@@ -329,7 +283,7 @@ static void startlogger(void)
                if ((logfd = open(LogFile, O_WRONLY | O_CREAT | O_APPEND, 0600)) >= 0) {
                        close(logfd);
                } else {
-                       bb_perror_msg("Failed to open log file '%s' reason", LogFile);
+                       bb_perror_msg("failed to open log file '%s': ", LogFile);
                }
        }
 #endif
@@ -392,7 +346,7 @@ static char *ParseField(char *user, char *ary, int modvalue, int off,
        int n2 = -1;
 
        if (base == NULL) {
-               return (NULL);
+               return NULL;
        }
 
        while (*ptr != ' ' && *ptr != '\t' && *ptr != '\n') {
@@ -435,7 +389,7 @@ static char *ParseField(char *user, char *ary, int modvalue, int off,
 
                if (skip == 0) {
                        crondlog("\111failed user %s parsing %s\n", user, base);
-                       return (NULL);
+                       return NULL;
                }
                if (*ptr == '-' && n2 < 0) {
                        ++ptr;
@@ -475,7 +429,7 @@ static char *ParseField(char *user, char *ary, int modvalue, int off,
 
                        if (failsafe == 0) {
                                crondlog("\111failed user %s parsing %s\n", user, base);
-                               return (NULL);
+                               return NULL;
                        }
                }
                if (*ptr != ',') {
@@ -488,7 +442,7 @@ static char *ParseField(char *user, char *ary, int modvalue, int off,
 
        if (*ptr != ' ' && *ptr != '\t' && *ptr != '\n') {
                crondlog("\111failed user %s parsing %s\n", user, base);
-               return (NULL);
+               return NULL;
        }
 
        while (*ptr == ' ' || *ptr == '\t' || *ptr == '\n') {
@@ -505,7 +459,7 @@ static char *ParseField(char *user, char *ary, int modvalue, int off,
        }
 #endif
 
-       return (ptr);
+       return ptr;
 }
 
 static void FixDayDow(CronLine * line)
@@ -557,7 +511,7 @@ static void SynchronizeFile(const char *fileName)
                        struct stat sbuf;
 
                        if (fstat(fileno(fi), &sbuf) == 0 && sbuf.st_uid == DaemonUid) {
-                               CronFile *file = calloc(1, sizeof(CronFile));
+                               CronFile *file = xzalloc(sizeof(CronFile));
                                CronLine **pline;
 
                                file->cf_User = strdup(fileName);
@@ -601,7 +555,7 @@ static void SynchronizeFile(const char *fileName)
 
                                        FixDayDow(&line);
 
-                                       *pline = calloc(1, sizeof(CronLine));
+                                       *pline = xzalloc(sizeof(CronLine));
                                        **pline = line;
 
                                        /* copy command */
@@ -669,7 +623,7 @@ static void SynchronizeDir(void)
 
        remove(CRONUPDATE);
        if (chdir(CDir) < 0) {
-               crondlog("\311unable to find %s\n", CDir);
+               crondlog("\311cannot find %s\n", CDir);
        }
        {
                DIR *dir = opendir(".");
@@ -688,7 +642,7 @@ static void SynchronizeDir(void)
                        }
                        closedir(dir);
                } else {
-                       crondlog("\311Unable to open current dir!\n");
+                       crondlog("\311cannot open current dir!\n");
                }
        }
 }
@@ -792,7 +746,7 @@ static int TestJobs(time_t t1, time_t t2)
                        }
                }
        }
-       return (nJobs);
+       return nJobs;
 }
 
 static void RunJobs(void)
@@ -858,7 +812,7 @@ static int CheckJobs(void)
                }
                nStillRunning += file->cf_Running;
        }
-       return (nStillRunning);
+       return nStillRunning;
 }
 
 
@@ -891,14 +845,14 @@ ForkJob(const char *user, CronLine * line, int mailFd,
                        close(mailFd);
                }
                execl(prog, prog, cmd, arg, NULL);
-               crondlog("\024unable to exec, user %s cmd %s %s %s\n", user, prog, cmd, arg);
+               crondlog("\024cannot exec, user %s cmd %s %s %s\n", user, prog, cmd, arg);
                if (mailf) {
                        fdprintf(1, "Exec failed: %s -c %s\n", prog, arg);
                }
                exit(0);
        } else if (pid < 0) {
                /* FORK FAILED */
-               crondlog("\024couldn't fork, user %s\n", user);
+               crondlog("\024cannot fork, user %s\n", user);
                line->cl_Pid = 0;
                if (mailf) {
                        remove(mailf);
@@ -939,9 +893,9 @@ static void RunJob(const char *user, CronLine * line)
                line->cl_MailFlag = 1;
                fdprintf(mailFd, "To: %s\nSubject: cron: %s\n\n", user,
                        line->cl_Shell);
-               line->cl_MailPos = lseek(mailFd, 0, 1);
+               line->cl_MailPos = lseek(mailFd, 0, SEEK_CUR);
        } else {
-               crondlog("\024unable to create mail file user %s file %s, output to /dev/null\n", user, mailFile);
+               crondlog("\024cannot create mail file user %s file %s, output to /dev/null\n", user, mailFile);
        }
 
        ForkJob(user, line, mailFd, DEFAULT_SHELL, "-c", line->cl_Shell, mailFile);
@@ -1018,12 +972,12 @@ static void RunJob(const char *user, CronLine * line)
 #endif
 
                execl(DEFAULT_SHELL, DEFAULT_SHELL, "-c", line->cl_Shell, NULL);
-               crondlog("\024unable to exec, user %s cmd %s -c %s\n", user,
+               crondlog("\024cannot exec, user %s cmd %s -c %s\n", user,
                                 DEFAULT_SHELL, line->cl_Shell);
                exit(0);
        } else if (pid < 0) {
                /* FORK FAILED */
-               crondlog("\024couldn't fork, user %s\n", user);
+               crondlog("\024cannot, user %s\n", user);
                pid = 0;
        }
        line->cl_Pid = pid;