Code

Under some circumstances, the 'url' path of a redirection target missed
[nagiosplug.git] / plugins / check_snmp.c
index 9ff3439e74783bed1d366c780e86514e70684997..544c5c0b281c05f61bd8a7c2729e8aca179633d2 100644 (file)
@@ -1,26 +1,39 @@
-/******************************************************************************
-
- 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
- (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$
-******************************************************************************/
+/*****************************************************************************
+* 
+* Nagios check_snmp plugin
+* 
+* License: GPL
+* Copyright (c) 1999-2007 Nagios Plugins Development Team
+* 
+* Last Modified: $Date$
+* 
+* Description:
+* 
+* This file contains the check_snmp plugin
+* 
+* Check status of remote machines and obtain system information via SNMP
+* 
+* 
+* 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 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, see <http://www.gnu.org/licenses/>.
+* 
+* $Id$
+* 
+*****************************************************************************/
 
 const char *progname = "check_snmp";
 const char *revision = "$Revision$";
-const char *copyright = "1999-2004";
+const char *copyright = "1999-2007";
 const char *email = "nagiosplug-devel@lists.sourceforge.net";
 
 #include "common.h"
@@ -68,8 +81,8 @@ int process_arguments (int, char **);
 int validate_arguments (void);
 char *clarify_message (char *);
 int check_num (int);
-int lu_getll (unsigned long *, char *);
-int lu_getul (unsigned long *, char *);
+int llu_getll (unsigned long long *, char *);
+int llu_getul (unsigned long long *, char *);
 char *thisarg (char *str);
 char *nextarg (char *str);
 void print_usage (void);
@@ -80,8 +93,8 @@ char regex_expect[MAX_INPUT_BUFFER] = "";
 regex_t preg;
 regmatch_t pmatch[10];
 char timestamp[10] = "";
-char errbuf[MAX_INPUT_BUFFER];
-char perfstr[MAX_INPUT_BUFFER];
+char errbuf[MAX_INPUT_BUFFER] = "";
+char perfstr[MAX_INPUT_BUFFER] = "";
 int cflags = REG_EXTENDED | REG_NOSUB | REG_NEWLINE;
 int eflags = 0;
 int errcode, excode;
@@ -108,15 +121,15 @@ size_t nunits = 0;
 size_t unitv_size = 8;
 int verbose = FALSE;
 int usesnmpgetnext = FALSE;
-unsigned long lower_warn_lim[MAX_OIDS];
-unsigned long upper_warn_lim[MAX_OIDS];
-unsigned long lower_crit_lim[MAX_OIDS];
-unsigned long upper_crit_lim[MAX_OIDS];
-unsigned long response_value[MAX_OIDS];
+unsigned long long lower_warn_lim[MAX_OIDS];
+unsigned long long upper_warn_lim[MAX_OIDS];
+unsigned long long lower_crit_lim[MAX_OIDS];
+unsigned long long upper_crit_lim[MAX_OIDS];
+unsigned long long response_value[MAX_OIDS];
 int check_warning_value = FALSE;
 int check_critical_value = FALSE;
 int retries = 0;
-unsigned long eval_method[MAX_OIDS];
+unsigned long long eval_method[MAX_OIDS];
 char *delimiter;
 char *output_delim;
 char *miblist = NULL;
@@ -132,14 +145,14 @@ main (int argc, char **argv)
        int result = STATE_DEPENDENT;
        char input_buffer[MAX_INPUT_BUFFER];
        char *command_line = NULL;
+       char *cl_hidden_auth = NULL;
        char *response = NULL;
        char *outbuff;
        char *output;
        char *ptr = NULL;
        char *p2 = NULL;
        char *show = NULL;
-       char type[8];
-       char *str[MAX_INPUT_BUFFER];
+       char type[8] = "";
 
        setlocale (LC_ALL, "");
        bindtextdomain (PACKAGE, LOCALEDIR);
@@ -163,19 +176,28 @@ main (int argc, char **argv)
        timeout_interval = DEFAULT_TIMEOUT;
        retries = DEFAULT_RETRIES;
 
+       /* Parse extra opts if any */
+       argv=np_extra_opts (&argc, argv, progname);
+
        if (process_arguments (argc, argv) == ERROR)
                usage4 (_("Could not parse arguments"));
 
        /* create the command line to execute */
                if(usesnmpgetnext == TRUE) {
                asprintf(&command_line, "%s -t %d -r %d -m %s -v %s %s %s:%s %s",
-                                   PATH_TO_SNMPGETNEXT, timeout_interval, retries, miblist, proto,
-                                               authpriv, server_address, port, oid);
+                       PATH_TO_SNMPGETNEXT, timeout_interval, retries, miblist, proto,
+                       authpriv, server_address, port, oid);
+               asprintf(&cl_hidden_auth, "%s -t %d -r %d -m %s -v %s %s %s:%s %s",
+                       PATH_TO_SNMPGETNEXT, timeout_interval, retries, miblist, proto,
+                       "[authpriv]", server_address, port, oid);
        }else{
 
                asprintf (&command_line, "%s -t %d -r %d -m %s -v %s %s %s:%s %s",
-                 PATH_TO_SNMPGET, timeout_interval, retries, miblist, proto,
-                 authpriv, server_address, port, oid);
+                       PATH_TO_SNMPGET, timeout_interval, retries, miblist, proto,
+                       authpriv, server_address, port, oid);
+               asprintf(&cl_hidden_auth, "%s -t %d -r %d -m %s -v %s %s %s:%s %s",
+                       PATH_TO_SNMPGET, timeout_interval, retries, miblist, proto,
+                       "[authpriv]", server_address, port, oid);
        }
        
        if (verbose)
@@ -185,14 +207,16 @@ main (int argc, char **argv)
        /* run the command */
        child_process = spopen (command_line);
        if (child_process == NULL) {
-               printf (_("Could not open pipe: %s\n"), command_line);
+               printf (_("Could not open pipe: %s\n"), cl_hidden_auth);
                exit (STATE_UNKNOWN);
        }
 
+#if 0          /* Removed May 29, 2007 */
        child_stderr = fdopen (child_stderr_array[fileno (child_process)], "r");
        if (child_stderr == NULL) {
-               printf (_("Could not open stderr for %s\n"), command_line);
+               printf (_("Could not open stderr for %s\n"), cl_hidden_auth);
        }
+#endif
 
        while (fgets (input_buffer, MAX_INPUT_BUFFER - 1, child_process))
                asprintf (&output, "%s%s", output, input_buffer);
@@ -202,12 +226,16 @@ main (int argc, char **argv)
 
        ptr = output;
 
-       strcat(perfstr, "| ");
+       strncat(perfstr, "| ", sizeof(perfstr)-strlen(perfstr)-1);
        while (ptr) {
-               char *foo;
+               char *foo, *ptr2;
+               unsigned int copylen;
 
                foo = strstr (ptr, delimiter);
-               strncat(perfstr, ptr, foo-ptr);
+               copylen = foo-ptr;
+               if (copylen > sizeof(perfstr)-strlen(perfstr)-1)
+                       copylen = sizeof(perfstr)-strlen(perfstr)-1;
+               ptr2 = ptr;
                ptr = foo; 
 
                if (ptr == NULL)
@@ -241,6 +269,10 @@ main (int argc, char **argv)
                }
 
                /* We strip out the datatype indicator for PHBs */
+
+               /* Clean up type array - Sol10 does not necessarily zero it out */
+               bzero(type, sizeof(type));
+
                if (strstr (response, "Gauge: "))
                        show = strstr (response, "Gauge: ") + 7;
                else if (strstr (response, "Gauge32: "))
@@ -249,6 +281,10 @@ main (int argc, char **argv)
                        show = strstr (response, "Counter32: ") + 11;
                        strcpy(type, "c");
                }
+               else if (strstr (response, "Counter64: ")) {
+                       show = strstr (response, "Counter64: ") + 11;
+                       strcpy(type, "c");
+               }
                else if (strstr (response, "INTEGER: "))
                        show = strstr (response, "INTEGER: ") + 9;
                else if (strstr (response, "STRING: "))
@@ -277,7 +313,7 @@ main (int argc, char **argv)
                                die (STATE_UNKNOWN,_("No valid data returned"));
                        response_value[i] = strtoul (p2, NULL, 10);
                        iresult = check_num (i);
-                       asprintf (&show, "%lu", response_value[i]);
+                       asprintf (&show, "%llu", response_value[i]);
                }
 
                /* Process this block for string matching */
@@ -320,11 +356,11 @@ main (int argc, char **argv)
                /* Prepend a label for this OID if there is one */
                if (nlabels > (size_t)1 && (size_t)i < nlabels && labels[i] != NULL)
                        asprintf (&outbuff, "%s%s%s %s%s%s", outbuff,
-                                 (i == 0) ? " " : output_delim,
-                                 labels[i], mark (iresult), show, mark (iresult));
+                               (i == 0) ? " " : output_delim,
+                               labels[i], mark (iresult), show, mark (iresult));
                else
                        asprintf (&outbuff, "%s%s%s%s%s", outbuff, (i == 0) ? " " : output_delim,
-                                 mark (iresult), show, mark (iresult));
+                               mark (iresult), show, mark (iresult));
 
                /* Append a unit string for this OID if there is one */
                if (nunits > (size_t)0 && (size_t)i < nunits && unitv[i] != NULL)
@@ -332,27 +368,39 @@ main (int argc, char **argv)
 
                i++;
 
-               asprintf(str, "=%s%s;;;; ", show, type ? type : "");
-               strcat(perfstr, *str);
+               if (is_numeric(show)) {
+                       strncat(perfstr, ptr2, copylen);
+                       strncat(perfstr, "=", sizeof(perfstr)-strlen(perfstr)-1);
+                       strncat(perfstr, show, sizeof(perfstr)-strlen(perfstr)-1);
+
+                       if (type)
+                               strncat(perfstr, type, sizeof(perfstr)-strlen(perfstr)-1);
+                       strncat(perfstr, " ", sizeof(perfstr)-strlen(perfstr)-1);
+               }
 
        }       /* end while (ptr) */
 
        if (found == 0)
                die (STATE_UNKNOWN,
-                    _("%s problem - No data received from host\nCMD: %s\n"),
-                    label,
-                    command_line);
+                       _("%s problem - No data received from host\nCMD: %s\n"),
+                       label,
+                       cl_hidden_auth);
 
+#if 0          /* Removed May 29, 2007 */
        /* WARNING if output found on stderr */
        if (fgets (input_buffer, MAX_INPUT_BUFFER - 1, child_stderr))
                result = max_state (result, STATE_WARNING);
 
        /* close stderr */
        (void) fclose (child_stderr);
+#endif
 
        /* close the pipe */
-       if (spclose (child_process))
-               result = max_state (result, STATE_WARNING);
+       if (spclose (child_process)) {
+               if (result == STATE_OK)
+                       result = STATE_UNKNOWN;
+               asprintf (&outbuff, "%s (%s)", outbuff, _("snmpget returned an error status"));
+       }
 
 /*     if (nunits == 1 || i == 1) */
 /*             printf ("%s %s -%s %s\n", label, state_text (result), outbuff, units); */
@@ -422,7 +470,7 @@ process_arguments (int argc, char **argv)
 
                switch (c) {
                case '?':       /* usage */
-                       usage2 (_("Unknown argument"), optarg);
+                       usage5 ();
                case 'h':       /* help */
                        print_help ();
                        exit (STATE_OK); 
@@ -440,31 +488,31 @@ process_arguments (int argc, char **argv)
                case 'H':                                                                       /* Host or server */
                        server_address = optarg;
                        break;
-               case 'p':       /* TCP port number */
+               case 'p':       /* TCP port number */
                        port = optarg;
                        break;
-               case 'm':      /* List of MIBS  */
+               case 'm':       /* List of MIBS  */
                        miblist = optarg;
                        break;
-               case 'n':     /* usesnmpgetnext */
+               case 'n':       /* usesnmpgetnext */
                        usesnmpgetnext = TRUE;
                        break;
-               case 'P':     /* SNMP protocol version */
+               case 'P':       /* SNMP protocol version */
                        proto = optarg;
                        break;
-               case 'L':     /* security level */
+               case 'L':       /* security level */
                        seclevel = optarg;
                        break;
-               case 'U':     /* security username */
+               case 'U':       /* security username */
                        secname = optarg;
                        break;
-               case 'a':     /* auth protocol */
+               case 'a':       /* auth protocol */
                        authproto = optarg;
                        break;
-               case 'A':     /* auth passwd */
+               case 'A':       /* auth passwd */
                        authpasswd = optarg;
                        break;
-               case 'X':     /* priv passwd */
+               case 'X':       /* priv passwd */
                        privpasswd = optarg;
                        break;
                case 't':       /* timeout period */
@@ -477,22 +525,22 @@ process_arguments (int argc, char **argv)
        /* Test parameters */
                case 'c':                                                                       /* critical time threshold */
                        if (strspn (optarg, "0123456789:,") < strlen (optarg))
-                               usage2 (_("Invalid critical threshold: %s\n"), optarg);
+                               usage2 (_("Invalid critical threshold"), optarg);
                        for (ptr = optarg; ptr && jj < MAX_OIDS; jj++) {
-                               if (lu_getll (&lower_crit_lim[jj], ptr) == 1)
+                               if (llu_getll (&lower_crit_lim[jj], ptr) == 1)
                                        eval_method[jj] |= CRIT_LT;
-                               if (lu_getul (&upper_crit_lim[jj], ptr) == 1)
+                               if (llu_getul (&upper_crit_lim[jj], ptr) == 1)
                                        eval_method[jj] |= CRIT_GT;
                                (ptr = index (ptr, ',')) ? ptr++ : ptr;
                        }
                        break;
                case 'w':                                                                       /* warning time threshold */
                        if (strspn (optarg, "0123456789:,") < strlen (optarg))
-                               usage2 (_("Invalid warning threshold: %s\n"), optarg);
+                               usage2 (_("Invalid warning threshold"), optarg);
                        for (ptr = optarg; ptr && ii < MAX_OIDS; ii++) {
-                               if (lu_getll (&lower_warn_lim[ii], ptr) == 1)
+                               if (llu_getll (&lower_warn_lim[ii], ptr) == 1)
                                        eval_method[ii] |= WARN_LT;
-                               if (lu_getul (&upper_warn_lim[ii], ptr) == 1)
+                               if (llu_getul (&upper_warn_lim[ii], ptr) == 1)
                                        eval_method[ii] |= WARN_GT;
                                (ptr = index (ptr, ',')) ? ptr++ : ptr;
                        }
@@ -505,11 +553,11 @@ process_arguments (int argc, char **argv)
                                retries = atoi(optarg);
                        break;
                case 'o':                                                                       /* object identifier */
-                       if ( strspn( optarg, "0123456789." ) != strlen( optarg ) ) {
+                       if ( strspn( optarg, "0123456789.," ) != strlen( optarg ) ) {
                                        /*
-                                        * we have something other than digits and periods, so we
-                                        * have a mib variable, rather than just an SNMP OID, so
-                                        * we have to actually read the mib files
+                                        * we have something other than digits, periods and comas,
+                                        * so we have a mib variable, rather than just an SNMP OID,
+                                        * so we have to actually read the mib files
                                         */
                                        needmibs = TRUE;
                        }
@@ -675,14 +723,14 @@ validate_arguments ()
        
         
        
-       if (proto == NULL || (strcmp(proto,DEFAULT_PROTOCOL) == 0) ) {        /* default protocol version */
+       if (proto == NULL || (strcmp(proto,DEFAULT_PROTOCOL) == 0) ) {  /* default protocol version */
                asprintf(&proto, DEFAULT_PROTOCOL);
                asprintf(&authpriv, "%s%s", "-c ", community);
        }
-       else if ( strcmp (proto, "2c") == 0 ) {                 /* snmpv2c args */
+       else if ( strcmp (proto, "2c") == 0 ) {         /* snmpv2c args */
                asprintf(&authpriv, "%s%s", "-c ", community);
        }
-       else if ( strcmp (proto, "3") == 0 ) {                 /* snmpv3 args */
+       else if ( strcmp (proto, "3") == 0 ) {          /* snmpv3 args */
                asprintf(&proto, "%s", "3");
                
                if ( (strcmp(seclevel, "noAuthNoPriv") == 0) || seclevel == NULL ) {
@@ -797,14 +845,14 @@ check_num (int i)
 
 
 int
-lu_getll (unsigned long *ll, char *str)
+llu_getll (unsigned long long *ll, char *str)
 {
        char tmp[100];
        if (strchr (str, ':') == NULL)
                return 0;
        if (strchr (str, ',') != NULL && (strchr (str, ',') < strchr (str, ':')))
                return 0;
-       if (sscanf (str, "%lu%[:]", ll, tmp) == 2)
+       if (sscanf (str, "%llu%[:]", ll, tmp) == 2)
                return 1;
        return 0;
 }
@@ -812,14 +860,14 @@ lu_getll (unsigned long *ll, char *str)
 
 
 int
-lu_getul (unsigned long *ul, char *str)
+llu_getul (unsigned long long *ul, char *str)
 {
        char tmp[100];
-       if (sscanf (str, "%lu%[^,]", ul, tmp) == 1)
+       if (sscanf (str, "%llu%[^,]", ul, tmp) == 1)
                return 1;
-       if (sscanf (str, ":%lu%[^,]", ul, tmp) == 1)
+       if (sscanf (str, ":%llu%[^,]", ul, tmp) == 1)
                return 1;
-       if (sscanf (str, "%*u:%lu%[^,]", ul, tmp) == 1)
+       if (sscanf (str, "%*u:%llu%[^,]", ul, tmp) == 1)
                return 1;
        return 0;
 }
@@ -884,105 +932,102 @@ print_help (void)
 
        printf (COPYRIGHT, copyright, email);
 
-       printf (_("\
-Check status of remote machines and obtain sustem information via SNMP\n\n"));
+       printf ("%s\n", _("Check status of remote machines and obtain sustem information via SNMP"));
+
+  printf ("\n\n");
 
        print_usage ();
 
        printf (_(UT_HELP_VRSN));
+       printf (_(UT_EXTRA_OPTS));
 
        printf (_(UT_HOST_PORT), 'p', DEFAULT_PORT);
 
        /* SNMP and Authentication Protocol */
-       printf (_("\
- -n, --next\n\
-    Use SNMP GETNEXT instead of SNMP GET\n\
- -P, --protocol=[1|2c|3]\n\
-    SNMP protocol version\n\
- -L, --seclevel=[noAuthNoPriv|authNoPriv|authPriv]\n\
-    SNMPv3 securityLevel\n\
- -a, --authproto=[MD5|SHA]\n\
-    SNMPv3 auth proto\n"));
+       printf (" %s\n", "-n, --next");
+  printf ("    %s\n", _("Use SNMP GETNEXT instead of SNMP GET"));
+  printf (" %s\n", "-P, --protocol=[1|2c|3]");
+  printf ("    %s\n", _("SNMP protocol version"));
+  printf (" %s\n", "-L, --seclevel=[noAuthNoPriv|authNoPriv|authPriv]");
+  printf ("    %s\n", _("SNMPv3 securityLevel"));
+  printf (" %s\n", "-a, --authproto=[MD5|SHA]");
+  printf ("    %s\n", _("SNMPv3 auth proto"));
 
        /* Authentication Tokens*/
-       printf (_("\
- -C, --community=STRING\n\
-    Optional community string for SNMP communication\n\
-    (default is \"%s\")\n\
- -U, --secname=USERNAME\n\
-    SNMPv3 username\n\
- -A, --authpassword=PASSWORD\n\
-    SNMPv3 authentication password\n\
- -X, --privpasswd=PASSWORD\n\
-    SNMPv3 crypt passwd (DES)\n"), DEFAULT_COMMUNITY);
+       printf (" %s\n", "-C, --community=STRING");
+  printf ("    %s ", _("Optional community string for SNMP communication"));
+  printf ("(%s \"%s\")\n", _("default is") ,DEFAULT_COMMUNITY);
+  printf (" %s\n", "-U, --secname=USERNAME");
+  printf ("    %s\n", _("SNMPv3 username"));
+  printf (" %s\n", "-A, --authpassword=PASSWORD");
+  printf ("    %s\n", _("SNMPv3 authentication password"));
+  printf (" %s\n", "-X, --privpasswd=PASSWORD");
+  printf ("    %s\n", _("SNMPv3 privacy password"));
 
        /* OID Stuff */
-       printf (_("\
- -o, --oid=OID(s)\n\
-    Object identifier(s) or SNMP variables whose value you wish to query\n\
- -m, --miblist=STRING\n\
-    List of MIBS to be loaded (default = none if using numeric oids or 'ALL'\n\
-               for symbolic oids.)\n\
- -d, --delimiter=STRING\n\
-    Delimiter to use when parsing returned data. Default is \"%s\"\n\
-    Any data on the right hand side of the delimiter is considered\n\
-    to be the data that should be used in the evaluation.\n"), DEFAULT_DELIMITER);
+       printf (" %s\n", "-o, --oid=OID(s)");
+  printf ("    %s\n", _("Object identifier(s) or SNMP variables whose value you wish to query"));
+  printf (" %s\n", "-m, --miblist=STRING");
+  printf ("    %s\n", _("List of MIBS to be loaded (default = none if using numeric oids or 'ALL'"));
+  printf ("    %s\n", _("for symbolic oids.)"));
+  printf (" %s\n", "-d, --delimiter=STRING");
+  printf (_("    Delimiter to use when parsing returned data. Default is \"%s\""), DEFAULT_DELIMITER);
+  printf ("    %s\n", _("Any data on the right hand side of the delimiter is considered"));
+  printf ("    %s\n", _("to be the data that should be used in the evaluation."));
 
        /* Tests Against Integers */
-       printf (_("\
- -w, --warning=INTEGER_RANGE(s)\n\
-    Range(s) which will not result in a WARNING status\n\
- -c, --critical=INTEGER_RANGE(s)\n\
-    Range(s) which will not result in a CRITICAL status\n"));
+       printf (" %s\n", "-w, --warning=INTEGER_RANGE(s)");
+  printf ("    %s\n", _("Range(s) which will not result in a WARNING status"));
+  printf (" %s\n", "-c, --critical=INTEGER_RANGE(s)");
+  printf ("    %s\n", _("Range(s) which will not result in a CRITICAL status"));
 
        /* Tests Against Strings */
-       printf (_("\
- -s, --string=STRING\n\
-    Return OK state (for that OID) if STRING is an exact match\n\
- -r, --ereg=REGEX\n\
-    Return OK state (for that OID) if extended regular expression REGEX matches\n\
- -R, --eregi=REGEX\n\
-    Return OK state (for that OID) if case-insensitive extended REGEX matches\n\
- -l, --label=STRING\n\
-    Prefix label for output from plugin (default -s 'SNMP')\n"));
+       printf (" %s\n", "-s, --string=STRING");
+  printf ("    %s\n", _("Return OK state (for that OID) if STRING is an exact match"));
+  printf (" %s\n", "-r, --ereg=REGEX");
+  printf ("    %s\n", _("Return OK state (for that OID) if extended regular expression REGEX matches"));
+  printf (" %s\n", "-R, --eregi=REGEX");
+  printf ("    %s\n", _("Return OK state (for that OID) if case-insensitive extended REGEX matches"));
+  printf (" %s\n", "-l, --label=STRING");
+  printf ("    %s\n", _("Prefix label for output from plugin (default -s 'SNMP')"));
 
        /* Output Formatting */
-       printf (_("\
- -u, --units=STRING\n\
-    Units label(s) for output data (e.g., 'sec.').\n\
- -D, --output-delimiter=STRING\n\
-    Separates output on multiple OID requests\n"));
+       printf (" %s\n", "-u, --units=STRING");
+  printf ("    %s\n", _("Units label(s) for output data (e.g., 'sec.')."));
+  printf (" %s\n", "-D, --output-delimiter=STRING");
+  printf ("    %s\n", _("Separates output on multiple OID requests"));
 
        printf (_(UT_TIMEOUT), DEFAULT_SOCKET_TIMEOUT);
 
        printf (_(UT_VERBOSE));
 
-       printf (_("\n\
-- This plugin uses the 'snmpget' command included with the NET-SNMP package.\n\
-  If you don't have the package installed, you will need to download it from\n\
-  http://net-snmp.sourceforge.net before you can use this plugin.\n"));
-
-       printf (_("\
-- Multiple OIDs may be indicated by a comma- or space-delimited list (lists with\n\
-  internal spaces must be quoted) [max 8 OIDs]\n"));
-
-       printf (_("\
-- Ranges are inclusive and are indicated with colons. When specified as\n\
-  'min:max' a STATE_OK will be returned if the result is within the indicated\n\
-  range or is equal to the upper or lower bound. A non-OK state will be\n\
-  returned if the result is outside the specified range.\n"));
-
-       printf (_("\
-- If specified in the order 'max:min' a non-OK state will be returned if the\n\
-  result is within the (inclusive) range.\n"));
-
-       printf (_("\
-- Upper or lower bounds may be omitted to skip checking the respective limit.\n\
-- Bare integers are interpreted as upper limits.\n\
-- When checking multiple OIDs, separate ranges by commas like '-w 1:10,1:,:20'\n\
-- Note that only one string and one regex may be checked at present\n\
-- All evaluation methods other than PR, STR, and SUBSTR expect that the value\n\
-  returned from the SNMP query is an unsigned integer.\n"));
+  printf ("\n");
+  printf ("%s\n", _("This plugin uses the 'snmpget' command included with the NET-SNMP package."));
+  printf ("%s\n", _("if you don't have the package installed, you will need to download it from"));
+  printf ("%s\n", _("http://net-snmp.sourceforge.net before you can use this plugin."));
+
+  printf ("\n");
+  printf ("%s\n", _("Notes:"));
+  printf (" %s\n", _("- Multiple OIDs may be indicated by a comma- or space-delimited list (lists with"));
+  printf ("   %s\n", _("internal spaces must be quoted) [max 8 OIDs]"));
+
+  printf (" %s\n", _("- Ranges are inclusive and are indicated with colons. When specified as"));
+  printf ("   %s\n", _("'min:max' a STATE_OK will be returned if the result is within the indicated"));
+  printf ("   %s\n", _("range or is equal to the upper or lower bound. A non-OK state will be"));
+  printf ("   %s\n", _("returned if the result is outside the specified range."));
+
+  printf (" %s\n", _("- If specified in the order 'max:min' a non-OK state will be returned if the"));
+  printf ("   %s\n", _("result is within the (inclusive) range."));
+
+  printf (" %s\n", _("- Upper or lower bounds may be omitted to skip checking the respective limit."));
+  printf (" %s\n", _("- Bare integers are interpreted as upper limits."));
+  printf (" %s\n", _("- When checking multiple OIDs, separate ranges by commas like '-w 1:10,1:,:20'"));
+  printf (" %s\n", _("- Note that only one string and one regex may be checked at present"));
+  printf (" %s\n", _("- All evaluation methods other than PR, STR, and SUBSTR expect that the value"));
+  printf ("   %s\n", _("returned from the SNMP query is an unsigned integer."));
+#ifdef NP_EXTRA_OPTS
+  printf (" -%s", _(UT_EXTRA_OPTS_NOTES));
+#endif
 
        printf (_(UT_SUPPORT));
 }
@@ -992,12 +1037,10 @@ Check status of remote machines and obtain sustem information via SNMP\n\n"));
 void
 print_usage (void)
 {
-       printf ("\
-Usage: %s -H <ip_address> -o <OID> [-w warn_range] [-c crit_range] \n\
-                  [-C community] [-s string] [-r regex] [-R regexi]\n\
-                  [-t timeout] [-e retries]\n\
-                  [-l label] [-u units] [-p port-number] [-d delimiter]\n\
-                  [-D output-delimiter] [-m miblist] [-P snmp version]\n\
-                  [-L seclevel] [-U secname] [-a authproto] [-A authpasswd]\n\
-                  [-X privpasswd]\n", progname);
+  printf (_("Usage:"));
+       printf ("%s -H <ip_address> -o <OID> [-w warn_range] [-c crit_range]\n",progname);
+  printf ("[-C community] [-s string] [-r regex] [-R regexi] [-t timeout] [-e retries]\n");
+  printf ("[-l label] [-u units] [-p port-number] [-d delimiter] [-D output-delimiter]\n");
+  printf ("[-m miblist] [-P snmp version] [-L seclevel] [-U secname] [-a authproto]\n");
+  printf ("[-A authpasswd] [-X privpasswd]\n");
 }