check_ping: Ignore ping(1)'s exit status

Don't return a WARNING state if the number of lost packets is greater
than zero but below the specified warning threshold.  This happened
because the check_ping plugin used the exit status of the ping(1)
utility.  (#3535140 - Tobias Brox)

2 files changed, 2 insertions, 3 deletions

diff --git a/THANKS.in b/THANKS.in
index b9368a42..470a7ee5 100644
--- a/THANKS.in
+++ b/THANKS.in
@@ -275,3 +275,4 @@ Marc Remy
 Matej Vela
 Jason Ellison
 Charles-Henri Larose
+Tobias Brox
diff --git a/plugins/check_ping.c b/plugins/check_ping.c
index 1367e903..bfdee492 100644
--- a/plugins/check_ping.c
+++ b/plugins/check_ping.c
@@ -500,9 +500,7 @@ run_ping (const char *cmd, const char *addr)
         (void) fclose (child_stderr);
 
 
-        /* close the pipe - WARNING if status is set */
-        if (spclose (child_process))
-                result = max_state (result, STATE_WARNING);
+        spclose (child_process);
 
         if (warn_text == NULL)
                 warn_text = strdup("");