Code

Fixed --help output (Ask Bjoern Hansen - #1714823)
[nagiosplug.git] / plugins / check_ntp.c
index 86709a1fbdebadb27486cbda5f2c737c630c797c..a884c15ea9b486311708171147bec4eab8cb9592 100644 (file)
@@ -1,21 +1,36 @@
 /******************************************************************************
- check_ntp.c: utility to check ntp servers independant of any commandline
-              programs or external libraries.
- original author: sean finney <seanius@seanius.net>
- ******************************************************************************
- 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.
+*
+* Nagios check_ntp plugin
+*
+* License: GPL
+* Copyright (c) 2006 sean finney <seanius@seanius.net>
+* Copyright (c) 2006 nagios-plugins team
+*
+* Last Modified: $Date$
+*
+* 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
+* 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$
  
@@ -32,12 +47,11 @@ const char *email = "nagiosplug-devel@lists.sourceforge.net";
 
 static char *server_address=NULL;
 static int verbose=0;
-static int zero_offset_bad=0;
-static double owarn=0;
-static double ocrit=0;
+static double owarn=60;
+static double ocrit=120;
 static short do_jitter=0;
-static double jwarn=0;
-static double jcrit=0;
+static double jwarn=5000;
+static double jcrit=10000;
 
 int process_arguments (int, char **);
 void print_help (void);
@@ -64,6 +78,16 @@ typedef struct {
        uint64_t txts;       /* time at which request departed server */
 } ntp_message;
 
+/* 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 */ 
+       int num_responses;      /* number of successfully recieved responses */
+       uint8_t stratum;        /* copied verbatim from the ntp_message */
+       double rtdelay;         /* converted from the ntp_message */
+       double rtdisp;          /* converted from the ntp_message */
+       double offset[AVG_NUM]; /* offsets from each response */
+} 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 */
@@ -114,8 +138,8 @@ typedef struct {
 #define OP_SET(x,y)   do{ x |= (y&OP_MASK); }while(0)
 #define OP_READSTAT 0x01
 #define OP_READVAR  0x02
-/* In peer status bytes, bytes 6,7,8 determine clock selection status */
-#define PEER_SEL(x) (x&0x07)
+/* In peer status bytes, bits 6,7,8 determine clock selection status */
+#define PEER_SEL(x) ((ntohs(x)>>8)&0x07)
 #define PEER_INCLUDED 0x04
 #define PEER_SYNCSOURCE 0x06
 
@@ -166,7 +190,7 @@ typedef struct {
        do{ if(!t.tv_usec && !t.tv_sec) n=0x0UL; \
                else { \
                        L32(n)=htonl(t.tv_sec + EPOCHDIFF); \
-                       R32(n)=htonl((4294.967296*t.tv_usec)+.5); \
+                       R32(n)=htonl((uint64_t)((4294.967296*t.tv_usec)+.5)); \
                } \
        } while(0)
 
@@ -184,13 +208,12 @@ typedef struct {
 
 /* calculate the offset of the local clock */
 static inline double calc_offset(const ntp_message *m, const struct timeval *t){
-       double client_tx, peer_rx, peer_tx, client_rx, rtdelay;
+       double client_tx, peer_rx, peer_tx, client_rx;
        client_tx = NTP64asDOUBLE(m->origts);
        peer_rx = NTP64asDOUBLE(m->rxts);
        peer_tx = NTP64asDOUBLE(m->txts);
        client_rx=TVasDOUBLE((*t));
-       rtdelay=NTP32asDOUBLE(m->rtdelay);
-       return (.5*((peer_tx-client_rx)+(peer_rx-client_tx)))-rtdelay;
+       return (.5*((peer_tx-client_rx)+(peer_rx-client_tx)));
 }
 
 /* print out a ntp packet in human readable/debuggable format */
@@ -263,7 +286,7 @@ void setup_request(ntp_message *p){
        VN_SET(p->flags, 4);
        MODE_SET(p->flags, MODE_CLIENT);
        p->poll=4;
-       p->precision=0xfa;
+       p->precision=(int8_t)0xfa;
        L16(p->rtdelay)=htons(1);
        L16(p->rtdisp)=htons(1);
 
@@ -271,38 +294,69 @@ void setup_request(ntp_message *p){
        TVtoNTP64(t,p->txts);
 }
 
-double offset_request(const char *host){
-       int i=0, conn=-1;
-       ntp_message req;
-       double next_offset=0., avg_offset=0.;
-       struct timeval recv_time;
+/* select the "best" server from a list of servers, and return its index.
+ * 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;
+
+       /* for each server */
+       for(cserver=0; cserver<nservers; cserver++){
+               /* 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 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];
+                       }
+               }
+               /* 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));
+               }
+       }
 
-       for(i=0; i<AVG_NUM; i++){
-               if(verbose) printf("offset run: %d/%d\n", i+1, AVG_NUM);
-               setup_request(&req);
-               my_udp_connect(server_address, 123, &conn);
-               write(conn, &req, sizeof(ntp_message));
-               read(conn, &req, sizeof(ntp_message));
-               gettimeofday(&recv_time, NULL);
-               /* if(verbose) print_packet(&req); */
-               close(conn);
-               next_offset=calc_offset(&req, &recv_time);
-               if(verbose) printf("offset: %g\n", next_offset);
-               avg_offset+=next_offset;
+       if(csize>0) {
+               DBG(printf("best server selected: peer %d\n", candidates[0]));
+               return candidates[0];
+       } else {
+               DBG(printf("no peers meeting synchronization criteria :(\n"));
+               return -1;
        }
-       avg_offset/=AVG_NUM;
-       if(verbose) printf("average offset: %g\n", avg_offset);
-       return avg_offset;
 }
 
-
-/* this should behave more like ntpdate, but needs optomisations... */
-double offset_request_ntpdate(const char *host){
-       int i=0, j=0, ga_result=0, num_hosts=0, *socklist=NULL;
-       ntp_message req;
-       double offset=0., avg_offset=0.;
+/* 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 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){
+       int i=0, j=0, ga_result=0, num_hosts=0, *socklist=NULL, respnum=0;
+       int servers_completed=0, one_written=0, one_read=0, servers_readable=0, best_index=-1;
+       time_t now_time=0, start_ts=0;
+       ntp_message *req=NULL;
+       double avg_offset=0.;
        struct timeval recv_time;
        struct addrinfo *ai=NULL, *ai_tmp=NULL, hints;
+       struct pollfd *ufds=NULL;
+       ntp_server_results *servers=NULL;
 
        /* setup hints to only return results from getaddrinfo that we'd like */
        memset(&hints, 0, sizeof(struct addrinfo));
@@ -310,24 +364,26 @@ double offset_request_ntpdate(const char *host){
        hints.ai_protocol = IPPROTO_UDP;
        hints.ai_socktype = SOCK_DGRAM;
 
-       /* XXX better error handling here... */
+       /* fill in ai with the list of hosts resolved by the host name */
        ga_result = getaddrinfo(host, "123", &hints, &ai);
        if(ga_result!=0){
-               fprintf(stderr, "error getting address for %s: %s\n",
-                               host, gai_strerror(ga_result));
-               return -1.0;
+               die(STATE_UNKNOWN, "error getting address for %s: %s\n",
+                   host, gai_strerror(ga_result));
        }
 
-       /* count te number of returned hosts, and allocate an array of sockets */
-       ai_tmp=ai;
-       while(ai_tmp){
-               ai_tmp = ai_tmp->ai_next;
-               num_hosts++;
-       }
+       /* count the number of returned hosts, and allocate stuff accordingly */
+       for(ai_tmp=ai; ai_tmp!=NULL; ai_tmp=ai_tmp->ai_next){ num_hosts++; }
+       req=(ntp_message*)malloc(sizeof(ntp_message)*num_hosts);
+       if(req==NULL) die(STATE_UNKNOWN, "can not allocate ntp message array");
        socklist=(int*)malloc(sizeof(int)*num_hosts);
        if(socklist==NULL) die(STATE_UNKNOWN, "can not allocate socket array");
+       ufds=(struct pollfd*)malloc(sizeof(struct pollfd)*num_hosts);
+       if(ufds==NULL) die(STATE_UNKNOWN, "can not allocate socket array");
+       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);
 
-       /* setup each socket for writing */
+       /* setup each socket for writing, and the corresponding struct pollfd */
        ai_tmp=ai;
        for(i=0;ai_tmp;i++){
                socklist[i]=socket(ai_tmp->ai_family, SOCK_DGRAM, IPPROTO_UDP);
@@ -337,37 +393,99 @@ double offset_request_ntpdate(const char *host){
                }
                if(connect(socklist[i], ai_tmp->ai_addr, ai_tmp->ai_addrlen)){
                        die(STATE_UNKNOWN, "can't create socket connection");
+               } else {
+                       ufds[i].fd=socklist[i];
+                       ufds[i].events=POLLIN;
+                       ufds[i].revents=0;
                }
                ai_tmp = ai_tmp->ai_next;
        }
 
-       /* now do AVG_NUM checks to each host. this needs to be optimized
-        * two ways:
-        *  - use some parellization w/poll for much faster results.  currently
-        *    we do send/recv, send/recv, etc, whereas we could use poll(), to
-        *    determine when to read and just do a bunch of writing when we
-        *    have free time.
-        *  - behave like ntpdate and only take the 5 best responses.
-        */
-       for(i=0; i<AVG_NUM; i++){
-               if(verbose) printf("offset calculation run %d/%d\n", i+1, AVG_NUM);
-               for(j=0; j<num_hosts; j++){
-                       if(verbose) printf("peer %d: ", j);
-                       setup_request(&req);
-                       write(socklist[j], &req, sizeof(ntp_message));
-                       read(socklist[j], &req, sizeof(ntp_message));
-                       gettimeofday(&recv_time, NULL);
-                       offset=calc_offset(&req, &recv_time);
-                       if(verbose) printf("offset: %g\n", offset);
-                       avg_offset+=offset;
+       /* 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,
+                * and update the "waiting" timestamp with the current time. */
+               one_written=0;
+               now_time=time(NULL);
+
+               for(i=0; i<num_hosts; i++){
+                       if(servers[i].waiting<now_time && servers[i].num_responses<AVG_NUM){
+                               if(verbose && servers[i].waiting != 0) printf("re-");
+                               if(verbose) printf("sending request to peer %d\n", i);
+                               setup_request(&req[i]);
+                               write(socklist[i], &req[i], sizeof(ntp_message));
+                               servers[i].waiting=now_time;
+                               one_written=1;
+                               break;
+                       }
+               }
+
+               /* quickly poll for any sockets with pending data */
+               servers_readable=poll(ufds, num_hosts, 100);
+               if(servers_readable==-1){
+                       perror("polling ntp sockets");
+                       die(STATE_UNKNOWN, "communication errors");
                }
-               avg_offset/=num_hosts;
+
+               /* read from any sockets with pending data */
+               for(i=0; servers_readable && i<num_hosts; i++){
+                       if(ufds[i].revents&POLLIN && servers[i].num_responses < AVG_NUM){
+                               if(verbose) {
+                                       printf("response from peer %d: ", i);
+                               }
+
+                               read(ufds[i].fd, &req[i], sizeof(ntp_message));
+                               gettimeofday(&recv_time, NULL);
+                               DBG(print_ntp_message(&req[i]));
+                               respnum=servers[i].num_responses++;
+                               servers[i].offset[respnum]=calc_offset(&req[i], &recv_time);
+                               if(verbose) {
+                                       printf("offset %.10g\n", servers[i].offset[respnum]);
+                               }
+                               servers[i].stratum=req[i].stratum;
+                               servers[i].rtdisp=NTP32asDOUBLE(req[i].rtdisp);
+                               servers[i].rtdelay=NTP32asDOUBLE(req[i].rtdelay);
+                               servers[i].waiting=0;
+                               servers_readable--;
+                               one_read = 1;
+                               if(servers[i].num_responses==AVG_NUM) servers_completed++;
+                       }
+               }
+               /* lather, rinse, repeat. */
+       }
+
+       if (one_read == 0) {
+               die(STATE_CRITICAL, "NTP CRITICAL: No response from NTP server\n");
        }
-       avg_offset/=AVG_NUM;
-       if(verbose) printf("overall average offset: %g\n", avg_offset);
 
-       for(j=0; j<num_hosts; j++){ close(socklist[j]); }
+       /* now, pick the best server from the list */
+       best_index=best_offset_server(servers, num_hosts);
+       if(best_index < 0){
+               *status=STATE_CRITICAL;
+       } else {
+               /* finally, calculate the average offset */
+               for(i=0; i<servers[best_index].num_responses;i++){
+                       avg_offset+=servers[best_index].offset[j];
+               }
+               avg_offset/=servers[best_index].num_responses;
+       }
+
+       /* 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]); } */
+       free(socklist);
+       free(ufds);
+       free(servers);
+       free(req);
        freeaddrinfo(ai);
+
+       if(verbose) printf("overall average offset: %.10g\n", avg_offset);
        return avg_offset;
 }
 
@@ -383,13 +501,16 @@ setup_control_request(ntp_control_message *p, uint8_t opcode, uint16_t seq){
 }
 
 /* XXX handle responses with the error bit set */
-double jitter_request(const char *host){
+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;
-       ntp_assoc_status_pair *peers;
+       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:
@@ -406,27 +527,31 @@ double jitter_request(const char *host){
         * 4) Extract the jitter value from the data[] (it's ASCII)
         */
        my_udp_connect(server_address, 123, &conn);
-       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
-        * Is it possible for an NTP server to have more than 117 synchronization
-        * sources?  If so, we will receive a second datagram with additional
-        * peers listed, since 117 is the maximum number that can fit in a
-        * single NTP control datagram.  This code doesn't handle that case */
-       /* XXX check the REM_MORE bit */
-       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.
-        */
-       npeers=ntohs(req.count)/sizeof(ntp_assoc_status_pair);
-       peers=(ntp_assoc_status_pair*)malloc(sizeof(ntp_assoc_status_pair)*npeers);
-       memcpy((void*)peers, (void*)req.data, sizeof(ntp_assoc_status_pair)*npeers);
+
+       /* 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. */
@@ -441,13 +566,18 @@ double jitter_request(const char *host){
        }
        if(verbose) printf("%d candiate peers available\n", num_candidates);
        if(verbose && syncsource_found) printf("synchronization source found\n");
-       /* XXX if ! syncsource_found set status to warning */
+       if(! syncsource_found){
+               *status = STATE_WARNING;
+               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
@@ -455,8 +585,10 @@ double jitter_request(const char *host){
                                 * thus reducing net traffic, guaranteeing us only a single
                                 * datagram in reply, and making intepretation much simpler
                                 */
-                               strncpy(req.data, "jitter", 6);
-                               req.count = htons(6);
+                               /* 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));
@@ -466,16 +598,26 @@ double jitter_request(const char *host){
                                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: ", peers[i].assoc);
+                                       printf("parsing jitter from peer %.2x: ", ntohs(peers[i].assoc));
                                }
-                               startofvalue = strchr(req.data, '=') + 1;
-                               jitter = strtod(startofvalue, &nptr);
-                               num_selected++;
-                               if(jitter == 0 && startofvalue==nptr){
-                                       printf("warning: unable to parse server response.\n");
-                                       /* XXX errors value ... */
+                               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_WARNING;
                                } else {
                                        if(verbose) printf("%g\n", jitter);
                                        num_valid++;
@@ -484,14 +626,14 @@ double jitter_request(const char *host){
                        }
                }
                if(verbose){
-                       printf("jitter parsed from %d/%d peers\n", num_selected, num_valid);
+                       printf("jitter parsed from %d/%d peers\n", num_valid, num_selected);
                }
        }
 
-       rval /= num_valid;
+       rval = num_valid ? rval / num_valid : -1.0;
 
        close(conn);
-       free(peers);
+       if(peers!=NULL) free(peers);
        /* If we return -1.0, it means no synchronization source was found */
        return rval;
 }
@@ -507,7 +649,6 @@ int process_arguments(int argc, char **argv){
                {"use-ipv6", no_argument, 0, '6'},
                {"warning", required_argument, 0, 'w'},
                {"critical", required_argument, 0, 'c'},
-               {"zero-offset", no_argument, 0, 'O'},
                {"jwarn", required_argument, 0, 'j'},
                {"jcrit", required_argument, 0, 'k'},
                {"timeout", required_argument, 0, 't'},
@@ -520,7 +661,7 @@ int process_arguments(int argc, char **argv){
                usage ("\n");
 
        while (1) {
-               c = getopt_long (argc, argv, "Vhv46w:c:Oj:k:t:H:", longopts, &option);
+               c = getopt_long (argc, argv, "Vhv46w:c:j:k:t:H:", longopts, &option);
                if (c == -1 || c == EOF || c == 1)
                        break;
 
@@ -558,9 +699,6 @@ int process_arguments(int argc, char **argv){
                case 't':
                        socket_timeout=atoi(optarg);
                        break;
-               case 'O':
-                       zero_offset_bad=1;
-                       break;
                case '4':
                        address_family = AF_INET;
                        break;
@@ -573,7 +711,7 @@ int process_arguments(int argc, char **argv){
                        break;
                case '?':
                        /* print short usage statement if args not parsable */
-                       usage2 (_("Unknown argument"), optarg);
+                       usage5 ();
                        break;
                }
        }
@@ -593,9 +731,28 @@ int process_arguments(int argc, char **argv){
        return 0;
 }
 
+char *perfd_offset (double offset)
+{
+       return fperfdata ("offset", offset, "s",
+               TRUE, owarn,
+               TRUE, ocrit,
+               FALSE, 0, FALSE, 0);
+}
+
+char *perfd_jitter (double jitter)
+{
+       return fperfdata ("jitter", jitter, "s",
+               do_jitter, jwarn,
+               do_jitter, jcrit,
+               TRUE, 0, FALSE, 0);
+}
+
 int main(int argc, char *argv[]){
-       int result = STATE_UNKNOWN;
+       int result, offset_result, jitter_result;
        double offset=0, jitter=0;
+       char *result_line, *perfdata_line;
+
+       result=offset_result=jitter_result=STATE_UNKNOWN;
 
        if (process_arguments (argc, argv) == ERROR)
                usage4 (_("Could not parse arguments"));
@@ -606,14 +763,15 @@ int main(int argc, char *argv[]){
        /* set socket timeout */
        alarm (socket_timeout);
 
-       offset = offset_request(server_address);
-       if(offset > ocrit){
+       offset = offset_request(server_address, &offset_result);
+       if(fabs(offset) > ocrit){
                result = STATE_CRITICAL;
-       } else if(offset > owarn) {
+       } else if(fabs(offset) > owarn) {
                result = STATE_WARNING;
        } else {
                result = STATE_OK;
        }
+       result=max_state(result, offset_result);
 
        /* If not told to check the jitter, we don't even send packets.
         * jitter is checked using NTP control packets, which not all
@@ -621,7 +779,7 @@ int main(int argc, char *argv[]){
         * (for example) will result in an error
         */
        if(do_jitter){
-               jitter=jitter_request(server_address);
+               jitter=jitter_request(server_address, &jitter_result);
                if(jitter > jcrit){
                        result = max_state(result, STATE_CRITICAL);
                } else if(jitter > jwarn) {
@@ -632,48 +790,72 @@ int main(int argc, char *argv[]){
                        result = STATE_UNKNOWN;
                }
        }
+       result=max_state(result, jitter_result);
 
        switch (result) {
                case STATE_CRITICAL :
-                       printf("NTP CRITICAL: ");
+                       asprintf(&result_line, "NTP CRITICAL:");
                        break;
                case STATE_WARNING :
-                       printf("NTP WARNING: ");
+                       asprintf(&result_line, "NTP WARNING:");
                        break;
                case STATE_OK :
-                       printf("NTP OK: ");
+                       asprintf(&result_line, "NTP OK:");
                        break;
                default :
-                       printf("NTP UNKNOWN: ");
+                       asprintf(&result_line, "NTP UNKNOWN:");
                        break;
        }
-
-       printf("Offset %g secs|offset=%g", offset, offset);
-       if (do_jitter) printf("|jitter=%f", jitter);
-       printf("\n");
+       if(offset_result==STATE_CRITICAL){
+               asprintf(&result_line, "%s %s", result_line, _("Offset unknown"));
+       } else {
+               if(offset_result==STATE_WARNING){
+                       asprintf(&result_line, "%s %s", result_line, _("Unable to fully sample sync server"));
+               }
+               asprintf(&result_line, "%s Offset %.10g secs", result_line, 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_usage(void){
-       printf("\
-Usage: %s -H <host> [-O] [-w <warn>] [-c <crit>] [-j <warn>] [-k <crit>] [-v verbose]\
-\n", progname);
-}
 
 void print_help(void){
        print_revision(progname, revision);
 
-       printf ("Copyright (c) 1999 Ethan Galstad\n");
+       printf ("Copyright (c) 2006 Sean Finney\n");
        printf (COPYRIGHT, copyright, email);
+  
+  printf ("%s\n", _("This plugin checks the selected ntp server"));
 
+  printf ("\n\n");
+  
        print_usage();
        printf (_(UT_HELP_VRSN));
        printf (_(UT_HOST_PORT), 'p', "123");
-       printf (_(UT_WARN_CRIT));
+       printf (" %s\n", "-w, --warning=DOUBLE");
+       printf ("    %s\n", _("Offset to result in warning status (seconds)"));
+       printf (" %s\n", "-c, --critical=DOUBLE");
+       printf ("    %s\n", _("Offset to result in critical status (seconds)"));
+       printf (" %s\n", "-j, --warning=DOUBLE");
+       printf ("    %s\n", _("Warning value for jitter"));
+       printf (" %s\n", "-k, --critical=DOUBLE");
+       printf ("    %s\n", _("Critical value for jitter"));
        printf (_(UT_TIMEOUT), DEFAULT_SOCKET_TIMEOUT);
        printf (_(UT_VERBOSE));
-       printf(_(UT_SUPPORT));
+       printf (_(UT_SUPPORT));
+}
+
+void
+print_usage(void)
+{
+  printf (_("Usage:"));
+  printf("%s -H <host> [-w <warn>] [-c <crit>] [-j <warn>] [-k <crit>] [-v verbose]\n", progname);
 }