Code

Fixed crash from check_nt when -H unspecified or given multiple times (sf.net #273175...
[nagiosplug.git] / plugins / check_by_ssh.c
index 1675071c187e08cc5776ca160d8a8f72dbaeb119..d0dbf34a140e289342465d80dcf6f955568790a7 100644 (file)
@@ -1,39 +1,33 @@
-/******************************************************************************
-*
+/*****************************************************************************
+* 
 * Nagios check_by_ssh plugin
-*
+* 
 * License: GPL
-* Copyright (c) 1999-2006 nagios-plugins team
-*
-* Last Modified: $Date$
-*
+* Copyright (c) 2000-2008 Nagios Plugins Development Team
+* 
 * Description:
-*
+* 
 * This file contains the check_by_ssh plugin
-*
-* License Information:
-*
-* This program is free software; you can redistribute it and/or modify
+* 
+* 
+* This program is free software: you can redistribute it and/or modify
 * it under the terms of the GNU General Public License as published by
-* the Free Software Foundation; either version 2 of the License, or
+* the Free Software Foundation, either version 3 of the License, or
 * (at your option) any later version.
-*
+* 
 * This program is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 * GNU General Public License for more details.
-*
+* 
 * You should have received a copy of the GNU General Public License
-* along with this program; if not, write to the Free Software
-* Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
-*
-* $Id$
+* along with this program.  If not, see <http://www.gnu.org/licenses/>.
+* 
 * 
-******************************************************************************/
+*****************************************************************************/
+
 const char *progname = "check_by_ssh";
-const char *revision = "$Revision$";
-const char *copyright = "2000-2006";
+const char *copyright = "2000-2008";
 const char *email = "nagiosplug-devel@lists.sourceforge.net";
 
 #include "common.h"
@@ -46,9 +40,10 @@ int validate_arguments (void);
 void print_help (void);
 void print_usage (void);
 
-int commands = 0;
-int services = 0;
-int skip = 0;
+unsigned int commands = 0;
+unsigned int services = 0;
+int skip_stdout = 0;
+int skip_stderr = 0;
 char *remotecmd = NULL;
 char *comm = NULL;
 char *hostname = NULL;
@@ -77,6 +72,9 @@ main (int argc, char **argv)
        bindtextdomain (PACKAGE, LOCALEDIR);
        textdomain (PACKAGE);
 
+       /* Parse extra opts if any */
+       argv=np_extra_opts (&argc, argv, progname);
+
        /* process arguments */
        if (process_arguments (argc, argv) == ERROR)
                usage_va(_("Could not parse arguments"));
@@ -92,17 +90,28 @@ main (int argc, char **argv)
                printf ("%s\n", comm);
 
        result = np_runcmd(comm, &chld_out, &chld_err, 0);
-       /* UNKNOWN if output found on stderr */
-       if(chld_err.buflen) {
-               printf(_("Remote command execution failed: %s\n"),
-                          chld_err.buflen ? chld_err.buf : _("Unknown error"));
-               return STATE_UNKNOWN;
+
+       if (skip_stdout == -1) /* --skip-stdout specified without argument */
+               skip_stdout = chld_out.lines;
+       if (skip_stderr == -1) /* --skip-stderr specified without argument */
+               skip_stderr = chld_err.lines;
+
+       /* UNKNOWN or worse if (non-skipped) output found on stderr */
+       if(chld_err.lines > skip_stderr) {
+               printf (_("Remote command execution failed: %s\n"),
+                       chld_err.line[skip_stderr]);
+               return max_state_alt(result, STATE_UNKNOWN);
        }
 
        /* this is simple if we're not supposed to be passive.
         * Wrap up quickly and keep the tricks below */
        if(!passive) {
-               printf ("%s\n", skip < chld_out.lines ? chld_out.line[skip] : chld_out.buf);
+               if (chld_out.lines > skip_stdout)
+                       for (i = skip_stdout; i < chld_out.lines; i++)
+                               puts (chld_out.line[i]);
+               else
+                       printf (_("%s - check_by_ssh: Remote command '%s' returned status %d\n"),
+                               state_text(result), remotecmd, result);
                return result;  /* return error status from remote command */
        }
 
@@ -119,22 +128,21 @@ main (int argc, char **argv)
 
        local_time = time (NULL);
        commands = 0;
-       for(i = skip; chld_out.line[i]; i++) {
-               status_text = strstr (chld_out.line[i], "STATUS CODE: ");
-               if (status_text == NULL) {
-                       printf ("%s", chld_out.line[i]);
-                       return result;
-               }
+       for(i = skip_stdout; i < chld_out.lines; i++) {
+               status_text = chld_out.line[i++];
+               if (i == chld_out.lines || strstr (chld_out.line[i], "STATUS CODE: ") == NULL)
+                       die (STATE_UNKNOWN, _("%s: Error parsing output\n"), progname);
+
                if (service[commands] && status_text
-                       && sscanf (status_text, "STATUS CODE: %d", &cresult) == 1)
+                       && sscanf (chld_out.line[i], "STATUS CODE: %d", &cresult) == 1)
                {
                        fprintf (fp, "[%d] PROCESS_SERVICE_CHECK_RESULT;%s;%s;%d;%s\n",
                                 (int) local_time, host_shortname, service[commands++],
-                                cresult, chld_out.line[i]);
+                                cresult, status_text);
                }
        }
        
-       /* force an OK state */
+       /* Multiple commands and passive checking should always return OK */
        return result;
 }
 
@@ -161,7 +169,9 @@ process_arguments (int argc, char **argv)
                {"user", required_argument, 0, 'u'},
                {"logname", required_argument, 0, 'l'},
                {"command", required_argument, 0, 'C'},
-               {"skip", required_argument, 0, 'S'},
+               {"skip", optional_argument, 0, 'S'}, /* backwards compatibility */
+               {"skip-stdout", optional_argument, 0, 'S'},
+               {"skip-stderr", optional_argument, 0, 'E'},
                {"proto1", no_argument, 0, '1'},
                {"proto2", no_argument, 0, '2'},
                {"use-ipv4", no_argument, 0, '4'},
@@ -179,7 +189,7 @@ process_arguments (int argc, char **argv)
                        strcpy (argv[c], "-t");
 
        while (1) {
-               c = getopt_long (argc, argv, "Vvh1246fqt:H:O:p:i:u:l:C:S:n:s:o:", longopts,
+               c = getopt_long (argc, argv, "Vvh1246fqt:H:O:p:i:u:l:C:S::E::n:s:o:", longopts,
                                 &option);
 
                if (c == -1 || c == EOF)
@@ -187,7 +197,7 @@ process_arguments (int argc, char **argv)
 
                switch (c) {
                case 'V':                                                                       /* version */
-                       print_revision (progname, revision);
+                       print_revision (progname, NP_VERSION);
                        exit (STATE_OK);
                case 'h':                                                                       /* help */
                        print_help ();
@@ -249,11 +259,21 @@ process_arguments (int argc, char **argv)
                                asprintf (&remotecmd, "%s;echo STATUS CODE: $?;", remotecmd);
                        asprintf (&remotecmd, "%s%s", remotecmd, optarg);
                        break;
-               case 'S':                                                                       /* Skip n lines in the output to ignore system banner */
-                       if (!is_integer (optarg))
-                               usage_va(_("skip lines must be an integer"));
+               case 'S':                                                                       /* skip n (or all) lines on stdout */
+                       if (optarg == NULL)
+                               skip_stdout = -1; /* skip all output on stdout */
+                       else if (!is_integer (optarg))
+                               usage_va(_("skip-stdout argument must be an integer"));
                        else
-                               skip = atoi (optarg);
+                               skip_stdout = atoi (optarg);
+                       break;
+               case 'E':                                                                       /* skip n (or all) lines on stderr */
+                       if (optarg == NULL)
+                               skip_stderr = -1; /* skip all output on stderr */
+                       else if (!is_integer (optarg))
+                               usage_va(_("skip-stderr argument must be an integer"));
+                       else
+                               skip_stderr = atoi (optarg);
                        break;
                case 'o':                                                                       /* Extra options for the ssh command */
                        asprintf (&comm, "%s -%c '%s'", comm, c, optarg);
@@ -283,7 +303,7 @@ process_arguments (int argc, char **argv)
                                asprintf (&remotecmd, "%s", argv[c]);
        }
 
-       if (commands > 1)
+       if (commands > 1 || passive)
                asprintf (&remotecmd, "%s;echo STATUS CODE: $?;", remotecmd);
 
        if (remotecmd == NULL || strlen (remotecmd) <= 1)
@@ -315,7 +335,7 @@ validate_arguments (void)
 void
 print_help (void)
 {
-       print_revision (progname, revision);
+       print_revision (progname, NP_VERSION);
 
        printf ("Copyright (c) 1999 Karl DeBisschop <kdebisschop@users.sourceforge.net>\n");
        printf (COPYRIGHT, copyright, email);
@@ -323,23 +343,27 @@ print_help (void)
        printf (_("This plugin uses SSH to execute commands on a remote host"));
 
   printf ("\n\n");
-  
+
        print_usage ();
 
        printf (_(UT_HELP_VRSN));
 
+       printf (_(UT_EXTRA_OPTS));
+
        printf (_(UT_HOST_PORT), 'p', "none");
 
        printf (_(UT_IPv46));
 
   printf (" %s\n", "-1, --proto1");
-  printf ("    %s\n", _("tell ssh to use Protocol 1"));
+  printf ("    %s\n", _("tell ssh to use Protocol 1 [optional]"));
   printf (" %s\n", "-2, --proto2");
-  printf ("    %s\n", _("tell ssh to use Protocol 2"));
-  printf (" %s\n", "-S, --skiplines=n");
-  printf ("    %s\n", _("Ignore first n lines on STDERR (to suppress a logon banner)"));
+  printf ("    %s\n", _("tell ssh to use Protocol 2 [optional]"));
+  printf (" %s\n", "-S, --skip-stdout[=n]");
+  printf ("    %s\n", _("Ignore all or (if specified) first n lines on STDOUT [optional]"));
+  printf (" %s\n", "-E, --skip-stderr[=n]");
+  printf ("    %s\n", _("Ignore all or (if specified) first n lines on STDERR [optional]"));
   printf (" %s\n", "-f");
-  printf ("    %s\n", _("tells ssh to fork rather than create a tty"));
+  printf ("    %s\n", _("tells ssh to fork rather than create a tty [optional]. This will always return OK if ssh is executed"));
   printf (" %s\n","-C, --command='COMMAND STRING'");
   printf ("    %s\n", _("command to execute on the remote machine"));
   printf (" %s\n","-l, --logname=USERNAME");
@@ -358,6 +382,8 @@ print_help (void)
   printf ("    %s\n", _("Tell ssh to suppress warning and diagnostic messages [optional]"));
        printf (_(UT_WARN_CRIT));
        printf (_(UT_TIMEOUT), DEFAULT_SOCKET_TIMEOUT);
+       printf (_(UT_VERBOSE));
+       printf("\n");
   printf (" %s\n", _("The most common mode of use is to refer to a local identity file with"));
   printf (" %s\n", _("the '-i' option. In this mode, the identity pair should have a null"));
   printf (" %s\n", _("passphrase and the public key should be listed in the authorized_keys"));
@@ -365,6 +391,7 @@ print_help (void)
   printf (" %s\n", _("only one command on the remote server. If the remote SSH server tracks"));
   printf (" %s\n", _("invocation arguments, the one remote program may be an agent that can"));
   printf (" %s\n", _("execute additional commands as proxy"));
+  printf("\n");
   printf (" %s\n", _("To use passive mode, provide multiple '-C' options, and provide"));
   printf (" %s\n", _("all of -O, -s, and -n options (servicelist order must match '-C'options)"));
   printf ("\n");
@@ -374,7 +401,14 @@ print_help (void)
   printf (" %s\n", "[1080933700] PROCESS_SERVICE_CHECK_RESULT;flint;c1;0; up 2 days");
   printf (" %s\n", "[1080933700] PROCESS_SERVICE_CHECK_RESULT;flint;c2;0; up 2 days");
   printf (" %s\n", "[1080933700] PROCESS_SERVICE_CHECK_RESULT;flint;c3;0; up 2 days");
-       printf (_(UT_SUPPORT));
+
+#ifdef NP_EXTRA_OPTS
+       printf("\n");
+       printf("%s\n", _("Notes:"));
+       printf(_(UT_EXTRA_OPTS_NOTES));
+#endif
+
+       printf(_(UT_SUPPORT));
 }
 
 
@@ -383,6 +417,9 @@ void
 print_usage (void)
 {
        printf (_("Usage:"));
-  printf(" %s [-fq46] [-t timeout] [-i identity] [-l user] -H <host> -C <command>",progname);
-  printf(" [-n name] [-s servicelist] [-O outputfile] [-p port] [-o ssh-option]\n");
+       printf (" %s -H <host> -C <command> [-fqv] [-1|-2] [-4|-6]\n"
+               "       [-S [lines]] [-E [lines]] [-t timeout] [-i identity]\n"
+               "       [-l user] [-n name] [-s servicelist] [-O outputfile]\n"
+               "       [-p port] [-o ssh-option]\n",
+               progname);
 }