[Nagiosplug-devel] [ nagiosplug-Bugs-1396593 ] Perfdata Output by check_dns

[SourceForge.net]

Bugs item #1396593, was opened at 2006-01-04 15:06
Message generated for change (Comment added) made by mfavas
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=397597&aid=1396593&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: General plugin execution
Group: Release (specify)
Status: Open
Resolution: None
Priority: 5
Submitted By: Tobias Mucke (mucke)
Assigned to: Nobody/Anonymous (nobody)
Summary: Perfdata Output by check_dns

Initial Comment:
Hi everybody,

just a little bug in perfdata output of the plugin
check_dns. I'am using 1.4.2.

Example for dns reverse lookup:

/opt/nagios/plugins/check_dns -H IP -t 10
DNS OK: 0.004 seconds response time IP returns FQHN.
|time=0.003981s;;;0.000000

Between plugin output and perfdata output is a  	line
break, so perfdata will not get processed by Nagios.

If I call it like a dns forward lookup, there is no
line break.

/opt/nagios/plugins/check_dns -H FQHN -a IP -t 10
DNS OK: 0.005 seconds response time FQHN returns
IP|time=0.004883s;;;0.000000

By the way, it would be very nice to really test dns
forward lookups by check_dns. I tried it the following,
but this doesn't work, of course. Although it seems
strange that FQHN ist not FQHN.

/opt/nagios/plugins/check_dns -H IP -a FQHN -t 10
DNS CRITICAL - expected FQHN but got FQHN.

Thanks a lot.

Tobias


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

Comment By: Mark Favas (mfavas)
Date: 2006-01-15 10:56

Message:
Logged In: YES 
user_id=44979

Fixed linebreak issue:

diff -c check_dns.c.orig check_dns.c
*** check_dns.c.orig    Sun Jan 15 10:30:55 2006
--- check_dns.c Sun Jan 15 10:55:07 2006
***************
*** 173,178 ****
--- 173,179 ----

        /* If we got here, we should have an address string,
                 and we can segfault if we do not */
+       strip(address);
        if (address==NULL || strlen(address)==0)
                die (STATE_CRITICAL,
                     _("DNS CRITICAL - '%s' output parsing
exited with no address\n"),


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

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