fixes for using POSIX return codes

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

1 files changed, 24 insertions, 6 deletions

diff --git a/plugins/check_ping.c b/plugins/check_ping.c
index 5eada4f4..420f148c 100644
--- a/plugins/check_ping.c
+++ b/plugins/check_ping.c
@@ -121,8 +121,12 @@ main (int argc, char **argv)
 	else if (pl >= wpl || rta >= wrta)
 		result = STATE_WARNING;
 	else if (pl < wpl && rta < wrta && pl >= 0 && rta >= 0)
-		result = max (result, STATE_OK);
-
+		/* cannot use the max function because STATE_UNKNOWN is now 3 gt STATE_OK			
+		result = max (result, STATE_OK);  */
+		if( !( (result == STATE_WARNING) || (result == STATE_CRITICAL) )  ) {
+			result = STATE_OK;	
+		}
+	
 	if (display_html == TRUE)
 		printf ("<A HREF='%s/traceroute.cgi?%s'>", CGIURL, server_address);
 	if (pl == 100)
@@ -381,7 +385,12 @@ run_ping (char *command_line)
 	while (fgets (input_buffer, MAX_INPUT_BUFFER - 1, child_process)) {
 
 		if (strstr (input_buffer, "(DUP!)")) {
-			result = max (result, STATE_WARNING);
+			/* cannot use the max function since STATE_UNKNOWN is max
+			result = max (result, STATE_WARNING); */
+			if( !(result == STATE_CRITICAL) ){
+				result = STATE_WARNING;
+			}
+			
 			warn_text = realloc (warn_text, strlen (WARN_DUPLICATES) + 1);
 			if (warn_text == NULL)
 				terminate (STATE_UNKNOWN, "unable to realloc warn_text");
@@ -445,6 +454,9 @@ run_ping (char *command_line)
 		else if (strstr (input_buffer, "Destination Host Unreachable"))
 			terminate (STATE_CRITICAL, "PING CRITICAL - Host Unreachable (%s)",
 								 server_address);
+		else if (strstr (input_buffer, "unknown host" ) )
+			terminate (STATE_CRITICAL, "PING CRITICAL - Host not found (%s)",
+								server_address);
 
 		warn_text =
 			realloc (warn_text, strlen (warn_text) + strlen (input_buffer) + 2);
@@ -456,9 +468,15 @@ run_ping (char *command_line)
 			sprintf (warn_text, "%s %s", warn_text, input_buffer);
 
 		if (strstr (input_buffer, "DUPLICATES FOUND"))
-			result = max (result, STATE_WARNING);
+			/* cannot use the max function since STATE_UNKNOWN is max
+			result = max (result, STATE_WARNING); */
+			if( !(result == STATE_CRITICAL) ){
+				result = STATE_WARNING;
+			}
 		else
-			result = max (result, STATE_CRITICAL);
+			/* cannot use the max function since STATE_UNKNOWN is max
+			result = max (result, STATE_CRITICAL); */
+			result = STATE_CRITICAL ;
 	}
 	(void) fclose (child_stderr);
 
@@ -469,7 +487,7 @@ run_ping (char *command_line)
 
 	return result;
 }
-
+
 
 void
 print_usage (void)