Code

Detect arguments passed via --with-ping[6]-command (#2908236)
[nagiosplug.git] / plugins / check_ntp_time.c
index 164d519051f1d3b498c214f8a8269c974dd93e28..c7c7a754ca3b82409eb0c25fde9c22d0295597b1 100644 (file)
@@ -1,44 +1,41 @@
-/******************************************************************************
-*
-* Nagios check_ntp plugin
-*
+/*****************************************************************************
+* 
+* Nagios check_ntp_time plugin
+* 
 * License: GPL
-* Copyright (c) 2006 sean finney <seanius@seanius.net>
-* Copyright (c) 2007 nagios-plugins team
-*
-* Last Modified: $Date$
-*
+* Copyright (c) 2006 Sean Finney <seanius@seanius.net>
+* Copyright (c) 2006-2008 Nagios Plugins Development Team
+* 
 * Description:
-*
-* This file contains the check_ntp plugin
-*
-*  This plugin to check ntp servers independant of any commandline
-*  programs or external libraries.
-*
-*
-* License Information:
-*
-* This program is free software; you can redistribute it and/or modify
+* 
+* This file contains the check_ntp_time plugin
+* 
+* This plugin checks the clock offset between the local host and a
+* remote NTP server. It is independent of any commandline programs or
+* external libraries.
+* 
+* If you'd rather want to monitor an NTP server, please use
+* check_ntp_peer.
+* 
+* 
+* 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_ntp";
-const char *revision = "$Revision$";
-const char *copyright = "2007";
+const char *progname = "check_ntp_time";
+const char *copyright = "2006-2008";
 const char *email = "nagiosplug-devel@lists.sourceforge.net";
 
 #include "common.h"
@@ -46,17 +43,14 @@ const char *email = "nagiosplug-devel@lists.sourceforge.net";
 #include "utils.h"
 
 static char *server_address=NULL;
+static char *port="123";
 static int verbose=0;
-static short do_offset=0;
+static int quiet=0;
 static char *owarn="60";
 static char *ocrit="120";
-static short do_jitter=0;
-static char *jwarn="5000";
-static char *jcrit="10000";
 
 int process_arguments (int, char **);
 thresholds *offset_thresholds = NULL;
-thresholds *jitter_thresholds = NULL;
 void print_help (void);
 void print_usage (void);
 
@@ -83,7 +77,7 @@ typedef struct {
 
 /* this structure holds data about results from querying offset from a peer */
 typedef struct {
-       time_t waiting;         /* ts set when we started waiting for a response */ 
+       time_t waiting;         /* ts set when we started waiting for a response */
        int num_responses;      /* number of successfully recieved responses */
        uint8_t stratum;        /* copied verbatim from the ntp_message */
        double rtdelay;         /* converted from the ntp_message */
@@ -92,25 +86,6 @@ typedef struct {
        uint8_t flags;       /* byte with leapindicator,vers,mode. see macros */
 } ntp_server_results;
 
-/* this structure holds everything in an ntp control message as per rfc1305 */
-typedef struct {
-       uint8_t flags;       /* byte with leapindicator,vers,mode. see macros */
-       uint8_t op;          /* R,E,M bits and Opcode */
-       uint16_t seq;        /* Packet sequence */
-       uint16_t status;     /* Clock status */
-       uint16_t assoc;      /* Association */
-       uint16_t offset;     /* Similar to TCP sequence # */
-       uint16_t count;      /* # bytes of data */
-       char data[MAX_CM_SIZE]; /* ASCII data of the request */
-                               /* NB: not necessarily NULL terminated! */
-} ntp_control_message;
-
-/* this is an association/status-word pair found in control packet reponses */
-typedef struct {
-       uint16_t assoc;
-       uint16_t status;
-} ntp_assoc_status_pair;
-
 /* bits 1,2 are the leap indicator */
 #define LI_MASK 0xc0
 #define LI(x) ((x&LI_MASK)>>6)
@@ -153,7 +128,7 @@ typedef struct {
  they are divided into halves, each being a 16-bit int in network byte order:
  - the first 16 bits are an int on the left side of a decimal point.
  - the second 16 bits represent a fraction n/(2^16)
- likewise for the 64-bit "fixed point" numbers with everything doubled :) 
+ likewise for the 64-bit "fixed point" numbers with everything doubled :)
  **/
 
 /* macros to access the left/right 16 bits of a 32-bit ntp "fixed point"
@@ -246,42 +221,6 @@ void print_ntp_message(const ntp_message *p){
        printf("\ttxts = %-.16g\n", NTP64asDOUBLE(p->txts));
 }
 
-void print_ntp_control_message(const ntp_control_message *p){
-       int i=0, numpeers=0;
-       const ntp_assoc_status_pair *peer=NULL;
-
-       printf("control packet contents:\n");
-       printf("\tflags: 0x%.2x , 0x%.2x\n", p->flags, p->op);
-       printf("\t  li=%d (0x%.2x)\n", LI(p->flags), p->flags&LI_MASK);
-       printf("\t  vn=%d (0x%.2x)\n", VN(p->flags), p->flags&VN_MASK);
-       printf("\t  mode=%d (0x%.2x)\n", MODE(p->flags), p->flags&MODE_MASK);
-       printf("\t  response=%d (0x%.2x)\n", (p->op&REM_RESP)>0, p->op&REM_RESP);
-       printf("\t  more=%d (0x%.2x)\n", (p->op&REM_MORE)>0, p->op&REM_MORE);
-       printf("\t  error=%d (0x%.2x)\n", (p->op&REM_ERROR)>0, p->op&REM_ERROR);
-       printf("\t  op=%d (0x%.2x)\n", p->op&OP_MASK, p->op&OP_MASK);
-       printf("\tsequence: %d (0x%.2x)\n", ntohs(p->seq), ntohs(p->seq));
-       printf("\tstatus: %d (0x%.2x)\n", ntohs(p->status), ntohs(p->status));
-       printf("\tassoc: %d (0x%.2x)\n", ntohs(p->assoc), ntohs(p->assoc));
-       printf("\toffset: %d (0x%.2x)\n", ntohs(p->offset), ntohs(p->offset));
-       printf("\tcount: %d (0x%.2x)\n", ntohs(p->count), ntohs(p->count));
-       numpeers=ntohs(p->count)/(sizeof(ntp_assoc_status_pair));
-       if(p->op&REM_RESP && p->op&OP_READSTAT){
-               peer=(ntp_assoc_status_pair*)p->data;
-               for(i=0;i<numpeers;i++){
-                       printf("\tpeer id %.2x status %.2x", 
-                              ntohs(peer[i].assoc), ntohs(peer[i].status));
-                       if (PEER_SEL(peer[i].status) >= PEER_INCLUDED){
-                               if(PEER_SEL(peer[i].status) >= PEER_SYNCSOURCE){
-                                       printf(" <-- current sync source");
-                               } else {
-                                       printf(" <-- current sync candidate");
-                               }
-                       }
-                       printf("\n");
-               }
-       }
-}
-
 void setup_request(ntp_message *p){
        struct timeval t;
 
@@ -302,50 +241,52 @@ void setup_request(ntp_message *p){
  * this is done by filtering servers based on stratum, dispersion, and
  * finally round-trip delay. */
 int best_offset_server(const ntp_server_results *slist, int nservers){
-       int i=0, j=0, cserver=0, candidates[5], csize=0;
+       int i=0, cserver=0, best_server=-1;
 
        /* for each server */
        for(cserver=0; cserver<nservers; cserver++){
-               /* sort out servers with error flags */
-               if ( LI(slist[cserver].flags) != LI_NOWARNING ){
-                       if (verbose) printf("discarding peer id %d: flags=%d\n", cserver, LI(slist[cserver].flags));
-                       break;
+               /* We don't want any servers that fails these tests */
+               /* Sort out servers that didn't respond or responede with a 0 stratum;
+                * stratum 0 is for reference clocks so no NTP server should ever report
+                * a stratum 0 */
+               if ( slist[cserver].stratum == 0){
+                       if (verbose) printf("discarding peer %d: stratum=%d\n", cserver, slist[cserver].stratum);
+                       continue;
+               }
+               /* Sort out servers with error flags */
+               if ( LI(slist[cserver].flags) == LI_ALARM ){
+                       if (verbose) printf("discarding peer %d: flags=%d\n", cserver, LI(slist[cserver].flags));
+                       continue;
                }
 
-               /* compare it to each of the servers already in the candidate list */
-               for(i=0; i<csize; i++){
-                       /* does it have an equal or better stratum? */
-                       if(slist[cserver].stratum <= slist[i].stratum){
-                               /* does it have an equal or better dispersion? */
-                               if(slist[cserver].rtdisp <= slist[i].rtdisp){
-                                       /* does it have a better rtdelay? */
-                                       if(slist[cserver].rtdelay < slist[i].rtdelay){
-                                               break;
-                                       }
-                               }
-                       }
+               /* If we don't have a server yet, use the first one */
+               if (best_server == -1) {
+                       best_server = cserver;
+                       DBG(printf("using peer %d as our first candidate\n", best_server));
+                       continue;
                }
 
-               /* if we haven't reached the current list's end, move everyone
-                * over one to the right, and insert the new candidate */
-               if(i<csize){
-                       for(j=5; j>i; j--){
-                               candidates[j]=candidates[j-1];
+               /* compare the server to the best one we've seen so far */
+               /* does it have an equal or better stratum? */
+               DBG(printf("comparing peer %d with peer %d\n", cserver, best_server));
+               if(slist[cserver].stratum <= slist[best_server].stratum){
+                       DBG(printf("stratum for peer %d <= peer %d\n", cserver, best_server));
+                       /* does it have an equal or better dispersion? */
+                       if(slist[cserver].rtdisp <= slist[best_server].rtdisp){
+                               DBG(printf("dispersion for peer %d <= peer %d\n", cserver, best_server));
+                               /* does it have a better rtdelay? */
+                               if(slist[cserver].rtdelay < slist[best_server].rtdelay){
+                                       DBG(printf("rtdelay for peer %d < peer %d\n", cserver, best_server));
+                                       best_server = cserver;
+                                       DBG(printf("peer %d is now our best candidate\n", best_server));
+                               }
                        }
                }
-               /* regardless, if they should be on the list... */
-               if(i<5) {
-                       candidates[i]=cserver;
-                       if(csize<5) csize++;
-               /* otherwise discard the server */
-               } else {
-                       DBG(printf("discarding peer id %d\n", cserver));
-               }
        }
 
-       if(csize>0) {
-               DBG(printf("best server selected: peer %d\n", candidates[0]));
-               return candidates[0];
+       if(best_server >= 0) {
+               DBG(printf("best server selected: peer %d\n", best_server));
+               return best_server;
        } else {
                DBG(printf("no peers meeting synchronization criteria :(\n"));
                return -1;
@@ -354,7 +295,7 @@ int best_offset_server(const ntp_server_results *slist, int nservers){
 
 /* do everything we need to get the total average offset
  * - we use a certain amount of parallelization with poll() to ensure
- *   we don't waste time sitting around waiting for single packets. 
+ *   we don't waste time sitting around waiting for single packets.
  * - we also "manually" handle resolving host names and connecting, because
  *   we have to do it in a way that our lazy macros don't handle currently :( */
 double offset_request(const char *host, int *status){
@@ -375,7 +316,7 @@ double offset_request(const char *host, int *status){
        hints.ai_socktype = SOCK_DGRAM;
 
        /* fill in ai with the list of hosts resolved by the host name */
-       ga_result = getaddrinfo(host, "123", &hints, &ai);
+       ga_result = getaddrinfo(host, port, &hints, &ai);
        if(ga_result!=0){
                die(STATE_UNKNOWN, "error getting address for %s: %s\n",
                    host, gai_strerror(ga_result));
@@ -392,6 +333,7 @@ double offset_request(const char *host, int *status){
        servers=(ntp_server_results*)malloc(sizeof(ntp_server_results)*num_hosts);
        if(servers==NULL) die(STATE_UNKNOWN, "can not allocate server array");
        memset(servers, 0, sizeof(ntp_server_results)*num_hosts);
+       DBG(printf("Found %d peers to check\n", num_hosts));
 
        /* setup each socket for writing, and the corresponding struct pollfd */
        ai_tmp=ai;
@@ -411,13 +353,13 @@ double offset_request(const char *host, int *status){
                ai_tmp = ai_tmp->ai_next;
        }
 
-       /* now do AVG_NUM checks to each host.  we stop before timeout/2 seconds
+       /* now do AVG_NUM checks to each host. We stop before timeout/2 seconds
         * have passed in order to ensure post-processing and jitter time. */
        now_time=start_ts=time(NULL);
        while(servers_completed<num_hosts && now_time-start_ts <= socket_timeout/2){
                /* loop through each server and find each one which hasn't
                 * been touched in the past second or so and is still lacking
-                * some responses.  for each of these servers, send a new request,
+                * some responses. For each of these servers, send a new request,
                 * and update the "waiting" timestamp with the current time. */
                one_written=0;
                now_time=time(NULL);
@@ -486,10 +428,7 @@ double offset_request(const char *host, int *status){
        }
 
        /* cleanup */
-       /* FIXME: Not closing the socket to avoid re-use of the local port
-        * which can cause old NTP packets to be read instead of NTP control
-        * pactets in jitter_request(). THERE MUST BE ANOTHER WAY...
-        * for(j=0; j<num_hosts; j++){ close(socklist[j]); } */
+       for(j=0; j<num_hosts; j++){ close(socklist[j]); }
        free(socklist);
        free(ufds);
        free(servers);
@@ -500,155 +439,6 @@ double offset_request(const char *host, int *status){
        return avg_offset;
 }
 
-void
-setup_control_request(ntp_control_message *p, uint8_t opcode, uint16_t seq){
-       memset(p, 0, sizeof(ntp_control_message));
-       LI_SET(p->flags, LI_NOWARNING);
-       VN_SET(p->flags, VN_RESERVED);
-       MODE_SET(p->flags, MODE_CONTROLMSG);
-       OP_SET(p->op, opcode);
-       p->seq = htons(seq);
-       /* Remaining fields are zero for requests */
-}
-
-/* XXX handle responses with the error bit set */
-double jitter_request(const char *host, int *status){
-       int conn=-1, i, npeers=0, num_candidates=0, syncsource_found=0;
-       int run=0, min_peer_sel=PEER_INCLUDED, num_selected=0, num_valid=0;
-       int peers_size=0, peer_offset=0;
-       ntp_assoc_status_pair *peers=NULL;
-       ntp_control_message req;
-       const char *getvar = "jitter";
-       double rval = 0.0, jitter = -1.0;
-       char *startofvalue=NULL, *nptr=NULL;
-       void *tmp;
-
-       /* Long-winded explanation:
-        * Getting the jitter requires a number of steps:
-        * 1) Send a READSTAT request.
-        * 2) Interpret the READSTAT reply
-        *  a) The data section contains a list of peer identifiers (16 bits)
-        *     and associated status words (16 bits)
-        *  b) We want the value of 0x06 in the SEL (peer selection) value,
-        *     which means "current synchronizatin source".  If that's missing,
-        *     we take anything better than 0x04 (see the rfc for details) but
-        *     set a minimum of warning.
-        * 3) Send a READVAR request for information on each peer identified
-        *    in 2b greater than the minimum selection value.
-        * 4) Extract the jitter value from the data[] (it's ASCII)
-        */
-       my_udp_connect(server_address, 123, &conn);
-
-       /* keep sending requests until the server stops setting the
-        * REM_MORE bit, though usually this is only 1 packet. */
-       do{
-               setup_control_request(&req, OP_READSTAT, 1);
-               DBG(printf("sending READSTAT request"));
-               write(conn, &req, SIZEOF_NTPCM(req));
-               DBG(print_ntp_control_message(&req));
-               /* Attempt to read the largest size packet possible */
-               req.count=htons(MAX_CM_SIZE);
-               DBG(printf("recieving READSTAT response"))
-               read(conn, &req, SIZEOF_NTPCM(req));
-               DBG(print_ntp_control_message(&req));
-               /* Each peer identifier is 4 bytes in the data section, which
-                * we represent as a ntp_assoc_status_pair datatype.
-                */
-               peers_size+=ntohs(req.count);
-               if((tmp=realloc(peers, peers_size)) == NULL)
-                       free(peers), die(STATE_UNKNOWN, "can not (re)allocate 'peers' buffer\n");
-               peers=tmp;
-               memcpy((void*)((ptrdiff_t)peers+peer_offset), (void*)req.data, ntohs(req.count));
-               npeers=peers_size/sizeof(ntp_assoc_status_pair);
-               peer_offset+=ntohs(req.count);
-       } while(req.op&REM_MORE);
-
-       /* first, let's find out if we have a sync source, or if there are
-        * at least some candidates.  in the case of the latter we'll issue
-        * a warning but go ahead with the check on them. */
-       for (i = 0; i < npeers; i++){
-               if (PEER_SEL(peers[i].status) >= PEER_INCLUDED){
-                       num_candidates++;
-                       if(PEER_SEL(peers[i].status) >= PEER_SYNCSOURCE){
-                               syncsource_found=1;
-                               min_peer_sel=PEER_SYNCSOURCE;
-                       }
-               }
-       }
-       if(verbose) printf("%d candiate peers available\n", num_candidates);
-       if(verbose && syncsource_found) printf("synchronization source found\n");
-       if(! syncsource_found){
-               *status = STATE_UNKNOWN;
-               if(verbose) printf("warning: no synchronization source found\n");
-       }
-
-
-       for (run=0; run<AVG_NUM; run++){
-               if(verbose) printf("jitter run %d of %d\n", run+1, AVG_NUM);
-               for (i = 0; i < npeers; i++){
-                       /* Only query this server if it is the current sync source */
-                       if (PEER_SEL(peers[i].status) >= min_peer_sel){
-                               num_selected++;
-                               setup_control_request(&req, OP_READVAR, 2);
-                               req.assoc = peers[i].assoc;
-                               /* By spec, putting the variable name "jitter"  in the request
-                                * should cause the server to provide _only_ the jitter value.
-                                * thus reducing net traffic, guaranteeing us only a single
-                                * datagram in reply, and making intepretation much simpler
-                                */
-                               /* Older servers doesn't know what jitter is, so if we get an
-                                * error on the first pass we redo it with "dispersion" */
-                               strncpy(req.data, getvar, MAX_CM_SIZE-1);
-                               req.count = htons(strlen(getvar));
-                               DBG(printf("sending READVAR request...\n"));
-                               write(conn, &req, SIZEOF_NTPCM(req));
-                               DBG(print_ntp_control_message(&req));
-
-                               req.count = htons(MAX_CM_SIZE);
-                               DBG(printf("recieving READVAR response...\n"));
-                               read(conn, &req, SIZEOF_NTPCM(req));
-                               DBG(print_ntp_control_message(&req));
-
-                               if(req.op&REM_ERROR && strstr(getvar, "jitter")) {
-                                       if(verbose) printf("The 'jitter' command failed (old ntp server?)\nRestarting with 'dispersion'...\n");
-                                       getvar = "dispersion";
-                                       num_selected--;
-                                       i--;
-                                       continue;
-                               }
-
-                               /* get to the float value */
-                               if(verbose) {
-                                       printf("parsing jitter from peer %.2x: ", ntohs(peers[i].assoc));
-                               }
-                               startofvalue = strchr(req.data, '=');
-                               if(startofvalue != NULL) {
-                                       startofvalue++;
-                                       jitter = strtod(startofvalue, &nptr);
-                               }
-                               if(startofvalue == NULL || startofvalue==nptr){
-                                       printf("warning: unable to read server jitter response.\n");
-                                       *status = STATE_UNKNOWN;
-                               } else {
-                                       if(verbose) printf("%g\n", jitter);
-                                       num_valid++;
-                                       rval += jitter;
-                               }
-                       }
-               }
-               if(verbose){
-                       printf("jitter parsed from %d/%d peers\n", num_valid, num_selected);
-               }
-       }
-
-       rval = num_valid ? rval / num_valid : -1.0;
-
-       close(conn);
-       if(peers!=NULL) free(peers);
-       /* If we return -1.0, it means no synchronization source was found */
-       return rval;
-}
-
 int process_arguments(int argc, char **argv){
        int c;
        int option=0;
@@ -658,21 +448,21 @@ int process_arguments(int argc, char **argv){
                {"verbose", no_argument, 0, 'v'},
                {"use-ipv4", no_argument, 0, '4'},
                {"use-ipv6", no_argument, 0, '6'},
+               {"quiet", no_argument, 0, 'q'},
                {"warning", required_argument, 0, 'w'},
                {"critical", required_argument, 0, 'c'},
-               {"jwarn", required_argument, 0, 'j'},
-               {"jcrit", required_argument, 0, 'k'},
                {"timeout", required_argument, 0, 't'},
                {"hostname", required_argument, 0, 'H'},
+               {"port", required_argument, 0, 'p'},
                {0, 0, 0, 0}
        };
 
-       
+
        if (argc < 2)
                usage ("\n");
 
        while (1) {
-               c = getopt_long (argc, argv, "Vhv46w:c:j:k:t:H:", longopts, &option);
+               c = getopt_long (argc, argv, "Vhv46qw:c:t:H:p:", longopts, &option);
                if (c == -1 || c == EOF || c == 1)
                        break;
 
@@ -682,33 +472,29 @@ int process_arguments(int argc, char **argv){
                        exit(STATE_OK);
                        break;
                case 'V':
-                       print_revision(progname, revision);
+                       print_revision(progname, NP_VERSION);
                        exit(STATE_OK);
                        break;
                case 'v':
                        verbose++;
                        break;
+               case 'q':
+                       quiet = 1;
+                       break;
                case 'w':
-                       do_offset=1;
                        owarn = optarg;
                        break;
                case 'c':
-                       do_offset=1;
                        ocrit = optarg;
                        break;
-               case 'j':
-                       do_jitter=1;
-                       jwarn = optarg;
-                       break;
-               case 'k':
-                       do_jitter=1;
-                       jcrit = optarg;
-                       break;
                case 'H':
                        if(is_host(optarg) == FALSE)
                                usage2(_("Invalid hostname/address"), optarg);
                        server_address = strdup(optarg);
                        break;
+               case 'p':
+                       port = strdup(optarg);
+                       break;
                case 't':
                        socket_timeout=atoi(optarg);
                        break;
@@ -744,26 +530,24 @@ char *perfd_offset (double offset)
                FALSE, 0, FALSE, 0);
 }
 
-char *perfd_jitter (double jitter)
-{
-       return fperfdata ("jitter", jitter, "s",
-               do_jitter, jitter_thresholds->warning->end,
-               do_jitter, jitter_thresholds->critical->end,
-               TRUE, 0, FALSE, 0);
-}
-
 int main(int argc, char *argv[]){
-       int result, offset_result, jitter_result;
-       double offset=0, jitter=0;
+       int result, offset_result;
+       double offset=0;
        char *result_line, *perfdata_line;
 
-       result = offset_result = jitter_result = STATE_OK;
+       setlocale (LC_ALL, "");
+       bindtextdomain (PACKAGE, LOCALEDIR);
+       textdomain (PACKAGE);
+
+       result = offset_result = STATE_OK;
+
+       /* Parse extra opts if any */
+       argv=np_extra_opts (&argc, argv, progname);
 
        if (process_arguments (argc, argv) == ERROR)
                usage4 (_("Could not parse arguments"));
 
        set_thresholds(&offset_thresholds, owarn, ocrit);
-       set_thresholds(&jitter_thresholds, jwarn, jcrit);
 
        /* initialize alarm signal handling */
        signal (SIGALRM, socket_timeout_alarm_handler);
@@ -772,99 +556,81 @@ int main(int argc, char *argv[]){
        alarm (socket_timeout);
 
        offset = offset_request(server_address, &offset_result);
-       /* check_ntp used to always return CRITICAL if offset_result == STATE_UNKNOWN.
-        * Now we'll only do that is the offset thresholds were set */
-       if (do_offset && offset_result == STATE_UNKNOWN) {
-               result = STATE_CRITICAL;
+       if (offset_result == STATE_UNKNOWN) {
+               result = (quiet == 1 ? STATE_UNKNOWN : STATE_CRITICAL);
        } else {
                result = get_status(fabs(offset), offset_thresholds);
        }
 
-       /* If not told to check the jitter, we don't even send packets.
-        * jitter is checked using NTP control packets, which not all
-        * servers recognize.  Trying to check the jitter on OpenNTPD
-        * (for example) will result in an error
-        */
-       if(do_jitter){
-               jitter=jitter_request(server_address, &jitter_result);
-               result = max_state_alt(result, get_status(jitter, jitter_thresholds));
-               /* -1 indicates that we couldn't calculate the jitter
-                * Only overrides STATE_OK from the offset */
-               if(jitter == -1.0 && result == STATE_OK)
-                       result = STATE_UNKNOWN;
-       }
-       result = max_state_alt(result, jitter_result);
-
        switch (result) {
                case STATE_CRITICAL :
-                       asprintf(&result_line, "NTP CRITICAL:");
+                       asprintf(&result_line, _("NTP CRITICAL:"));
                        break;
                case STATE_WARNING :
-                       asprintf(&result_line, "NTP WARNING:");
+                       asprintf(&result_line, _("NTP WARNING:"));
                        break;
                case STATE_OK :
-                       asprintf(&result_line, "NTP OK:");
+                       asprintf(&result_line, _("NTP OK:"));
                        break;
                default :
-                       asprintf(&result_line, "NTP UNKNOWN:");
+                       asprintf(&result_line, _("NTP UNKNOWN:"));
                        break;
        }
        if(offset_result == STATE_UNKNOWN){
                asprintf(&result_line, "%s %s", result_line, _("Offset unknown"));
                asprintf(&perfdata_line, "");
        } else {
-               asprintf(&result_line, "%s Offset %.10g secs", result_line, offset);
+               asprintf(&result_line, "%s %s %.10g secs", result_line, _("Offset"), offset);
                asprintf(&perfdata_line, "%s", perfd_offset(offset));
        }
-       if (do_jitter) {
-               asprintf(&result_line, "%s, jitter=%f", result_line, jitter);
-               asprintf(&perfdata_line, "%s %s", perfdata_line,  perfd_jitter(jitter));
-       }
        printf("%s|%s\n", result_line, perfdata_line);
 
        if(server_address!=NULL) free(server_address);
        return result;
 }
 
-
-
 void print_help(void){
-       print_revision(progname, revision);
+       print_revision(progname, NP_VERSION);
 
        printf ("Copyright (c) 2006 Sean Finney\n");
        printf (COPYRIGHT, copyright, email);
-  
-  printf ("%s\n", _("This plugin checks the selected ntp server"));
 
-  printf ("\n\n");
-  
+       printf ("%s\n", _("This plugin checks the clock offset with the ntp server"));
+
+       printf ("\n\n");
+
        print_usage();
        printf (_(UT_HELP_VRSN));
+       printf (_(UT_EXTRA_OPTS));
        printf (_(UT_HOST_PORT), 'p', "123");
+       printf (" %s\n", "-q, --quiet");
+       printf ("    %s\n", _("Returns UNKNOWN instead of CRITICAL if offset cannot be found"));
        printf (" %s\n", "-w, --warning=THRESHOLD");
        printf ("    %s\n", _("Offset to result in warning status (seconds)"));
        printf (" %s\n", "-c, --critical=THRESHOLD");
        printf ("    %s\n", _("Offset to result in critical status (seconds)"));
-       printf (" %s\n", "-j, --warning=THRESHOLD");
-       printf ("    %s\n", _("Warning threshold for jitter"));
-       printf (" %s\n", "-k, --critical=THRESHOLD");
-       printf ("    %s\n", _("Critical threshold for jitter"));
        printf (_(UT_TIMEOUT), DEFAULT_SOCKET_TIMEOUT);
        printf (_(UT_VERBOSE));
 
+       printf("\n");
+       printf("%s\n", _("This plugin checks the clock offset between the local host and a"));
+       printf("%s\n", _("remote NTP server. It is independent of any commandline programs or"));
+       printf("%s\n", _("external libraries."));
+
        printf("\n");
        printf("%s\n", _("Notes:"));
-       printf(" %s\n", _("See:"));
-       printf(" %s\n", ("http://nagiosplug.sourceforge.net/developer-guidelines.html#THRESHOLDFORMAT"));
-       printf(" %s\n", _("for THRESHOLD format and examples."));
+       printf(" %s\n", _("If you'd rather want to monitor an NTP server, please use"));
+       printf(" %s\n", _("check_ntp_peer."));
+       printf("\n");
+       printf(_(UT_THRESHOLDS_NOTES));
+#ifdef NP_EXTRA_OPTS
+       printf("\n");
+       printf(_(UT_EXTRA_OPTS_NOTES));
+#endif
 
        printf("\n");
        printf("%s\n", _("Examples:"));
-       printf(" %s\n", _("Normal offset check:"));
-       printf("  %s\n", ("./check_ntp -H ntpserv -w 0.5 -c 1"));
-       printf(" %s\n", _("Check jitter too, avoiding critical notifications if jitter isn't available"));
-       printf(" %s\n", _("(See Notes above for more details on thresholds formats):"));
-       printf("  %s\n", ("./check_ntp -H ntpserv -w 0.5 -c 1 -j -1:100 -k -1:200"));
+       printf("  %s\n", ("./check_ntp_time -H ntpserv -w 0.5 -c 1"));
 
        printf (_(UT_SUPPORT));
 }
@@ -872,6 +638,7 @@ void print_help(void){
 void
 print_usage(void)
 {
-  printf (_("Usage:"));
-  printf(" %s -H <host> [-w <warn>] [-c <crit>] [-j <warn>] [-k <crit>] [-v verbose]\n", progname);
+       printf (_("Usage:"));
+       printf(" %s -H <host> [-w <warn>] [-c <crit>] [-v verbose]\n", progname);
 }
+