From 930650812a19245a8d45853064e8fe32d87f71f2 Mon Sep 17 00:00:00 2001 From: Karl DeBisschop Date: Fri, 8 Aug 2003 16:49:32 +0000 Subject: [PATCH] cleanups from pedantic complier warnings git-svn-id: https://nagiosplug.svn.sourceforge.net/svnroot/nagiosplug/nagiosplug/trunk@666 f882894a-f735-0410-b71e-b25c423dba1c --- plugins/check_disk.c | 15 +++++++++++++-- plugins/netutils.c | 34 +++++++++++++++++----------------- plugins/netutils.h | 34 +++++++++++++++++----------------- 3 files changed, 47 insertions(+), 36 deletions(-) diff --git a/plugins/check_disk.c b/plugins/check_disk.c index f696672..bd06021 100644 --- a/plugins/check_disk.c +++ b/plugins/check_disk.c @@ -233,9 +233,10 @@ process_arguments (int argc, char **argv) struct name_list **fstail = &fs_exclude_list; struct name_list **dptail = &dp_exclude_list; struct name_list *temp_list; - unsigned long l; int result = OK; + unsigned long l; + int option_index = 0; static struct option long_options[] = { {"timeout", required_argument, 0, 't'}, @@ -322,6 +323,8 @@ process_arguments (int argc, char **argv) usage (_("Critical threshold must be integer or percentage!\n")); } case 'u': + if (units) + free(units); if (! strcmp (optarg, "bytes")) { mult = (uintmax_t)1; units = strdup ("B"); @@ -340,13 +343,19 @@ process_arguments (int argc, char **argv) } else { die (STATE_UNKNOWN, _("unit type %s not known\n"), optarg); } + if (units == NULL) + die (STATE_UNKNOWN, _("failed allocating storage for '%s'\n"), "units"); break; case 'k': /* display mountpoint */ mult = 1024; + if (units) + free(units); units = strdup ("kB"); break; case 'm': /* display mountpoint */ mult = 1024 * 1024; + if (units) + free(units); units = strdup ("kB"); break; case 'l': @@ -475,8 +484,10 @@ INPUT ERROR: C_DF (%lu) should be less than W_DF (%lu) and both should be greate return ERROR; } - if (units == NULL) + if (units == NULL) { units = strdup ("MB"); + mult = (uintmax_t)1024 * 1024; + } return OK; } diff --git a/plugins/netutils.c b/plugins/netutils.c index 92fd142..58b3fb4 100644 --- a/plugins/netutils.c +++ b/plugins/netutils.c @@ -53,8 +53,8 @@ socket_timeout_alarm_handler (int sig) /* connects to a host on a specified TCP port, sends a string, and gets a response */ int -process_tcp_request (char *server_address, int server_port, - char *send_buffer, char *recv_buffer, int recv_size) +process_tcp_request (const char *server_address, int server_port, + const char *send_buffer, char *recv_buffer, int recv_size) { int result; @@ -68,8 +68,8 @@ process_tcp_request (char *server_address, int server_port, /* connects to a host on a specified UDP port, sends a string, and gets a response */ int -process_udp_request (char *server_address, int server_port, - char *send_buffer, char *recv_buffer, int recv_size) +process_udp_request (const char *server_address, int server_port, + const char *send_buffer, char *recv_buffer, int recv_size) { int result; @@ -85,8 +85,8 @@ process_udp_request (char *server_address, int server_port, response. loops on select-recv until timeout or eof to get all of a multi-packet answer */ int -process_tcp_request2 (char *server_address, int server_port, - char *send_buffer, char *recv_buffer, int recv_size) +process_tcp_request2 (const char *server_address, int server_port, + const char *send_buffer, char *recv_buffer, int recv_size) { int result; @@ -163,8 +163,8 @@ process_tcp_request2 (char *server_address, int server_port, /* connects to a host on a specified port, sends a string, and gets a response */ int -process_request (char *server_address, int server_port, int proto, - char *send_buffer, char *recv_buffer, int recv_size) +process_request (const char *server_address, int server_port, int proto, + const char *send_buffer, char *recv_buffer, int recv_size) { int result; int send_result; @@ -223,7 +223,7 @@ process_request (char *server_address, int server_port, int proto, /* opens a connection to a remote host/tcp port */ int -my_tcp_connect (char *host_name, int port, int *sd) +my_tcp_connect (const char *host_name, int port, int *sd) { int result; @@ -235,7 +235,7 @@ my_tcp_connect (char *host_name, int port, int *sd) /* opens a connection to a remote host/udp port */ int -my_udp_connect (char *host_name, int port, int *sd) +my_udp_connect (const char *host_name, int port, int *sd) { int result; @@ -247,7 +247,7 @@ my_udp_connect (char *host_name, int port, int *sd) /* opens a tcp or udp connection to a remote host */ int -my_connect (char *host_name, int port, int *sd, int proto) +my_connect (const char *host_name, int port, int *sd, int proto) { struct addrinfo hints; struct addrinfo *res; @@ -315,7 +315,7 @@ my_connect (char *host_name, int port, int *sd, int proto) } int -is_host (char *address) +is_host (const char *address) { if (is_addr (address) || is_hostname (address)) return (TRUE); @@ -324,7 +324,7 @@ is_host (char *address) } int -is_addr (char *address) +is_addr (const char *address) { #ifdef USE_IPV6 if (is_inet_addr (address) && address_family != AF_INET6) @@ -342,7 +342,7 @@ is_addr (char *address) } int -resolve_host_or_addr (char *address, int family) +resolve_host_or_addr (const char *address, int family) { struct addrinfo hints; struct addrinfo *res; @@ -361,21 +361,21 @@ resolve_host_or_addr (char *address, int family) } int -is_inet_addr (char *address) +is_inet_addr (const char *address) { return resolve_host_or_addr (address, AF_INET); } #ifdef USE_IPV6 int -is_inet6_addr (char *address) +is_inet6_addr (const char *address) { return resolve_host_or_addr (address, AF_INET6); } #endif int -is_hostname (char *s1) +is_hostname (const char *s1) { #ifdef USE_IPV6 return resolve_host_or_addr (s1, address_family); diff --git a/plugins/netutils.h b/plugins/netutils.h index f5df7af..c560f44 100644 --- a/plugins/netutils.h +++ b/plugins/netutils.h @@ -36,27 +36,27 @@ RETSIGTYPE socket_timeout_alarm_handler (int) __attribute__((noreturn)); -int process_tcp_request2 (char *address, int port, char *sbuffer, - char *rbuffer, int rsize); -int process_tcp_request (char *address, int port, char *sbuffer, - char *rbuffer, int rsize); -int process_udp_request (char *address, int port, char *sbuffer, - char *rbuffer, int rsize); -int process_request (char *address, int port, int proto, char *sbuffer, - char *rbuffer, int rsize); +int process_tcp_request2 (const char *address, int port, + const char *sbuffer, char *rbuffer, int rsize); +int process_tcp_request (const char *address, int port, + const char *sbuffer, char *rbuffer, int rsize); +int process_udp_request (const char *address, int port, + const char *sbuffer, char *rbuffer, int rsize); +int process_request (const char *address, int port, int proto, + const char *sbuffer, char *rbuffer, int rsize); -int my_tcp_connect (char *address, int port, int *sd); -int my_udp_connect (char *address, int port, int *sd); -int my_connect (char *address, int port, int *sd, int proto); +int my_tcp_connect (const char *address, int port, int *sd); +int my_udp_connect (const char *address, int port, int *sd); +int my_connect (const char *address, int port, int *sd, int proto); -int is_host (char *); -int is_addr (char *); -int resolve_host_or_addr (char *, int); -int is_inet_addr (char *); +int is_host (const char *); +int is_addr (const char *); +int resolve_host_or_addr (const char *, int); +int is_inet_addr (const char *); #ifdef USE_IPV6 -int is_inet6_addr (char *); +int is_inet6_addr (const char *); #endif -int is_hostname (char *); +int is_hostname (const char *); extern unsigned int socket_timeout; extern int econn_refuse_state; -- 2.30.2