[Nagiosplug-devel] check_tcp and local sockets
sean finney
seanius at seanius.net
Sun Oct 30 02:28:00 CEST 2005
hey alex,
On Sat, Oct 29, 2005 at 09:24:04PM +0300, Alex Samorukov wrote:
> Big thanks to Seanius for looking into my patch. I tried latest CVS and
> found some errors on it.
> This small error make check_tcp completely unusable (core on startup).
> Here is a correction:
> - else if (strncmp(SERVICE, "CLAMD", 5)) {
> + else if (!strncmp(SERVICE, "CLAMD", 5)) {
d'oh. that's what i get for manually merging in the patch :)
> - else if (is_host (optarg) == FALSE && optarg[0] != '/')
> + else if (is_host (server_address) == FALSE && server_address[0]
> != '/')
yeah. and that's what i get for making a code optimization without
double checking myself. anyway, both of these are fixed in cvs now.
on a related note, i believe that providing this feature also provides
a much better/thorough way to test the tcp-based plugins. i don't know
much about how the test suite works, but it wouldn't be too hard to
write a test program that creates a unix domain socket and spoke
the proper things to the plugins...
sean
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://nagios-plugins.org/archive/devel/attachments/20051030/59f74b81/attachment.sig>
More information about the Devel
mailing list