[Nagiosplug-devel] [ nagiosplug-Bugs-1753506 ] check_ntp : typo

[SourceForge.net]

Bugs item #1753506, was opened at 2007-07-13 16:17
Message generated for change (Comment added) made by psychotrahe
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=397597&aid=1753506&group_id=29880

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Argument proccessing
Group: CVS
>Status: Closed
>Resolution: Accepted
Priority: 1
Private: No
Submitted By: Aurelien Bompard (abompard)
>Assigned to: Matthias Eble (psychotrahe)
Summary: check_ntp : typo

Initial Comment:
There's a typo in the argument checks in check_ntp. In the current CVS : 
http://nagiosplug.cvs.sourceforge.net/nagiosplug/nagiosplug/plugins/check_ntp.c?view=markup
line 723. There is a second check for ocrit < owarn, instead of checking for jcrit < jwarn. Here's the current code:
if (ocrit < owarn){
  usage4(_("Critical offset should be larger than warning offset"));
}
if (ocrit < owarn){
  usage4(_("Critical jitter should be larger than warning jitter"));
}

The second check should be jcrit < jwarn

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

>Comment By: Matthias Eble (psychotrahe)
Date: 2007-07-14 20:34

Message:
Logged In: YES 
user_id=1694341
Originator: NO

Hi Aurelien,

I've fixed this in cvs.
Thanks for your report.

Matthias

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

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