[Nagiosplug-devel] [ nagiosplug-Patches-992583 ] regex fix for check_ntp

[SourceForge.net]

Patches item #992583, was opened at 2004-07-16 22:46
Message generated for change (Comment added) made by tonvoon
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=397599&aid=992583&group_id=29880

Category: Bugfix
Group: None
>Status: Closed
Resolution: None
Priority: 5
Submitted By: Nayden Naydenov (naidd)
>Assigned to: Ton Voon (tonvoon)
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

----------------------------------------------------------------------

>Comment By: Ton Voon (tonvoon)
Date: 2004-08-18 21:44

Message:
Logged In: YES 
user_id=664364

Nayden, Thanks for the report. This appears to be fixed in 1.18 of 
check_ntp.pl.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=397599&aid=992583&group_id=29880