diff options
author | Thomas Guyot-Sionnest <dermoth@users.sourceforge.net> | 2007-12-04 12:24:48 +0000 |
---|---|---|
committer | Thomas Guyot-Sionnest <dermoth@users.sourceforge.net> | 2007-12-04 12:24:48 +0000 |
commit | 084a9b444fab26b756bc2bf751e93c5494b926f4 (patch) | |
tree | a7d19f36f0c8d5100d05677e9bc432ab6914db6c | |
parent | 79cc962eb8c17d849e7c698e25aed25938a0efae (diff) | |
download | monitoring-plugins-084a9b444fab26b756bc2bf751e93c5494b926f4.tar.gz |
Cleanups only
git-svn-id: https://nagiosplug.svn.sourceforge.net/svnroot/nagiosplug/nagiosplug/branches/dermoth_ntp_rework@1842 f882894a-f735-0410-b71e-b25c423dba1c
-rw-r--r-- | NEWS | 10 | ||||
-rw-r--r-- | plugins/check_ntp_peer.c | 32 | ||||
-rw-r--r-- | plugins/check_ntp_time.c | 18 |
3 files changed, 30 insertions, 30 deletions
@@ -9,7 +9,7 @@ This file documents the major additions and syntax changes between releases. | |||
9 | and the offset is unavailable, will return UNKNOWN as well. | 9 | and the offset is unavailable, will return UNKNOWN as well. |
10 | NOTE: If jitter thresholds are specified as integers it will return CRITICAL if jitter | 10 | NOTE: If jitter thresholds are specified as integers it will return CRITICAL if jitter |
11 | is "-1" as the default range starts at 0. See Examples in --help output. | 11 | is "-1" as the default range starts at 0. See Examples in --help output. |
12 | Merge two new checks that deprecates check_ntp: check_ntp_peer and check_ntp_time. | 12 | Merge two new checks that deprecates check_ntp: check_ntp_peer and check_ntp_time. |
13 | You should read the --help output so see which one is suitable for you. check_ntp_peer | 13 | You should read the --help output so see which one is suitable for you. check_ntp_peer |
14 | implement stratum thresholds support (feature request #1703823). | 14 | implement stratum thresholds support (feature request #1703823). |
15 | 15 | ||
@@ -17,9 +17,9 @@ This file documents the major additions and syntax changes between releases. | |||
17 | Fix check_http buffer overflow vulnerability when following HTTP redirects | 17 | Fix check_http buffer overflow vulnerability when following HTTP redirects |
18 | check_http now explicitly asks HTTP/1.1 servers to close the connection | 18 | check_http now explicitly asks HTTP/1.1 servers to close the connection |
19 | after completion of the response | 19 | after completion of the response |
20 | Check_ldaps' guessing which secure method to use (starttls vs. ssl on connect) | 20 | Check_ldaps' guessing which secure method to use (starttls vs. ssl on connect) |
21 | is now deprecated. See --help for further information. | 21 | is now deprecated. See --help for further information. |
22 | Check_disk now calls stat() on all filesystems to check. (Old: only the ones selected using -p) | 22 | Check_disk now calls stat() on all filesystems to check. (Old: only the ones selected using -p) |
23 | A meaningful error message (eg "Stale NFS Handle") is printed if stat fails. | 23 | A meaningful error message (eg "Stale NFS Handle") is printed if stat fails. |
24 | New check_disk option -L: Only check local filesystems, but call stat() on remote ones, too. | 24 | New check_disk option -L: Only check local filesystems, but call stat() on remote ones, too. |
25 | Thus accessibility of remote filesystems can be checked without any threshold comparison. | 25 | Thus accessibility of remote filesystems can be checked without any threshold comparison. |
@@ -38,7 +38,7 @@ This file documents the major additions and syntax changes between releases. | |||
38 | Fix possible check_icmp bus errors on some (non-x86/AMD64) platforms | 38 | Fix possible check_icmp bus errors on some (non-x86/AMD64) platforms |
39 | Fix check_smtp's handling of multiple-packet server responses | 39 | Fix check_smtp's handling of multiple-packet server responses |
40 | WARNING: Fix for negate which may break existing commands: | 40 | WARNING: Fix for negate which may break existing commands: |
41 | - stop evaluating command line options through shell twice | 41 | - stop evaluating command line options through shell twice |
42 | - enforce a full path for the command to run | 42 | - enforce a full path for the command to run |
43 | The "negate" utility can now remap custom states | 43 | The "negate" utility can now remap custom states |
44 | Check_radius now supports radiusclient-ng | 44 | Check_radius now supports radiusclient-ng |
@@ -113,7 +113,7 @@ This file documents the major additions and syntax changes between releases. | |||
113 | SSL plugins work with gnutls as well as OpenSSL | 113 | SSL plugins work with gnutls as well as OpenSSL |
114 | check_mysql_query added to run arbitrary SQL commands, with threshold checking | 114 | check_mysql_query added to run arbitrary SQL commands, with threshold checking |
115 | libtool now required for development systems | 115 | libtool now required for development systems |
116 | Notice: check_udp (compiled from check_udp.c) will be deprecated in the next release. | 116 | Notice: check_udp (compiled from check_udp.c) will be deprecated in the next release. |
117 | check_udp2 should be used instead and will be renamed to check_udp | 117 | check_udp2 should be used instead and will be renamed to check_udp |
118 | 118 | ||
119 | 1.4.2 | 119 | 1.4.2 |
diff --git a/plugins/check_ntp_peer.c b/plugins/check_ntp_peer.c index 83fa8aba..1c4702c8 100644 --- a/plugins/check_ntp_peer.c +++ b/plugins/check_ntp_peer.c | |||
@@ -181,7 +181,7 @@ char *extract_value(const char *varlist, const char *name){ | |||
181 | char *tmpvarlist=NULL, *tmpkey=NULL, *value=NULL; | 181 | char *tmpvarlist=NULL, *tmpkey=NULL, *value=NULL; |
182 | int last=0; | 182 | int last=0; |
183 | 183 | ||
184 | /* The following code require a non-empty varlist */ | 184 | /* The following code require a non-empty varlist */ |
185 | if(strlen(varlist) == 0) | 185 | if(strlen(varlist) == 0) |
186 | return NULL; | 186 | return NULL; |
187 | 187 | ||
@@ -279,7 +279,7 @@ int ntp_request(const char *host, double *offset, int *offset_result, double *ji | |||
279 | } while(req.op&REM_MORE); | 279 | } while(req.op&REM_MORE); |
280 | 280 | ||
281 | /* first, let's find out if we have a sync source, or if there are | 281 | /* first, let's find out if we have a sync source, or if there are |
282 | * at least some candidates. in the case of the latter we'll issue | 282 | * at least some candidates. In the latter case we'll issue |
283 | * a warning but go ahead with the check on them. */ | 283 | * a warning but go ahead with the check on them. */ |
284 | for (i = 0; i < npeers; i++){ | 284 | for (i = 0; i < npeers; i++){ |
285 | if (PEER_SEL(peers[i].status) >= PEER_INCLUDED){ | 285 | if (PEER_SEL(peers[i].status) >= PEER_INCLUDED){ |
@@ -302,7 +302,7 @@ int ntp_request(const char *host, double *offset, int *offset_result, double *ji | |||
302 | /* Only query this server if it is the current sync source */ | 302 | /* Only query this server if it is the current sync source */ |
303 | /* If there's no sync.peer, query all candidates and use the best one */ | 303 | /* If there's no sync.peer, query all candidates and use the best one */ |
304 | if (PEER_SEL(peers[i].status) >= min_peer_sel){ | 304 | if (PEER_SEL(peers[i].status) >= min_peer_sel){ |
305 | if(verbose) printf("Getting offset, jitter and stratum for peer %.2x\n", ntohs(peers[i].assoc)); | 305 | if(verbose) printf("Getting offset, jitter and stratum for peer %.2x\n", ntohs(peers[i].assoc)); |
306 | asprintf(&data, ""); | 306 | asprintf(&data, ""); |
307 | do{ | 307 | do{ |
308 | setup_control_request(&req, OP_READVAR, 2); | 308 | setup_control_request(&req, OP_READVAR, 2); |
@@ -335,7 +335,7 @@ int ntp_request(const char *host, double *offset, int *offset_result, double *ji | |||
335 | getvar = "stratum,offset,dispersion"; | 335 | getvar = "stratum,offset,dispersion"; |
336 | i--; | 336 | i--; |
337 | continue; | 337 | continue; |
338 | } else if(strlen(getvar)) { | 338 | } else if(strlen(getvar)) { |
339 | if(verbose) printf("Server didn't like dispersion either; will retrieve everything\n"); | 339 | if(verbose) printf("Server didn't like dispersion either; will retrieve everything\n"); |
340 | getvar = ""; | 340 | getvar = ""; |
341 | i--; | 341 | i--; |
@@ -541,7 +541,7 @@ char *perfd_stratum (int stratum) | |||
541 | 541 | ||
542 | int main(int argc, char *argv[]){ | 542 | int main(int argc, char *argv[]){ |
543 | int result, offset_result, stratum; | 543 | int result, offset_result, stratum; |
544 | double offset=0, jitter=0; | 544 | double offset=0, jitter=0; |
545 | char *result_line, *perfdata_line; | 545 | char *result_line, *perfdata_line; |
546 | 546 | ||
547 | if (process_arguments (argc, argv) == ERROR) | 547 | if (process_arguments (argc, argv) == ERROR) |
@@ -557,11 +557,11 @@ int main(int argc, char *argv[]){ | |||
557 | /* set socket timeout */ | 557 | /* set socket timeout */ |
558 | alarm (socket_timeout); | 558 | alarm (socket_timeout); |
559 | 559 | ||
560 | /* This returns either OK or WARNING (See comment preceeding ntp_request) */ | 560 | /* This returns either OK or WARNING (See comment preceeding ntp_request) */ |
561 | result = ntp_request(server_address, &offset, &offset_result, &jitter, &stratum); | 561 | result = ntp_request(server_address, &offset, &offset_result, &jitter, &stratum); |
562 | 562 | ||
563 | if(offset_result == STATE_UNKNOWN) { | 563 | if(offset_result == STATE_UNKNOWN) { |
564 | /* if there's no sync peer (this overrides ntp_request output): */ | 564 | /* if there's no sync peer (this overrides ntp_request output): */ |
565 | result = (quiet == 1 ? STATE_UNKNOWN : STATE_CRITICAL); | 565 | result = (quiet == 1 ? STATE_UNKNOWN : STATE_CRITICAL); |
566 | } else { | 566 | } else { |
567 | /* Be quiet if there's no candidates either */ | 567 | /* Be quiet if there's no candidates either */ |
@@ -602,11 +602,11 @@ int main(int argc, char *argv[]){ | |||
602 | } | 602 | } |
603 | if (do_jitter) { | 603 | if (do_jitter) { |
604 | asprintf(&result_line, "%s, jitter=%f", result_line, jitter); | 604 | asprintf(&result_line, "%s, jitter=%f", result_line, jitter); |
605 | asprintf(&perfdata_line, "%s %s", perfdata_line, perfd_jitter(jitter)); | 605 | asprintf(&perfdata_line, "%s %s", perfdata_line, perfd_jitter(jitter)); |
606 | } | 606 | } |
607 | if (do_stratum) { | 607 | if (do_stratum) { |
608 | asprintf(&result_line, "%s, stratum=%i", result_line, stratum); | 608 | asprintf(&result_line, "%s, stratum=%i", result_line, stratum); |
609 | asprintf(&perfdata_line, "%s %s", perfdata_line, perfd_stratum(stratum)); | 609 | asprintf(&perfdata_line, "%s %s", perfdata_line, perfd_stratum(stratum)); |
610 | } | 610 | } |
611 | printf("%s|%s\n", result_line, perfdata_line); | 611 | printf("%s|%s\n", result_line, perfdata_line); |
612 | 612 | ||
@@ -621,11 +621,11 @@ void print_help(void){ | |||
621 | 621 | ||
622 | printf ("Copyright (c) 2006 Sean Finney\n"); | 622 | printf ("Copyright (c) 2006 Sean Finney\n"); |
623 | printf (COPYRIGHT, copyright, email); | 623 | printf (COPYRIGHT, copyright, email); |
624 | |||
625 | printf ("%s\n", _("This plugin checks the selected ntp server")); | ||
626 | 624 | ||
627 | printf ("\n\n"); | 625 | printf ("%s\n", _("This plugin checks the selected ntp server")); |
628 | 626 | ||
627 | printf ("\n\n"); | ||
628 | |||
629 | print_usage(); | 629 | print_usage(); |
630 | printf (_(UT_HELP_VRSN)); | 630 | printf (_(UT_HELP_VRSN)); |
631 | printf (_(UT_HOST_PORT), 'p', "123"); | 631 | printf (_(UT_HOST_PORT), 'p', "123"); |
@@ -675,7 +675,7 @@ void print_help(void){ | |||
675 | void | 675 | void |
676 | print_usage(void) | 676 | print_usage(void) |
677 | { | 677 | { |
678 | printf (_("Usage:")); | 678 | printf (_("Usage:")); |
679 | printf(" %s -H <host> [-w <warn>] [-c <crit>] [-W <warn>] [-C <crit>]\n", progname); | 679 | printf(" %s -H <host> [-w <warn>] [-c <crit>] [-W <warn>] [-C <crit>]\n", progname); |
680 | printf(" [-j <warn>] [-k <crit>] [-v verbose]\n"); | 680 | printf(" [-j <warn>] [-k <crit>] [-v verbose]\n"); |
681 | } | 681 | } |
diff --git a/plugins/check_ntp_time.c b/plugins/check_ntp_time.c index 042d3037..ad7f6c4b 100644 --- a/plugins/check_ntp_time.c +++ b/plugins/check_ntp_time.c | |||
@@ -356,13 +356,13 @@ double offset_request(const char *host, int *status){ | |||
356 | ai_tmp = ai_tmp->ai_next; | 356 | ai_tmp = ai_tmp->ai_next; |
357 | } | 357 | } |
358 | 358 | ||
359 | /* now do AVG_NUM checks to each host. we stop before timeout/2 seconds | 359 | /* now do AVG_NUM checks to each host. We stop before timeout/2 seconds |
360 | * have passed in order to ensure post-processing and jitter time. */ | 360 | * have passed in order to ensure post-processing and jitter time. */ |
361 | now_time=start_ts=time(NULL); | 361 | now_time=start_ts=time(NULL); |
362 | while(servers_completed<num_hosts && now_time-start_ts <= socket_timeout/2){ | 362 | while(servers_completed<num_hosts && now_time-start_ts <= socket_timeout/2){ |
363 | /* loop through each server and find each one which hasn't | 363 | /* loop through each server and find each one which hasn't |
364 | * been touched in the past second or so and is still lacking | 364 | * been touched in the past second or so and is still lacking |
365 | * some responses. for each of these servers, send a new request, | 365 | * some responses. For each of these servers, send a new request, |
366 | * and update the "waiting" timestamp with the current time. */ | 366 | * and update the "waiting" timestamp with the current time. */ |
367 | one_written=0; | 367 | one_written=0; |
368 | now_time=time(NULL); | 368 | now_time=time(NULL); |
@@ -586,11 +586,11 @@ void print_help(void){ | |||
586 | 586 | ||
587 | printf ("Copyright (c) 2006 Sean Finney\n"); | 587 | printf ("Copyright (c) 2006 Sean Finney\n"); |
588 | printf (COPYRIGHT, copyright, email); | 588 | printf (COPYRIGHT, copyright, email); |
589 | |||
590 | printf ("%s\n", _("This plugin checks the clock offset with the ntp server")); | ||
591 | 589 | ||
592 | printf ("\n\n"); | 590 | printf ("%s\n", _("This plugin checks the clock offset with the ntp server")); |
593 | 591 | ||
592 | printf ("\n\n"); | ||
593 | |||
594 | print_usage(); | 594 | print_usage(); |
595 | printf (_(UT_HELP_VRSN)); | 595 | printf (_(UT_HELP_VRSN)); |
596 | printf (_(UT_HOST_PORT), 'p', "123"); | 596 | printf (_(UT_HOST_PORT), 'p', "123"); |
@@ -625,8 +625,8 @@ void print_help(void){ | |||
625 | void | 625 | void |
626 | print_usage(void) | 626 | print_usage(void) |
627 | { | 627 | { |
628 | printf (_("Usage:")); | 628 | printf (_("Usage:")); |
629 | printf(" %s -H <host> [-w <warn>] [-c <crit>] [-W <warn>] [-C <crit>]\n", progname); | 629 | printf(" %s -H <host> [-w <warn>] [-c <crit>] [-W <warn>] [-C <crit>]\n", progname); |
630 | printf(" [-j <warn>] [-k <crit>] [-v verbose]\n"); | 630 | printf(" [-j <warn>] [-k <crit>] [-v verbose]\n"); |
631 | } | 631 | } |
632 | 632 | ||