Module: monitoring-plugins
    Branch: master
    Commit: a7d7992777963bd192cfef21988d281bc371c794
    Author: Sven Nierlein <[email protected]>
 Committer: Jan Wagner <[email protected]>
      Date: Mon Jun 30 14:52:10 2014 +0200
       URL: 
https://www.monitoring-plugins.org/repositories/monitoring-plugins/commit/?id=a7d7992

check_ssh: change warning to critical for protocal/version errors

It makes more sense to exit critical if a explicit version/protocol is 
requested. This
would also be more consistent with other plugins. Other string matching plugins 
like
check_snmp or check_http exit critical if the result does not match.

Signed-off-by: Sven Nierlein <[email protected]>

Closes #1268

---

 plugins/check_ssh.c | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/plugins/check_ssh.c b/plugins/check_ssh.c
index b6187d6..3658965 100644
--- a/plugins/check_ssh.c
+++ b/plugins/check_ssh.c
@@ -253,18 +253,18 @@ ssh_connect (char *haddr, int hport, char 
*remote_version, char *remote_protocol
 
                if (remote_version && strcmp(remote_version, ssh_server)) {
                        printf
-                               (_("SSH WARNING - %s (protocol %s) version 
mismatch, expected '%s'\n"),
+                               (_("SSH CRITICAL - %s (protocol %s) version 
mismatch, expected '%s'\n"),
                                 ssh_server, ssh_proto, remote_version);
                        close(sd);
-                       exit (STATE_WARNING);
+                       exit (STATE_CRITICAL);
                }
 
                if (remote_protocol && strcmp(remote_protocol, ssh_proto)) {
                        printf
-                               (_("SSH WARNING - %s (protocol %s) protocol 
version mismatch, expected '%s'\n"),
+                               (_("SSH CRITICAL - %s (protocol %s) protocol 
version mismatch, expected '%s'\n"),
                                 ssh_server, ssh_proto, remote_protocol);
                        close(sd);
-                       exit (STATE_WARNING);
+                       exit (STATE_CRITICAL);
                }
 
                elapsed_time = (double)deltime(tv) / 1.0e6;
@@ -307,10 +307,10 @@ print_help (void)
        printf (UT_CONN_TIMEOUT, DEFAULT_SOCKET_TIMEOUT);
 
        printf (" %s\n", "-r, --remote-version=STRING");
-  printf ("    %s\n", _("Warn if string doesn't match expected server version 
(ex: OpenSSH_3.9p1)"));
+  printf ("    %s\n", _("Alert if string doesn't match expected server version 
(ex: OpenSSH_3.9p1)"));
 
        printf (" %s\n", "-P, --remote-protocol=STRING");
-  printf ("    %s\n", _("Warn if protocol doesn't match expected protocol 
version (ex: 2.0)"));
+  printf ("    %s\n", _("Alert if protocol doesn't match expected protocol 
version (ex: 2.0)"));
 
        printf (UT_VERBOSE);
 

Reply via email to