[Nagiosplug-devel] [ nagiosplug-Patches-992583 ] regex fix for check_ntp
SourceForge.net
noreply at sourceforge.net
Fri Jul 16 14:47:02 CEST 2004
Patches item #992583, was opened at 2004-07-16 17:46
Message generated for change (Tracker Item Submitted) made by Item Submitter
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=397599&aid=992583&group_id=29880
Category: Bugfix
Group: None
Status: Open
Resolution: None
Priority: 5
Submitted By: Nayden Naydenov (naidd)
Assigned to: Nobody/Anonymous (nobody)
Summary: regex fix for check_ntp
Initial Comment:
The check_ntp plugin included with nagios-plugins-1.3.1
has a problem parsing the output from various ntp
servers. Here is an example:
./check_ntp -H clock
Use of uninitialized value in concatenation (.) or
string at ./123 line 308.
NTP UNKNOWN: Jitter too high
And here is the verbose output:
./check_ntp -H clock -v
server 192.168.2.222, stratum 1, offset -0.000741,
delay 0.03159
ntperr = 0
16 Jul 17:36:37 ntpdate[12099]: adjust time server
192.168.2.222 offset -0.000741 sec
ntperr = 0
remote refid st t when poll reach
delay offset jitter
==============================================================================
*127.127.1.0 .GPS. 0 - 11 64 377
0.000 0.000 0.000
Candiate count= 1
192.168.2.255 0.0.0.0 16 u - 64 0
0.000 0.000 0.000
192.168.3.255 0.0.0.0 16 u - 64 0
0.000 0.000 0.000
192.168.6.255 0.0.0.0 16 u - 64 0
0.000 0.000 0.000
192.168.9.255 0.0.0.0 16 u - 64 0
0.000 0.000 0.000
Use of uninitialized value in concatenation (.) or
string at ./123 line 308.
NTP UNKNOWN: Jitter too high
As you can see, the "t" (peer type) field in the first
table row is "-", and the script is only expecting to
see "l", "u", "m", or "b". Changing the following
regular expression (line 265) fixes the problem (notice
the extra "-"):
BEFORE:
# match sys.peer or pps.peer
if
(/^(\*|o)([-0-9.\s]+)\s+([-0-9A-Za-z.]+)\s+([-0-9.]+)\s+([lumb]+)\s+([-0-9m.]+)\s+([-0-9.]+)\s+([-0-9.]+)\s+([-0-9.]+)\s+([-0-9.]+)\s+([-0-9.]+)/)
{
AFTER:
# match sys.peer or pps.peer
if
(/^(\*|o)([-0-9.\s]+)\s+([-0-9A-Za-z.]+)\s+([-0-9.]+)\s+([lumb-]+)\s+([-0-9m.]+)\s+([-0-9.]+)\s+([-0-9.]+)\s+([-0-9.]+)\s+([-0-9.]+)\s+([-0-9.]+)/)
{
Finally, here is the brief output after the change:
./check_ntp -H clock
NTP OK: Offset -0.000771 secs, jitter 0.000 msec, peer
is stratum 0
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=397599&aid=992583&group_id=29880
More information about the Devel
mailing list