code cleanup to clear strict compiler warnings

git-svn-id: https://nagiosplug.svn.sourceforge.net/svnroot/nagiosplug/nagiosplug/trunk@495 f882894a-f735-0410-b71e-b25c423dba1c
This commit is contained in:
Karl DeBisschop 2003-04-17 05:36:20 +00:00
parent e01c91d3a0
commit 614071738f
3 changed files with 158 additions and 151 deletions

View file

@ -1,37 +1,49 @@
/******************************************************************************
*
* Program: SNMP plugin for Nagios
* License: GPL
*
* License Information:
*
* This program is free software; you can redistribute it and/or modify
* it under the terms of the GNU General Public License as published by
* the Free Software Foundation; either version 2 of the License, or
* (at your option) any later version.
*
* This program is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
* GNU General Public License for more details.
*
* You should have received a copy of the GNU General Public License
* along with this program; if not, write to the Free Software
* Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
*****************************************************************************/
/*****************************************************************************
*
* This program is free software; you can redistribute it and/or modify
* it under the terms of the GNU General Public License as published by
* the Free Software Foundation; either version 2 of the License, or
* (at your option) any later version.
*
* This program is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
* GNU General Public License for more details.
*
* You should have received a copy of the GNU General Public License
* along with this program; if not, write to the Free Software
* Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
*
*****************************************************************************/
const char *progname = "check_dig";
const char *revision = "$Revision$";
const char *copyright = "2002-2003";
const char *authors = "Nagios Plugin Development Team";
const char *email = "nagiosplug-devel@lists.sourceforge.net";
const char *summary = "Test the DNS service on the specified host using dig\n";
const char *option_summary = "-H host -l lookup [-t timeout] [-v]";
const char *options = "\
-H, --hostname=STRING or IPADDRESS\n\
Check server on the indicated host\n\
-l, --lookup=STRING\n\
machine name to lookup\n\
-t, --timeout=INTEGER\n\
Seconds before connection attempt times out (default: %d)\n\
-v, --verbose\n\
Print extra information (command-line use only)\n\
-h, --help\n\
Print detailed help screen\n\
-V, --version\n\
Print version information\n\n";
#include "config.h"
#include "common.h"
#include "utils.h"
#include "popen.h"
const char *progname = "check_dig";
#define REVISION "$Revision$"
#define COPYRIGHT "2000-2002"
#define AUTHOR "Karl DeBisschop"
#define EMAIL "karl@debisschop.net"
#define SUMMARY "Test the DNS service on the specified host using dig\n"
int process_arguments (int, char **);
int validate_arguments (void);
void print_help (void);
@ -210,7 +222,7 @@ process_arguments (int argc, char **argv)
}
}
else {
asprintf (&dns_server, "127.0.0.1");
dns_server = strdup ("127.0.0.1");
}
}
@ -234,25 +246,13 @@ validate_arguments (void)
void
print_help (void)
{
print_revision (progname, "$Revision$");
print_revision (progname, revision);
printf
("Copyright (c) %s %s <%s>\n\n%s\n",
COPYRIGHT, AUTHOR, EMAIL, SUMMARY);
copyright, authors, email, summary);
print_usage ();
printf
("\nOptions:\n"
" -H, --hostname=STRING or IPADDRESS\n"
" Check server on the indicated host\n"
" -l, --lookup=STRING\n"
" machine name to lookup\n"
" -t, --timeout=INTEGER\n"
" Seconds before connection attempt times out (default: %d)\n"
" -v, --verbose\n"
" Print extra information (command-line use only)\n"
" -h, --help\n"
" Print detailed help screen\n"
" -V, --version\n"
" Print version information\n\n", DEFAULT_SOCKET_TIMEOUT);
printf ("\nOptions:\n");
printf (options, DEFAULT_SOCKET_TIMEOUT);
support ();
}
@ -263,8 +263,7 @@ print_help (void)
void
print_usage (void)
{
printf
("Usage: %s -H host -l lookup [-t timeout] [-v]\n"
" %s --help\n"
" %s --version\n", progname, progname, progname);
printf ("Usage: %s %s\n", progname, option_summary);
printf (" %s (-h|--help)\n", progname);
printf (" %s (-V|--version)\n", progname);
}

View file

@ -1,4 +1,4 @@
/******************************************************************************
/*****************************************************************************
*
* This program is free software; you can redistribute it and/or modify
* it under the terms of the GNU General Public License as published by
@ -51,7 +51,9 @@ const char *options = "\
-r, --refuse=ok|warn|crit\n\
Accept tcp refusals with states ok, warn, crit (default: crit)\n\
-v\n\
Show details for command-line debugging (do not use with nagios server)\n\
Show details for command-line debugging (do not use with nagios server)\n";
const char *standard_options = "\
-h, --help\n\
Print detailed help screen\n\
-V, --version\n\
@ -63,21 +65,21 @@ const char *options = "\
#include "utils.h"
#ifdef HAVE_SSL_H
#include <rsa.h>
#include <crypto.h>
#include <x509.h>
#include <pem.h>
#include <ssl.h>
#include <err.h>
#endif
#ifdef HAVE_OPENSSL_SSL_H
#include <openssl/rsa.h>
#include <openssl/crypto.h>
#include <openssl/x509.h>
#include <openssl/pem.h>
#include <openssl/ssl.h>
#include <openssl/err.h>
# include <rsa.h>
# include <crypto.h>
# include <x509.h>
# include <pem.h>
# include <ssl.h>
# include <err.h>
#else
# ifdef HAVE_OPENSSL_SSL_H
# include <openssl/rsa.h>
# include <openssl/crypto.h>
# include <openssl/x509.h>
# include <openssl/pem.h>
# include <openssl/ssl.h>
# include <openssl/err.h>
# endif
#endif
#ifdef HAVE_SSL
@ -134,8 +136,8 @@ main (int argc, char **argv)
struct timeval tv;
if (strstr (argv[0], "check_udp")) {
asprintf (&progname, "check_udp");
asprintf (&SERVICE, "UDP");
progname = strdup ("check_udp");
SERVICE = strdup ("UDP");
SEND = NULL;
EXPECT = NULL;
QUIT = NULL;
@ -143,8 +145,8 @@ main (int argc, char **argv)
PORT = 0;
}
else if (strstr (argv[0], "check_tcp")) {
asprintf (&progname, "check_tcp");
asprintf (&SERVICE, "TCP");
progname = strdup ("check_tcp");
SERVICE = strdup ("TCP");
SEND = NULL;
EXPECT = NULL;
QUIT = NULL;
@ -152,66 +154,66 @@ main (int argc, char **argv)
PORT = 0;
}
else if (strstr (argv[0], "check_ftp")) {
asprintf (&progname, "check_ftp");
asprintf (&SERVICE, "FTP");
progname = strdup ("check_ftp");
SERVICE = strdup ("FTP");
SEND = NULL;
asprintf (&EXPECT, "220");
asprintf (&QUIT, "QUIT\r\n");
EXPECT = strdup ("220");
QUIT = strdup ("QUIT\r\n");
PROTOCOL = TCP_PROTOCOL;
PORT = 21;
}
else if (strstr (argv[0], "check_smtp")) {
asprintf (&progname, "check_smtp");
asprintf (&SERVICE, "SMTP");
progname = strdup ("check_smtp");
SERVICE = strdup ("SMTP");
SEND = NULL;
asprintf (&EXPECT, "220");
asprintf (&QUIT, "QUIT\r\n");
EXPECT = strdup ("220");
QUIT = strdup ("QUIT\r\n");
PROTOCOL = TCP_PROTOCOL;
PORT = 25;
}
else if (strstr (argv[0], "check_pop")) {
asprintf (&progname, "check_pop");
asprintf (&SERVICE, "POP");
progname = strdup ("check_pop");
SERVICE = strdup ("POP");
SEND = NULL;
asprintf (&EXPECT, "+OK");
asprintf (&QUIT, "QUIT\r\n");
EXPECT = strdup ("+OK");
QUIT = strdup ("QUIT\r\n");
PROTOCOL = TCP_PROTOCOL;
PORT = 110;
}
else if (strstr (argv[0], "check_imap")) {
asprintf (&progname, "check_imap");
asprintf (&SERVICE, "IMAP");
progname = strdup ("check_imap");
SERVICE = strdup ("IMAP");
SEND = NULL;
asprintf (&EXPECT, "* OK");
asprintf (&QUIT, "a1 LOGOUT\r\n");
EXPECT = strdup ("* OK");
QUIT = strdup ("a1 LOGOUT\r\n");
PROTOCOL = TCP_PROTOCOL;
PORT = 143;
}
#ifdef HAVE_SSL
else if (strstr(argv[0],"check_simap")) {
asprintf (&progname, "check_simap");
asprintf (&SERVICE, "SIMAP");
progname = strdup ("check_simap");
SERVICE = strdup ("SIMAP");
SEND=NULL;
asprintf (&EXPECT, "* OK");
asprintf (&QUIT, "a1 LOGOUT\r\n");
EXPECT = strdup ("* OK");
QUIT = strdup ("a1 LOGOUT\r\n");
PROTOCOL=TCP_PROTOCOL;
use_ssl=TRUE;
PORT=993;
}
else if (strstr(argv[0],"check_spop")) {
asprintf (&progname, "check_spop");
asprintf (&SERVICE, "SPOP");
progname = strdup ("check_spop");
SERVICE = strdup ("SPOP");
SEND=NULL;
asprintf (&EXPECT, "+OK");
asprintf (&QUIT, "QUIT\r\n");
EXPECT = strdup ("+OK");
QUIT = strdup ("QUIT\r\n");
PROTOCOL=TCP_PROTOCOL;
use_ssl=TRUE;
PORT=995;
}
#endif
else if (strstr (argv[0], "check_nntp")) {
asprintf (&progname, "check_nntp");
asprintf (&SERVICE, "NNTP");
progname = strdup ("check_nntp");
SERVICE = strdup ("NNTP");
SEND = NULL;
EXPECT = NULL;
server_expect = realloc (server_expect, ++server_expect_count);
@ -226,7 +228,7 @@ main (int argc, char **argv)
usage ("ERROR: Generic check_tcp called with unknown service\n");
}
asprintf (&server_address, "127.0.0.1");
server_address = strdup ("127.0.0.1");
server_port = PORT;
server_send = SEND;
server_quit = QUIT;
@ -312,17 +314,21 @@ main (int argc, char **argv)
}
}
if (server_quit != NULL)
if (server_quit != NULL) {
#ifdef HAVE_SSL
if (use_ssl) {
SSL_write (ssl, QUIT, strlen (QUIT));
SSL_write (ssl, server_quit, strlen (server_quit));
SSL_shutdown (ssl);
SSL_free (ssl);
SSL_CTX_free (ctx);
}
else
else {
#endif
send (sd, server_quit, strlen (server_quit), 0);
send (sd, server_quit, strlen (server_quit), 0);
#ifdef HAVE_SSL
}
#endif
}
/* close the connection */
if (sd)
@ -511,23 +517,23 @@ process_arguments (int argc, char **argv)
void
print_help (void)
{
print_revision (progname, revision);
printf ("Copyright (c) %s %s\n\t<%s>\n\n",
copyright, authors, email);
print_revision (progname, revision);
printf ("Copyright (c) %s %s\n\t<%s>\n\n",
copyright, authors, email);
printf (summary, SERVICE);
print_usage ();
printf ("\nOptions:\n");
printf (options, DEFAULT_SOCKET_TIMEOUT);
support ();
print_usage ();
printf ("\nOptions:\n");
printf (options, DEFAULT_SOCKET_TIMEOUT);
printf (standard_options);
support ();
}
void
print_usage (void)
{
printf
("Usage: %s %s\n"
" %s (-h|--help)\n"
" %s (-V|--version)\n", progname, option_summary, progname, progname);
printf ("Usage: %s %s\n", progname, option_summary);
printf (" %s (-h|--help)\n", progname);
printf (" %s (-V|--version)\n", progname);
}
#ifdef HAVE_SSL

View file

@ -40,8 +40,10 @@ int was_refused = FALSE;
void
socket_timeout_alarm_handler (int sig)
{
printf ("CRITICAL - Socket timeout after %d seconds\n", socket_timeout);
if (sig == SIGALRM)
printf ("CRITICAL - Socket timeout after %d seconds\n", socket_timeout);
else
printf ("CRITICAL - Abnormal timeout after %d seconds\n", socket_timeout);
exit (STATE_CRITICAL);
}
@ -99,7 +101,7 @@ process_tcp_request2 (char *server_address, int server_port,
return STATE_CRITICAL;
send_result = send (sd, send_buffer, strlen (send_buffer), 0);
if (send_result != strlen (send_buffer)) {
if (send_result<0 || (size_t)send_result!=strlen(send_buffer)) {
printf ("send() failed\n");
result = STATE_WARNING;
}
@ -177,7 +179,7 @@ process_request (char *server_address, int server_port, int proto,
return STATE_CRITICAL;
send_result = send (sd, send_buffer, strlen (send_buffer), 0);
if (send_result != strlen (send_buffer)) {
if (send_result<0 || (size_t)send_result!=strlen(send_buffer)) {
printf ("send() failed\n");
result = STATE_WARNING;
}
@ -248,7 +250,6 @@ my_connect (char *host_name, int port, int *sd, int proto)
{
struct addrinfo hints;
struct addrinfo *res;
struct addrinfo *ptrp;
char port_str[6];
int result;
@ -267,7 +268,7 @@ my_connect (char *host_name, int port, int *sd, int proto)
while (res) {
/* attempt to create a socket */
*sd = socket (res->ai_family, (proto == IPPROTO_UDP) ?
SOCK_DGRAM : SOCK_STREAM, res->ai_protocol);
SOCK_DGRAM : SOCK_STREAM, res->ai_protocol);
if (*sd < 0) {
printf ("Socket creation failed\n");
@ -285,13 +286,13 @@ my_connect (char *host_name, int port, int *sd, int proto)
if (result < 0) {
switch (errno) {
case ECONNREFUSED:
switch (econn_refuse_state) {
case STATE_OK:
case STATE_WARNING:
was_refused = TRUE;
}
break;
case ECONNREFUSED:
switch (econn_refuse_state) {
case STATE_OK:
case STATE_WARNING:
was_refused = TRUE;
}
break;
}
}
@ -314,55 +315,56 @@ my_connect (char *host_name, int port, int *sd, int proto)
int
is_host (char *address)
{
if (is_addr (address) || is_hostname (address))
return (TRUE);
if (is_addr (address) || is_hostname (address))
return (TRUE);
return (FALSE);
return (FALSE);
}
int
is_addr (char *address)
{
#ifdef USE_IPV6
if (is_inet_addr (address) || is_inet6_addr (address))
#else
if (is_inet_addr (address))
#endif
return (TRUE);
if (is_inet_addr (address))
return (TRUE);
return (FALSE);
#ifdef USE_IPV6
if (is_inet6_addr (address))
return (TRUE);
#endif
return (FALSE);
}
int
resolve_host_or_addr (char *address, int family)
{
struct addrinfo hints;
struct addrinfo *res;
int retval;
struct addrinfo hints;
struct addrinfo *res;
int retval;
memset (&hints, 0, sizeof (hints));
hints.ai_family = family;
retval = getaddrinfo (address, NULL, &hints, &res);
memset (&hints, 0, sizeof (hints));
hints.ai_family = family;
retval = getaddrinfo (address, NULL, &hints, &res);
if (retval != 0)
return FALSE;
else {
freeaddrinfo (res);
return TRUE;
}
if (retval != 0)
return FALSE;
else {
freeaddrinfo (res);
return TRUE;
}
}
int
is_inet_addr (char *address)
{
return resolve_host_or_addr (address, AF_INET);
return resolve_host_or_addr (address, AF_INET);
}
#ifdef USE_IPV6
int
is_inet6_addr (char *address)
{
return resolve_host_or_addr (address, AF_INET6);
return resolve_host_or_addr (address, AF_INET6);
}
#endif
@ -370,9 +372,9 @@ int
is_hostname (char *s1)
{
#ifdef USE_IPV6
return resolve_host_or_addr (s1, AF_UNSPEC);
return resolve_host_or_addr (s1, AF_UNSPEC);
#else
return resolve_host_or_addr (s1, AF_INET);
return resolve_host_or_addr (s1, AF_INET);
#endif
}