scanf parsing fix for check_swap from tracker id 1123292. now use floor(3)

[nagiosplug.git] / plugins / check_nt.c

diff --git a/plugins/check_nt.c b/plugins/check_nt.c
index e52b5a211d6437c5c91757c0427ebf75412eca40..1bdddc45a4b1ce517fe16dcf8e8f0a982dfb79ed 100644 (file)
--- a/plugins/check_nt.c
+++ b/plugins/check_nt.c
@@ -103,7 +103,7 @@ int main(int argc, char **argv){
        unsigned long utilization;
        unsigned long uptime;
        unsigned long age_in_minutes;
-       double counter_value;
+       double counter_value = 0.0;
        int offset=0;
        int updays=0;
        int uphours=0;
@@ -355,14 +355,18 @@ int main(int argc, char **argv){
                        if (allRight)
                        {
                                /* Let's format the output string, finally... */
-
-                               asprintf (&output_message, "%s = %.2f %s", description, counter_value, counter_unit);
-                               output_message = strcat (output_message, "|");
-                               output_message = strcat (output_message,
-                                                       fperfdata (description, counter_value, counter_unit,
-                                                                  1, warning_value, 1, critical_value,
-                                                                  (!(isPercent) && (minval != NULL)), fminval,
-                                                                  (!(isPercent) && (minval != NULL)), fmaxval));
+                                       if (strstr(description, "%") == NULL) {
+                                               asprintf (&output_message, "%s = %.2f %s",                                                                                      description, counter_value, counter_unit);
+                                       } else {
+                                               /* has formatting, will segv if wrong */
+                                       asprintf (&output_message, description, counter_value);
+                                       }
+                                       asprintf (&output_message, "%s |", output_message);
+                               asprintf (&output_message,"%s %s", output_message, 
+                                               fperfdata (description, counter_value, 
+                                                       counter_unit, 1, warning_value, 1, critical_value,
+                                                       (!(isPercent) && (minval != NULL)), fminval,
+                                                       (!(isPercent) && (minval != NULL)), fmaxval));
                        }
                }
 
@@ -418,7 +422,7 @@ int main(int argc, char **argv){
 
        case CHECK_NONE:
        default:
-               usage (_("Please specify a variable to check"));
+               usage4 (_("Please specify a variable to check"));
                break;
 
        }
@@ -482,14 +486,12 @@ int process_arguments(int argc, char **argv){
                switch (c)
                        {
                        case '?': /* print short usage statement if args not parsable */
-                               printf("%s: Unknown argument: %s\n\n",progname,optarg);
-                               print_usage();
-                               exit(STATE_UNKNOWN);
+                       usage2 (_("Unknown argument"), optarg);
                        case 'h': /* help */
                                print_help();
                                exit(STATE_OK);
                        case 'V': /* version */
-                               print_revision(progname,"$Revision$");
+                               print_revision(progname,revision);
                                exit(STATE_OK);
                        case 'H': /* hostname */
                                if (server_address)     free(server_address);
@@ -502,7 +504,7 @@ int process_arguments(int argc, char **argv){
                                if (is_intnonneg(optarg))
                                        server_port=atoi(optarg);
                                else
-                                       die(STATE_UNKNOWN,_("Server port an integer (seconds)\nType '%s -h' for additional help\n"),progname);
+                                       die(STATE_UNKNOWN,_("Server port must be an integer\n"));
                                break;
                        case 'v':
                                if(strlen(optarg)<4)
@@ -568,7 +570,7 @@ void fetch_data (const char *address, int port, const char *sendb) {
        result=process_tcp_request(address, port, sendb, recv_buffer,sizeof(recv_buffer));
 
        if(result!=STATE_OK)
-               die (result, "could not fetch information from server\n");
+               die (result, _("could not fetch information from server\n"));
                
        if (!strncmp(recv_buffer,"ERROR",5))
                die (STATE_UNKNOWN, "NSClient - %s\n",recv_buffer);
@@ -609,7 +611,7 @@ void print_help(void)
 {
        print_revision(progname,revision);
        
-       printf (_("Copyright (c) 2000 Yves Rubin (rubiyz@yahoo.com)\n"));
+       printf ("Copyright (c) 2000 Yves Rubin (rubiyz@yahoo.com)\n");
        printf (COPYRIGHT, copyright, email);
        
        printf (_("This plugin collects data from the NSClient service running on a\n\
@@ -672,7 +674,8 @@ Windows NT/2000/XP/2003 server.\n\n"));
      Request a -l parameters with the following syntax:\n\
                 -l \"\\\\<performance object>\\\\counter\",\"<description>\n\
      The <description> parameter is optional and \n\
-     is given to a printf output command which require a float parameters.\n\
+     is given to a printf output command which requires a float parameter.\n\
+     If <description> does not include \"%%\", it is used as a label.\n\
      Some examples:\n\
        \"Paging file usage is %%.2f %%%%\"\n\
        \"%%.f %%%% paging file used.\"\n"));