more pedantic compiler warnings

git-svn-id: https://nagiosplug.svn.sourceforge.net/svnroot/nagiosplug/nagiosplug/trunk@668 f882894a-f735-0410-b71e-b25c423dba1c

diff --git a/plugins/check_disk.c b/plugins/check_disk.c
index bd060218a8aba6504220faeca8252b17a068c357..7974d1415dd2e26ddb792315a3170f6ffaf12439 100644 (file)
--- a/plugins/check_disk.c
+++ b/plugins/check_disk.c
@@ -109,7 +109,7 @@ enum
 #endif
 
 int process_arguments (int, char **);
-void print_path (char *mypath);
+void print_path (const char *mypath);
 int validate_arguments (uintmax_t, uintmax_t, double, double, char *);
 int check_disk (double usp, uintmax_t free_disk);
 int walk_name_list (struct name_list *list, const char *name);
@@ -216,7 +216,7 @@ main (int argc, char **argv)
                temp_list = temp_list->name_next;
        }
 
-       die (result, "DISK %s%s\n", state_text (result), output, details);
+       die (result, "DISK %s%s%s\n", state_text (result), output, details);
        return STATE_UNKNOWN;
 }
 
@@ -453,11 +453,15 @@ process_arguments (int argc, char **argv)
 }
 
 
-void print_path (char *mypath) 
+void
+print_path (const char *mypath) 
 {
-       if (mypath)
-               printf (" for %s", mypath);
-       printf ("\n");
+       if (mypath == NULL)
+               printf ("\n");
+       else
+               printf (" for %s\n", mypath);
+
+       return;
 }
 
 int
@@ -473,14 +477,14 @@ validate_arguments (uintmax_t w, uintmax_t c, double wp, double cp, char *mypath
                printf (_("\
 INPUT ERROR: C_DFP (%f) should be less than W_DFP (%.1f) and both should be between zero and 100 percent, inclusive"),
                        cp, wp);
-               print_path (path);
+               print_path (mypath);
                return ERROR;
        }
        else if ((w > 0 || c > 0) && (w == 0 || c == 0 || c > w)) {
                printf (_("\
 INPUT ERROR: C_DF (%lu) should be less than W_DF (%lu) and both should be greater than zero"),
                        (unsigned long)c, (unsigned long)w);
-               print_path (path);
+               print_path (mypath);
                return ERROR;
        }
        
@@ -542,6 +546,7 @@ print_help (void)
 {
        print_revision (progname, revision);
 
+       printf (_("Copyright (c) 1999 Ethan Galstad <nagios@nagios.org>\n"));
        printf (_(COPYRIGHT), copyright, email);
 
        printf (_("\
@@ -596,7 +601,7 @@ and generates an alert if free space is less than one of the threshold values.")
  check_disk -w 10% -c 5% -p /tmp -p /var -C -w 100000 -c 50000 -p /\n\
    Checks /tmp and /var at 10%,5% and / at 100MB, 50MB\n"));
 
-       support ();
+       printf (_(UT_SUPPORT));
 }
 
 

diff --git a/plugins/check_dummy.c b/plugins/check_dummy.c
index d90b1409874d0c56ce144f6642f0ed75396b1b26..3db5238f8eb3ce1837e7c77728d2559bf111b220 100644 (file)
--- a/plugins/check_dummy.c
+++ b/plugins/check_dummy.c
@@ -1,39 +1,21 @@
-/*************************************************************
- *
- * CHECK_DUMMY.C
- *
- * Program: Dummy plugin for Nagios
- * License: GPL
- * Copyright (c) 1999 Ethan Galstad (nagios@nagios.org)
- *
- * Last Modified: $Date$
- *
- * Command line: CHECK_DUMMY <state>
- *
- * Description:
- *
- * This plugin will simply return the state corresponding to the
- * numerical value of the <state> argument.
- *
- * License Information:
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
- * (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
- *
- **************************************************************/
-
-#include "config.h"
+/******************************************************************************
+
+ This program is free software; you can redistribute it and/or modify
+ it under the terms of the GNU General Public License as published by
+ the Free Software Foundation; either version 2 of the License, or
+ (at your option) any later version.
+
+ This program is distributed in the hope that it will be useful,
+ but WITHOUT ANY WARRANTY; without even the implied warranty of
+ MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ GNU General Public License for more details.
+
+ You should have received a copy of the GNU General Public License
+ along with this program; if not, write to the Free Software
+ Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
+
+******************************************************************************/
+
 #include "common.h"
 #include "utils.h"
 
@@ -42,39 +24,18 @@ const char *revision = "$Revision$";
 const char *copyright = "1999-2003";
 const char *email = "nagiosplug-devel@lists.sourceforge.net";
 
-void
-print_usage (void)
-{
-       printf (_("Usage: %s <integer state>\n"), progname);
-}
+void print_help (void);
+void print_usage (void);
 
-void
-print_help (void)
-{
-       print_revision (progname, revision);
 
-       printf (_(COPYRIGHT), copyright, email);
-
-       print_usage ();
-
-       printf (_(UT_HELP_VRSN));
-
-       printf (_("\n\
-This plugin will simply return the state corresponding to the numeric value\n\
-of the <state> argument.\n"));
-
-       support ();
-}
 
 int
 main (int argc, char **argv)
 {
        int result;
 
-       if (argc != 2) {
-               printf (_("Incorrect number of arguments supplied\n"));
-               exit (STATE_UNKNOWN);
-       }
+       if (argc != 2)
+               usage (_("Incorrect number of arguments supplied\n"));
        else if (strcmp (argv[1], "-V") == 0 || strcmp (argv[1], "--version") == 0) {
                print_revision (progname, revision);
                exit (STATE_OK);
@@ -83,11 +44,10 @@ main (int argc, char **argv)
                print_help ();
                exit (STATE_OK);
        }
-       else if (!is_integer (argv[1])) {
-               print_usage ();
-               exit (STATE_UNKNOWN);
-       }
-       result = atoi (argv[1]);
+       else if (!is_integer (argv[1]))
+               usage (_("Arguments to check_dummy must be an integer\n"));
+       else
+               result = atoi (argv[1]);
 
        switch (result) {
        case STATE_OK:
@@ -99,10 +59,42 @@ main (int argc, char **argv)
        case STATE_CRITICAL:
                printf ("Status is CRITICAL\n");
                break;
-       default:
+       case STATE_UNKNOWN:
                printf ("Status is UNKNOWN\n");
-               result = STATE_UNKNOWN;
+               break;
+       default:
+               printf ("Status %d is not a supported error state\n", result);
+               break;
        }
 
        return result;
 }
+
+
+
+void
+print_help (void)
+{
+       print_revision (progname, revision);
+
+       printf (_("Copyright (c) 1999 Ethan Galstad <nagios@nagios.org>\n"));
+       printf (_(COPYRIGHT), copyright, email);
+
+       print_usage ();
+
+       printf (_(UT_HELP_VRSN));
+
+       printf (_("\n\
+This plugin will simply return the state corresponding to the numeric value\n\
+of the <state> argument.\n"));
+
+       printf (_(UT_SUPPORT));
+}
+
+
+
+void
+print_usage (void)
+{
+       printf (_("Usage: %s <integer state>\n"), progname);
+}

diff --git a/plugins/popen.c b/plugins/popen.c
index fb855d7bbb134641c6195005162524ac4983778f..98ba085b79bb9e989643849b1297cee2cf02f361 100644 (file)
--- a/plugins/popen.c
+++ b/plugins/popen.c
@@ -57,9 +57,11 @@ RETSIGTYPE popen_timeout_alarm_handler (int);
 #define        min(a,b)        ((a) < (b) ? (a) : (b))
 #define        max(a,b)        ((a) > (b) ? (a) : (b))
 int open_max (void);                                           /* {Prog openmax} */
-void err_sys (const char *, ...);
+static void err_sys (const char *, ...) __attribute__((noreturn,format(printf, 1, 2)));
 char *rtrim (char *, const char *);
 
+char *pname = NULL;                                                    /* caller can set this from argv[0] */
+
 /*int *childerr = NULL;*//* ptr to array allocated at run-time */
 /*extern pid_t *childpid = NULL; *//* ptr to array allocated at run-time */
 static int maxfd;                                                              /* from our open_max(), {Prog openmax} */
@@ -67,7 +69,7 @@ static int maxfd;                                                             /* from our open_max(), {Prog openmax} */
 FILE *
 spopen (const char *cmdstring)
 {
-       char *env[] = { "LC_ALL=C", (char*)0 };
+       char *env[2];
        char *cmd = NULL;
        char **argv = NULL;
        char *str;
@@ -84,6 +86,9 @@ spopen (const char *cmdstring)
        setrlimit (RLIMIT_CORE, &limit);
 #endif
 
+       env[0] = strdup("LC_ALL=C");
+       env[1] = '\0';
+
        /* if no command was passed, return with no error */
        if (cmdstring == NULL)
                return (NULL);
@@ -148,13 +153,13 @@ spopen (const char *cmdstring)
 
        if (childpid == NULL) {                         /* first time through */
                maxfd = open_max ();                            /* allocate zeroed out array for child pids */
-               if ((childpid = calloc (maxfd, sizeof (pid_t))) == NULL)
+               if ((childpid = calloc ((size_t)maxfd, sizeof (pid_t))) == NULL)
                        return (NULL);
        }
 
        if (child_stderr_array == NULL) {       /* first time through */
                maxfd = open_max ();                            /* allocate zeroed out array for child pids */
-               if ((child_stderr_array = calloc (maxfd, sizeof (int))) == NULL)
+               if ((child_stderr_array = calloc ((size_t)maxfd, sizeof (int))) == NULL)
                        return (NULL);
        }
 
@@ -259,34 +264,22 @@ open_max (void)
 }
 
 
-static void err_doit (int, const char *, va_list);
-
-char *pname = NULL;                                                    /* caller can set this from argv[0] */
 
 /* Fatal error related to a system call.
  * Print a message and die. */
 
-void
-err_sys (const char *fmt, ...)
-{
-       va_list ap;
-
-       va_start (ap, fmt);
-       err_doit (1, fmt, ap);
-       va_end (ap);
-       exit (1);
-}
-
-/* Print a message and return to caller.
- * Caller specifies "errnoflag". */
-
 #define MAXLINE 2048
 static void
-err_doit (int errnoflag, const char *fmt, va_list ap)
+err_sys (const char *fmt, ...)
 {
+       int errnoflag = 1;
        int errno_save;
        char buf[MAXLINE];
 
+       va_list ap;
+
+       va_start (ap, fmt);
+       /* err_doit (1, fmt, ap); */
        errno_save = errno;                                             /* value caller might want printed */
        vsprintf (buf, fmt, ap);
        if (errnoflag)
@@ -295,7 +288,8 @@ err_doit (int errnoflag, const char *fmt, va_list ap)
        fflush (stdout);                                                        /* in case stdout and stderr are the same */
        fputs (buf, stderr);
        fflush (NULL);                                                          /* flushes all stdio output streams */
-       return;
+       va_end (ap);
+       exit (1);
 }
 
 char *
@@ -313,3 +307,4 @@ rtrim (char *str, const char *tok)
        }
        return str;
 }
+

diff --git a/plugins/utils.c b/plugins/utils.c
index aed03c4e49402e2b0dd3c2dc53a4835e791dc0b8..c2261dc39477d3d140e1a83513257656791082c0 100644 (file)
--- a/plugins/utils.c
+++ b/plugins/utils.c
@@ -54,7 +54,7 @@ max_state (int a, int b)
 
 void usage (char *msg)
 {
-       printf (msg);
+       printf ("%s", msg);
        print_usage ();
        exit (STATE_UNKNOWN);
 }

diff --git a/plugins/utils.h b/plugins/utils.h
index a852e86aeb3ae32752084fcedb1cf2fc918d5b2b..c378e459ab8b8753b6686d7362e3ff438236e247 100644 (file)
--- a/plugins/utils.h
+++ b/plugins/utils.h
@@ -16,7 +16,7 @@ suite of plugins. */
 void support (void);
 char *clean_revstring (const char *revstring);
 void print_revision (const char *, const char *);
-void die (int result, const char *fmt, ...) __attribute__((noreturn));
+void die (int result, const char *fmt, ...) __attribute__((noreturn,format(printf, 2, 3)));
 
 /* Handle timeouts */