more pedantic compiler warns, change to microsecond output for perf data, add size to perf data

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

1 files changed, 100 insertions, 82 deletions

diff --git a/plugins/check_http.c b/plugins/check_http.c
index eba8ac2..e2047ed 100644
--- a/plugins/check_http.c
+++ b/plugins/check_http.c
@@ -40,22 +40,22 @@ enum {
 #include <ssl.h>
 #include <err.h>
 #include <rand.h>
-#endif
-
-#ifdef HAVE_OPENSSL_SSL_H
-#include <openssl/rsa.h>
-#include <openssl/crypto.h>
-#include <openssl/x509.h>
-#include <openssl/pem.h>
-#include <openssl/ssl.h>
-#include <openssl/err.h>
-#include <openssl/rand.h>
+#else
+# ifdef HAVE_OPENSSL_SSL_H
+# include <openssl/rsa.h>
+# include <openssl/crypto.h>
+# include <openssl/x509.h>
+# include <openssl/pem.h>
+# include <openssl/ssl.h>
+# include <openssl/err.h>
+# include <openssl/rand.h>
+# endif
 #endif
 
 #ifdef HAVE_SSL
 int check_cert = FALSE;
 int days_till_exp;
-char *randbuff = "";
+char *randbuff;
 SSL_CTX *ctx;
 SSL *ssl;
 X509 *server_cert;
@@ -84,14 +84,6 @@ struct timeval tv;
 
 #define server_port_check(use_ssl) (use_ssl ? HTTPS_PORT : HTTP_PORT)
 
-/* per RFC 2396 */
- 
-#define HDR_LOCATION "%*[Ll]%*[Oo]%*[Cc]%*[Aa]%*[Tt]%*[Ii]%*[Oo]%*[Nn]: "
-#define URI_HTTP "%[HTPShtps]://"
-#define URI_HOST "%[-.abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789]"
-#define URI_PORT ":%[0123456789]"
-#define URI_PATH "%[-_.!~*'();/?:@&=+$,%#abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789]"
-
 #define HTTP_URL "/"
 #define CRLF "\r\n"
 
@@ -100,9 +92,9 @@ int specify_port = FALSE;
 int server_port = HTTP_PORT;
 char server_port_text[6] = "";
 char server_type[6] = "http";
-char *server_address = ""; 
-char *host_name = "";
-char *server_url = "";
+char *server_address;
+char *host_name;
+char *server_url;
 int server_url_length;
 int server_expect_yn = 0;
 char server_expect[MAX_INPUT_BUFFER] = HTTP_EXPECT;
@@ -118,8 +110,8 @@ int use_ssl = FALSE;
 int verbose = FALSE;
 int sd;
 int min_page_len = 0;
-char *http_method = "GET";
-char *http_post_data = "";
+char *http_method;
+char *http_post_data;
 char buffer[MAX_INPUT_BUFFER];
 
 int process_arguments (int, char **);
@@ -136,7 +128,7 @@ main (int argc, char **argv)
         int result = STATE_UNKNOWN;
 
         /* Set default URL. Must be malloced for subsequent realloc if --onredirect=follow */
-        asprintf (&server_url, "%s", HTTP_URL);
+        server_url = strdup(HTTP_URL);
         server_url_length = strlen(server_url);
 
         if (process_arguments (argc, argv) == ERROR)
@@ -161,8 +153,7 @@ main (int argc, char **argv)
 #ifdef HAVE_SSL
         if (use_ssl && check_cert == TRUE) {
                 if (connect_SSL () != OK)
-                        die (STATE_CRITICAL,
-                                   _("HTTP CRITICAL - Could not make SSL connection\n"));
+                        die (STATE_CRITICAL, _("HTTP CRITICAL - Could not make SSL connection\n"));
                 if ((server_cert = SSL_get_peer_certificate (ssl)) != NULL) {
                         result = check_certificate (&server_cert);
                         X509_free (server_cert);
@@ -313,16 +304,16 @@ process_arguments (int argc, char **argv)
                         break;
                 /* Note: H, I, and u must be malloc'd or will fail on redirects */
                 case 'H': /* Host Name (virtual host) */
-                        asprintf (&host_name, "%s", optarg);
+                        host_name = strdup (optarg);
                         break;
                 case 'I': /* Server IP-address */
-                        asprintf (&server_address, "%s", optarg);
+                        server_address = strdup(optarg);
                         break;
-                case 'u': /* Host or server */
+                case 'u': /* URL path */
                         asprintf (&server_url, "%s", optarg);
                         server_url_length = strlen (server_url);
                         break;
-                case 'p': /* Host or server */
+                case 'p': /* Server port */
                         if (!is_intnonneg (optarg))
                                 usage2 (_("invalid port number"), optarg);
                         else {
@@ -335,8 +326,9 @@ process_arguments (int argc, char **argv)
                         user_auth[MAX_INPUT_BUFFER - 1] = 0;
                         break;
                 case 'P': /* HTTP POST data in URL encoded format */
-                        asprintf (&http_method, "%s", "POST");
-                        asprintf (&http_post_data, "%s", optarg);
+                        if (http_method || http_post_data) break;
+                        http_method = strdup("POST");
+                        http_post_data = strdup(optarg);
                         break;
                 case 's': /* string or substring */
                         strncpy (string_expect, optarg, MAX_INPUT_BUFFER - 1);
@@ -391,22 +383,25 @@ process_arguments (int argc, char **argv)
 
         c = optind;
 
-        if (strcmp (server_address, "") == 0 && c < argc)
-                        asprintf (&server_address, "%s", argv[c++]);
+        if (server_address == NULL && c < argc)
+                server_address = strdup (argv[c++]);
 
-        if (strcmp (host_name, "") == 0 && c < argc)
+        if (host_name == NULL && c < argc)
                 asprintf (&host_name, "%s", argv[c++]);
 
-        if (strcmp (server_address ,"") == 0) {
-                if (strcmp (host_name, "") == 0)
+        if (server_address == NULL) {
+                if (host_name == NULL)
                         usage (_("check_http: you must specify a server address or host name\n"));
                 else
-                        asprintf (&server_address, "%s", host_name);
+                        server_address = strdup (host_name);
         }
 
         if (check_critical_time && critical_time>(double)socket_timeout)
                 socket_timeout = (int)critical_time + 1;
 
+        if (http_method == NULL)
+                http_method = strdup ("GET");
+
         return TRUE;
 }
 
@@ -455,21 +450,34 @@ base64 (char *bin, size_t len)
 
 
 
+/* per RFC 2396 */
+#define HDR_LOCATION "%*[Ll]%*[Oo]%*[Cc]%*[Aa]%*[Tt]%*[Ii]%*[Oo]%*[Nn]: "
+#define URI_HTTP "%[HTPShtps]://"
+#define URI_HOST "%[-.abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789]"
+#define URI_PORT ":%[0123456789]"
+#define URI_PATH "%[-_.!~*'();/?:@&=+$,%#abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789]"
+#define HD1 HDR_LOCATION URI_HTTP URI_HOST URI_PORT URI_PATH
+#define HD2 HDR_LOCATION URI_HTTP URI_HOST URI_PATH
+#define HD3 HDR_LOCATION URI_HTTP URI_HOST URI_PORT
+#define HD4 HDR_LOCATION URI_HTTP URI_HOST
+#define HD5 HDR_LOCATION URI_PATH
+
 int
 check_http (void)
 {
-        char *msg = NULL;
-        char *status_line = "";
-        char *header = NULL;
-        char *page = "";
-        char *auth = NULL;
+        char *msg;
+        char *status_line;
+        char *header;
+        char *page;
+        char *auth;
         int i = 0;
         size_t pagesize = 0;
-        char *full_page = "";
-        char *buf = NULL;
-        char *pos = "";
-        char *x = NULL;
-        char *orig_url = NULL;
+        char *full_page;
+        char *buf;
+        char *pos;
+        char *x;
+        char *orig_url;
+        long microsec;
         double elapsed_time;
         int page_len = 0;
 #ifdef HAVE_SSL
@@ -504,7 +512,7 @@ check_http (void)
         asprintf (&buf, "%s %s HTTP/1.0\r\n", http_method, server_url);
 
         /* optionally send the host header info (not clear if it's usable) */
-        if (strcmp (host_name, ""))
+        if (host_name)
                 asprintf (&buf, "%sHost: %s\r\n", buf, host_name);
 
         /* send user agent */
@@ -512,13 +520,13 @@ check_http (void)
                   buf, clean_revstring (revision), VERSION);
 
         /* optionally send the authentication info */
-        if (strcmp (user_auth, "")) {
+        if (strlen(user_auth)) {
                 auth = base64 (user_auth, strlen (user_auth));
                 asprintf (&buf, "%sAuthorization: Basic %s\r\n", buf, auth);
         }
 
         /* either send http POST data */
-        if (strlen (http_post_data)) {
+        if (http_post_data) {
                 asprintf (&buf, "%sContent-Type: application/x-www-form-urlencoded\r\n", buf);
                 asprintf (&buf, "%sContent-Length: %i\r\n\r\n", buf, strlen (http_post_data));
                 asprintf (&buf, "%s%s%s", buf, http_post_data, CRLF);
@@ -543,6 +551,7 @@ check_http (void)
 #endif
 
         /* fetch the page */
+        full_page = strdup("");
         while ((i = my_recv ()) > 0) {
                 buffer[i] = '\0';
                 asprintf (&full_page, "%s%s", full_page, buffer);
@@ -663,42 +672,49 @@ check_http (void)
                                 while (pos) {
                                         server_address = realloc (server_address, MAX_IPV4_HOSTLENGTH + 1);
                                         if (server_address == NULL)
-                                                die (STATE_UNKNOWN,
-                                                                                 _("HTTP UNKNOWN: could not allocate server_address"));
+                                                die (STATE_UNKNOWN,_("ERROR: could not allocate server_address"));
                                         if (strcspn (pos, "\r\n") > (size_t)server_url_length) {
                                                 server_url = realloc (server_url, strcspn (pos, "\r\n"));
                                                 if (server_url == NULL)
-                                                        die (STATE_UNKNOWN,
-                                                                   _("HTTP UNKNOWN: could not allocate server_url"));
+                                                        die (STATE_UNKNOWN, _("ERROR: could not allocate server_url"));
                                                 server_url_length = strcspn (pos, "\r\n");
                                         }
-                                        if (sscanf (pos, HDR_LOCATION URI_HTTP URI_HOST URI_PORT URI_PATH, server_type, server_address, server_port_text, server_url) == 4) {
-                                                asprintf (&host_name, "%s", server_address);
+                                        /* HDR_LOCATION, URI_HTTP, URI_HOST, URI_PORT, URI_PATH */
+                                        if (sscanf (pos, HD1, server_type, server_address, server_port_text, server_url) == 4) {
+                                                if (host_name != NULL) free(host_name);
+                                                host_name = strdup(server_address);
                                                 use_ssl = server_type_check (server_type);
                                                 server_port = atoi (server_port_text);
                                                 check_http ();
                                         }
-                                        else if (sscanf (pos, HDR_LOCATION URI_HTTP URI_HOST URI_PATH, server_type, server_address, server_url) == 3 ) { 
-                                                asprintf (&host_name, "%s", server_address);
+                                        /* HDR_LOCATION URI_HTTP URI_HOST URI_PATH */
+                                        else if (sscanf (pos, HD2, server_type, server_address, server_url) == 3 ) { 
+                                                if (host_name != NULL) free(host_name);
+                                                host_name = strdup(server_address);
                                                 use_ssl = server_type_check (server_type);
                                                 server_port = server_port_check (use_ssl);
                                                 check_http ();
                                         }
-                                        else if (sscanf (pos, HDR_LOCATION URI_HTTP URI_HOST URI_PORT, server_type, server_address, server_port_text) == 3) {
-                                                asprintf (&host_name, "%s", server_address);
+                                        /* HDR_LOCATION URI_HTTP URI_HOST URI_PORT */
+                                        else if(sscanf (pos, HD3, server_type, server_address, server_port_text) == 3) {
+                                                if (host_name != NULL) free(host_name);
+                                                host_name = strdup(server_address);
                                                 strcpy (server_url, "/");
                                                 use_ssl = server_type_check (server_type);
                                                 server_port = atoi (server_port_text);
                                                 check_http ();
                                         }
-                                        else if (sscanf (pos, HDR_LOCATION URI_HTTP URI_HOST, server_type, server_address) == 2) {
-                                                asprintf (&host_name, "%s", server_address);
+                                        /* HDR_LOCATION URI_HTTP URI_HOST */
+                                        else if(sscanf (pos, HD4, server_type, server_address) == 2) {
+                                                if (host_name != NULL) free(host_name);
+                                                host_name = strdup(server_address);
                                                 strcpy (server_url, "/");
                                                 use_ssl = server_type_check (server_type);
                                                 server_port = server_port_check (use_ssl);
                                                 check_http ();
                                         }
-                                        else if (sscanf (pos, HDR_LOCATION URI_PATH, server_url) == 1) {
+                                        /* HDR_LOCATION URI_PATH */
+                                        else if (sscanf (pos, HD5, server_url) == 1) {
                                                 if ((server_url[0] != '/') && (x = strrchr(orig_url, '/'))) {
                                                         *x = '\0';
                                                         asprintf (&server_url, "%s/%s", orig_url, server_url);
@@ -721,10 +737,11 @@ check_http (void)
                                 printf (_("WARNING"));
                         else if (onredirect == STATE_CRITICAL)
                                 printf (_("CRITICAL"));
-                        elapsed_time = delta_time (tv);
-                        asprintf (&msg, _(" - %s - %.3f second response time %s%s|time=%.3f\n"),
+                        microsec = deltime (tv);
+                        elapsed_time = (double)microsec / 1.0e6;
+                        asprintf (&msg, _(" - %s - %.3f second response time %s%s|time=%ldus size=%dB\n"),
                                  status_line, elapsed_time, timestamp,
-                           (display_html ? "</A>" : ""), elapsed_time);
+                           (display_html ? "</A>" : ""), microsec, pagesize);
                         die (onredirect, "%s", msg);
                 } /* end if (strstr (status_line, "30[0-4]") */
 
@@ -733,10 +750,11 @@ check_http (void)
 
                 
         /* check elapsed time */
-        elapsed_time = delta_time (tv);
-        asprintf (&msg, _("HTTP problem: %s - %.3f second response time %s%s|time=%.3f\n"),
+        microsec = deltime (tv);
+        elapsed_time = (double)microsec / 1.0e6;
+        asprintf (&msg, _("HTTP problem: %s - %.3f second response time %s%s|time=%ldus size=%dB\n"),
                        status_line, elapsed_time, timestamp,
-                       (display_html ? "</A>" : ""), elapsed_time);
+                       (display_html ? "</A>" : ""), microsec, pagesize);
         if (check_critical_time == TRUE && elapsed_time > critical_time)
                 die (STATE_CRITICAL, "%s", msg);
         if (check_warning_time == TRUE && elapsed_time > warning_time)
@@ -747,14 +765,14 @@ check_http (void)
 
         if (strlen (string_expect)) {
                 if (strstr (page, string_expect)) {
-                        printf (_("HTTP OK %s - %.3f second response time %s%s|time=%.3f\n"),
+                        printf (_("HTTP OK %s - %.3f second response time %s%s|time=%ldus size=%dB\n"),
                                 status_line, elapsed_time,
-                                timestamp, (display_html ? "</A>" : ""), elapsed_time);
+                                timestamp, (display_html ? "</A>" : ""), microsec, pagesize);
                         exit (STATE_OK);
                 }
                 else {
-                        printf (_("CRITICAL - string not found%s|time=%.3f\n"),
-                                (display_html ? "</A>" : ""), elapsed_time);
+                        printf (_("CRITICAL - string not found%s|time=%ldus\n size=%dB"),
+                                (display_html ? "</A>" : ""), microsec, pagesize);
                         exit (STATE_CRITICAL);
                 }
         }
@@ -762,15 +780,15 @@ check_http (void)
         if (strlen (regexp)) {
                 errcode = regexec (&preg, page, REGS, pmatch, 0);
                 if (errcode == 0) {
-                        printf (_("HTTP OK %s - %.3f second response time %s%s|time=%.3f\n"),
+                        printf (_("HTTP OK %s - %.3f second response time %s%s|time=%ldus size=%dB\n"),
                                 status_line, elapsed_time,
-                                timestamp, (display_html ? "</A>" : ""), elapsed_time);
+                                timestamp, (display_html ? "</A>" : ""), microsec, pagesize);
                         exit (STATE_OK);
                 }
                 else {
                         if (errcode == REG_NOMATCH) {
-                                printf (_("CRITICAL - pattern not found%s|time=%.3f\n"),
-                                        (display_html ? "</A>" : ""), elapsed_time);
+                                printf (_("CRITICAL - pattern not found%s|time=%ldus size=%dB\n"),
+                                        (display_html ? "</A>" : ""), microsec, pagesize);
                                 exit (STATE_CRITICAL);
                         }
                         else {
@@ -790,9 +808,9 @@ check_http (void)
                 exit (STATE_WARNING);
         }
         /* We only get here if all tests have been passed */
-        asprintf (&msg, _("HTTP OK %s - %.3f second response time %s%s|time=%.3f\n"),
-                        status_line, (float)elapsed_time,
-                        timestamp, (display_html ? "</A>" : ""), elapsed_time);
+        asprintf (&msg, _("HTTP OK %s - %.3f second response time %s%s|time=%ldus size=%dB\n"),
+                        status_line, elapsed_time,
+                        timestamp, (display_html ? "</A>" : ""), microsec, pagesize);
         die (STATE_OK, "%s", msg);
         return STATE_UNKNOWN;
 }