more pedantic compiler warns

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

1 files changed, 18 insertions, 14 deletions

diff --git a/plugins/check_ping.c b/plugins/check_ping.c
index 26c0192a..fd73b8ce 100644
--- a/plugins/check_ping.c
+++ b/plugins/check_ping.c
@@ -55,7 +55,7 @@ int verbose = FALSE;
 float rta = UNKNOWN_TRIP_TIME;
 int pl = UNKNOWN_PACKET_LOSS;
 
-char *warn_text = "";
+char *warn_text;
 
 
 
@@ -69,11 +69,10 @@ main (int argc, char **argv)
 	int this_result = STATE_UNKNOWN;
 	int i;
 
-	addresses = malloc (max_addr);
+	addresses = malloc ((size_t)max_addr);
 
 	if (process_arguments (argc, argv) == ERROR)
 		usage (_("Could not parse arguments"));
-	exit;
 
 	/* Set signal handling and alarm */
 	if (signal (SIGALRM, popen_timeout_alarm_handler) == SIG_ERR) {
@@ -113,7 +112,7 @@ main (int argc, char **argv)
 		/* run the command */
 		this_result = run_ping (cmd, addresses[i]);
 
-		if (pl == UNKNOWN_PACKET_LOSS || rta == UNKNOWN_TRIP_TIME) {
+		if (pl == UNKNOWN_PACKET_LOSS || rta < 0.0) {
 			printf ("%s\n", cmd);
 			die (STATE_UNKNOWN,
 			           _("Error: Could not interpret output from ping command\n"));
@@ -193,12 +192,15 @@ process_arguments (int argc, char **argv)
 		switch (c) {
 		case '?':	/* usage */
 			usage3 (_("Unknown argument"), optopt);
+			break;
 		case 'h':	/* help */
 			print_help ();
 			exit (STATE_OK);
+			break;
 		case 'V':	/* version */
 			print_revision (progname, revision);
 			exit (STATE_OK);
+			break;
 		case 't':	/* timeout period */
 			timeout_interval = atoi (optarg);
 			break;
@@ -221,7 +223,7 @@ process_arguments (int argc, char **argv)
 				n_addresses++;
 				if (n_addresses > max_addr) {
 					max_addr *= 2;
-					addresses = realloc (addresses, max_addr);
+					addresses = realloc (addresses, (size_t)max_addr);
 					if (addresses == NULL)
 						die (STATE_UNKNOWN, _("Could not realloc() addresses\n"));
 				}
@@ -292,7 +294,7 @@ process_arguments (int argc, char **argv)
 		}
 	}
 
-	if (wrta == UNKNOWN_TRIP_TIME) {
+	if (wrta < 0.0) {
 		if (is_negative (argv[c])) {
 			printf (_("<wrta> (%s) must be a non-negative number\n"), argv[c]);
 			return ERROR;
@@ -303,7 +305,7 @@ process_arguments (int argc, char **argv)
 		}
 	}
 
-	if (crta == UNKNOWN_TRIP_TIME) {
+	if (crta < 0.0) {
 		if (is_negative (argv[c])) {
 			printf (_("<crta> (%s) must be a non-negative number\n"), argv[c]);
 			return ERROR;
@@ -346,11 +348,11 @@ validate_arguments ()
 	float max_seconds;
 	int i;
 
-	if (wrta == UNKNOWN_TRIP_TIME) {
+	if (wrta < 0.0) {
 		printf (_("<wrta> was not set\n"));
 		return ERROR;
 	}
-	else if (crta == UNKNOWN_TRIP_TIME) {
+	else if (crta < 0.0) {
 		printf (_("<crta> was not set\n"));
 		return ERROR;
 	}
@@ -397,10 +399,9 @@ run_ping (char *cmd, char *server_address)
 	char buf[MAX_INPUT_BUFFER];
 	int result = STATE_UNKNOWN;
 
-	if ((child_process = spopen (cmd)) == NULL) {
-		printf (_("Cannot open pipe: "));
-		die (STATE_UNKNOWN, cmd);
-	}
+	if ((child_process = spopen (cmd)) == NULL)
+		die (STATE_UNKNOWN, _("Cannot open pipe: %s"), cmd);
+
 	child_stderr = fdopen (child_stderr_array[fileno (child_process)], "r");
 	if (child_stderr == NULL)
 		printf (_("Cannot open stderr for %s\n"), cmd);
@@ -455,7 +456,7 @@ run_ping (char *cmd, char *server_address)
 			           _("PING CRITICAL - Host not found (%s)"),
 			           server_address);
 
-		if (strlen (warn_text) == 0)
+		if (warn_text == NULL)
 			warn_text = strdup (buf);
 		else if (asprintf (&warn_text, "%s %s", warn_text, buf) == -1)
 			die (STATE_UNKNOWN, _("unable to realloc warn_text"));
@@ -472,6 +473,9 @@ run_ping (char *cmd, char *server_address)
 	if (spclose (child_process))
 		result = max_state (result, STATE_WARNING);
 
+	if (warn_text == NULL)
+		warn_text = strdup("");
+
 	return result;
 }