WARNS=6 cleanes
authorJoerg Sonnenberger <joerg@dragonflybsd.org>
Mon, 20 Sep 2004 13:11:54 +0000 (13:11 +0000)
committerJoerg Sonnenberger <joerg@dragonflybsd.org>
Mon, 20 Sep 2004 13:11:54 +0000 (13:11 +0000)
- proper prototypes
- constify certain strings
- remove pre-ISO C junk

usr.bin/at/Makefile
usr.bin/at/at.c
usr.bin/at/panic.c
usr.bin/at/panic.h
usr.bin/at/parsetime.c
usr.bin/at/perm.c
usr.bin/at/perm.h
usr.bin/at/privs.h

index cd14f31..5ddd3ce 100644 (file)
@@ -1,10 +1,10 @@
 # $FreeBSD: src/usr.bin/at/Makefile,v 1.10.2.1 2001/04/25 11:29:16 ru Exp $
-# $DragonFly: src/usr.bin/at/Makefile,v 1.2 2003/06/17 04:29:25 dillon Exp $
+# $DragonFly: src/usr.bin/at/Makefile,v 1.3 2004/09/20 13:11:54 joerg Exp $
 
 .include "${.CURDIR}/Makefile.inc"
 
 PROG=  at
-CFLAGS+=-Wall
+WARNS?=        6
 SRCS=  at.c panic.c parsetime.c perm.c
 LINKS= ${BINDIR}/at ${BINDIR}/atq \
        ${BINDIR}/at ${BINDIR}/atrm \
index 1084c56..a559c2d 100644 (file)
@@ -26,7 +26,7 @@
  * THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  *
  * $FreeBSD: src/usr.bin/at/at.c,v 1.18.2.1 2001/08/02 00:55:58 obrien Exp $
- * $DragonFly: src/usr.bin/at/at.c,v 1.4 2004/01/22 03:22:52 rob Exp $
+ * $DragonFly: src/usr.bin/at/at.c,v 1.5 2004/09/20 13:11:54 joerg Exp $
  */
 
 #define _USE_BSD 1
 #include <time.h>
 #include <unistd.h>
 #include <utmp.h>
-#ifndef __DragonFly__
-#include <getopt.h>
-#else
 #include <locale.h>
-#endif
 
 #if (MAXLOGNAME-1) > UT_NAMESIZE
 #define LOGNAMESIZE UT_NAMESIZE
@@ -96,19 +92,21 @@ enum { ATQ, ATRM, AT, BATCH, CAT }; /* what program we want to run */
 
 /* File scope variables */
 
-char *no_export[] =
+const char *no_export[] =
 {
     "TERM", "TERMCAP", "DISPLAY", "_"
 } ;
 static int send_mail = 0;
 
 /* External variables */
+uid_t real_uid, effective_uid;
+gid_t real_gid, effective_gid;
 
 extern char **environ;
 int fcreated;
-char atfile[] = ATJOB_DIR "12345678901234";
+char atfile[sizeof(ATJOB_DIR) + 14] = ATJOB_DIR;
 
-char *atinput = (char*)0;      /* where to get input from */
+char *atinput = NULL;          /* where to get input from */
 char atqueue = 0;              /* which queue to examine for jobs (atq) */
 char atverify = 0;             /* verify time instead of queuing job */
 
@@ -122,7 +120,8 @@ static void list_jobs(void);
 
 /* Signal catching functions */
 
-static void sigc(int signo)
+static
+void sigc(int signo __unused)
 {
 /* If the user presses ^C, remove the spool file and exit 
  */
@@ -136,7 +135,8 @@ static void sigc(int signo)
     exit(EXIT_FAILURE);
 }
 
-static void alarmc(int signo)
+static
+void alarmc(int sign __unused)
 {
 /* Time out after some seconds
  */
@@ -176,7 +176,7 @@ static char *cwdname(void)
 }
 
 static long
-nextjob()
+nextjob(void)
 {
     long jobno;
     FILE *fid;
@@ -217,9 +217,7 @@ writefile(time_t runtimer, char queue)
     mode_t cmask;
     struct flock lock;
     
-#ifdef __DragonFly__
-    (void) setlocale(LC_TIME, "");
-#endif
+    setlocale(LC_TIME, "");
 
 /* Install the signal handler for SIGINT; terminate after removing the
  * spool file if necessary
@@ -230,8 +228,6 @@ writefile(time_t runtimer, char queue)
 
     sigaction(SIGINT, &act, NULL);
 
-    ppos = atfile + strlen(ATJOB_DIR);
-
     /* Loop over all possible file names for running something at this
      * particular time, see if a file is there; the first empty slot at any
      * particular time is used.  Lock the file LFILE first to make sure
@@ -261,8 +257,9 @@ writefile(time_t runtimer, char queue)
     if ((jobno = nextjob()) == EOF)
        perr("cannot generate job number");
 
-    sprintf(ppos, "%c%5lx%8lx", queue, 
-           jobno, (unsigned long) (runtimer/60));
+    ppos = atfile + strlen(atfile);
+    snprintf(ppos, sizeof(atfile) - strlen(atfile), "%c%5lx%8lx", queue, 
+            jobno, (unsigned long) (runtimer/60));
 
     for(ap=ppos; *ap != '\0'; ap ++)
        if (*ap == ' ')
@@ -353,7 +350,7 @@ writefile(time_t runtimer, char queue)
            eqp = *atenv;
        else
        {
-           int i;
+           unsigned int i;
            for (i=0; i<sizeof(no_export)/sizeof(no_export[0]); i++)
            {
                export = export
@@ -455,9 +452,7 @@ list_jobs(void)
     char timestr[TIMESIZE];
     int first=1;
     
-#ifdef __DragonFly__
-    (void) setlocale(LC_TIME, "");
-#endif
+    setlocale(LC_TIME, "");
 
     PRIV_START
 
@@ -594,9 +589,8 @@ main(int argc, char **argv)
     char queue_set = 0;
     char *pgm;
 
-    enum { ATQ, ATRM, AT, BATCH, CAT };        /* what program we want to run */
     int program = AT;                  /* our default program */
-    char *options = "q:f:mvldbVc";     /* default options for at */
+    const char *options = "q:f:mvldbVc"; /* default options for at */
     int disp_version = 0;
     time_t timer;
 
index cc5a999..cdc8389 100644 (file)
@@ -23,7 +23,7 @@
  * THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  *
  * $FreeBSD: src/usr.bin/at/panic.c,v 1.10 1999/12/05 19:57:14 charnier Exp $
- * $DragonFly: src/usr.bin/at/panic.c,v 1.2 2003/06/17 04:29:25 dillon Exp $
+ * $DragonFly: src/usr.bin/at/panic.c,v 1.3 2004/09/20 13:11:54 joerg Exp $
  */
 
 /* System Headers */
@@ -44,7 +44,7 @@
 /* Global functions */
 
 void
-panic(char *a)
+panic(const char *a)
 {
 /* Something fatal has happened, print error message and exit.
  */
@@ -55,7 +55,7 @@ panic(char *a)
 }
 
 void
-perr(char *a)
+perr(const char *a)
 {
 /* Some operating system error; print error message and exit.
  */
index 9eb20d9..81e98da 100644 (file)
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF
  * THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  *
- * $DragonFly: src/usr.bin/at/panic.h,v 1.3 2004/01/22 03:22:52 rob Exp $
+ * $DragonFly: src/usr.bin/at/panic.h,v 1.4 2004/09/20 13:11:54 joerg Exp $
  */
 
-
-#ifdef __DragonFly__
-#define __NORETURN
-#endif
-
-void
-#ifdef __GNUC__
-__NORETURN
-#endif
-panic(char *a);
-void
-#ifdef __GNUC__
-__NORETURN
-#endif
-perr(char *a);
-void
-#ifdef __GNUC__
-__NORETURN
-#endif
-usage(void);
+void   panic(const char *a) __dead2;
+void   perr(const char *a) __dead2;
+void   usage(void) __dead2;
index 544bd30..c7ec04f 100644 (file)
@@ -33,7 +33,7 @@
  *                                   \PLUS NUMBER MINUTES|HOURS|DAYS|WEEKS/
  *
  * $FreeBSD: src/usr.bin/at/parsetime.c,v 1.19.2.3 2001/12/19 11:19:16 brian Exp $
- * $DragonFly: src/usr.bin/at/parsetime.c,v 1.4 2004/01/22 03:22:52 rob Exp $
+ * $DragonFly: src/usr.bin/at/parsetime.c,v 1.5 2004/09/20 13:11:54 joerg Exp $
  */
 
 /* System Headers */
 #include <string.h>
 #include <time.h>
 #include <unistd.h>
-#ifndef  __DragonFly__
-#include <getopt.h>
-#endif
 
 /* Local headers */
 
 #include "at.h"
 #include "panic.h"
+#include "parsetime.h"
 
 
 /* Structures and unions */
@@ -72,9 +70,9 @@ enum {        /* symbols */
 /* parse translation table - table driven parsers can be your FRIEND!
  */
 struct {
-    char *name;        /* token name */
-    int value; /* token id */
-    int plural;        /* is this plural? */
+    const char *name;  /* token name */
+    int value;         /* token id */
+    int plural;                /* is this plural? */
 } Specials[] = {
     { "midnight", MIDNIGHT,0 },        /* 00:00:00 of today or tomorrow */
     { "noon", NOON,0 },                /* 12:00:00 of today or tomorrow */
@@ -157,7 +155,7 @@ static int sc_tokplur;      /* scanner - is token plural? */
 static int
 parse_token(char *arg)
 {
-    int i;
+    unsigned int i;
 
     for (i=0; i<(sizeof Specials/sizeof Specials[0]); i++)
        if (strcasecmp(Specials[i].name, arg) == 0) {
@@ -191,7 +189,7 @@ init_scanner(int argc, char **argv)
  * token() fetches a token from the input stream
  */
 static int
-token()
+token(void)
 {
     int idx;
 
index f4e45c5..c696e0f 100644 (file)
@@ -23,7 +23,7 @@
  * THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  *
  * $FreeBSD: src/usr.bin/at/perm.c,v 1.9.2.1 2001/08/02 00:55:58 obrien Exp $
- * $DragonFly: src/usr.bin/at/perm.c,v 1.3 2003/10/02 17:42:25 hmp Exp $
+ * $DragonFly: src/usr.bin/at/perm.c,v 1.4 2004/09/20 13:11:54 joerg Exp $
  */
 
 /* System Headers */
@@ -40,6 +40,7 @@
 
 /* Local headers */
 
+#include "perm.h"
 #include "privs.h"
 #include "at.h"
 
 
 /* Function declarations */
 
-static int check_for_user(FILE *fp,const char *name);
+static int check_for_user(FILE *fp, const char *name);
 
 /* Local functions */
 
-static int check_for_user(FILE *fp,const char *name)
+static int
+check_for_user(FILE *fp, const char *name)
 {
     char *buffer;
     size_t len;
@@ -78,8 +80,10 @@ static int check_for_user(FILE *fp,const char *name)
     free(buffer);
     return found;
 }
+
 /* Global functions */
-int check_permission(void)
+int
+check_permission(void)
 {
     FILE *fp;
     uid_t uid = geteuid();
@@ -93,7 +97,7 @@ int check_permission(void)
 
     PRIV_START
 
-    fp=fopen(PERM_PATH "at.allow","r");
+    fp = fopen(PERM_PATH "at.allow","r");
 
     PRIV_END
 
@@ -106,7 +110,7 @@ int check_permission(void)
 
        PRIV_START
 
-       fp=fopen(PERM_PATH "at.deny", "r");
+       fp = fopen(PERM_PATH "at.deny", "r");
 
        PRIV_END
 
index 0a12a90..7df189c 100644 (file)
@@ -21,6 +21,8 @@
  * THEORY OF LIABILITY, WETHER IN CONTRACT, STRICT LIABILITY, OR TORT
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF
  * THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ *
+ * $DragonFly: src/usr.bin/at/perm.h,v 1.2 2004/09/20 13:11:54 joerg Exp $
  */
 
-int check_permission();
+int check_permission(void);
index d3508fd..aca74f1 100644 (file)
@@ -23,7 +23,7 @@
  * THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  *
  * $FreeBSD: src/usr.bin/at/privs.h,v 1.7 1999/12/05 19:57:14 charnier Exp $
- * $DragonFly: src/usr.bin/at/privs.h,v 1.2 2003/06/17 04:29:25 dillon Exp $
+ * $DragonFly: src/usr.bin/at/privs.h,v 1.3 2004/09/20 13:11:54 joerg Exp $
  */
 
 #ifndef _PRIVS_H
  * to the real userid before calling any of them.
  */
 
-#ifndef MAIN
-extern
-#endif
-uid_t real_uid, effective_uid;
-
-#ifndef MAIN 
-extern
-#endif
-gid_t real_gid, effective_gid;
+extern uid_t real_uid, effective_uid;
+extern gid_t real_gid, effective_gid;
 
 #define RELINQUISH_PRIVS { \
                              real_uid = getuid(); \