[monitoring-plugins] Use real booleans
RincewindsHat
git at monitoring-plugins.org
Sun Oct 1 23:30:12 CEST 2023
Module: monitoring-plugins
Branch: master
Commit: 1aaa2385031da8de6c9d74cecf589482a56b9e35
Author: RincewindsHat <12514511+RincewindsHat at users.noreply.github.com>
Date: Sun Oct 1 13:59:00 2023 +0200
URL: https://www.monitoring-plugins.org/repositories/monitoring-plugins/commit/?id=1aaa238
Use real booleans
---
plugins-root/check_dhcp.c | 31 ++++++++++++++-----------------
1 file changed, 14 insertions(+), 17 deletions(-)
diff --git a/plugins-root/check_dhcp.c b/plugins-root/check_dhcp.c
index 99df3d2..0c18d5f 100644
--- a/plugins-root/check_dhcp.c
+++ b/plugins-root/check_dhcp.c
@@ -115,9 +115,6 @@ long mac_addr_dlpi( const char *, int, u_char *);
#define OK 0
#define ERROR -1
-#define FALSE 0
-#define TRUE 1
-
/**** DHCP definitions ****/
@@ -158,7 +155,7 @@ typedef struct dhcp_offer_struct{
typedef struct requested_server_struct{
struct in_addr server_address;
- int answered;
+ bool answered;
struct requested_server_struct *next;
}requested_server;
@@ -217,8 +214,8 @@ int valid_responses=0; /* number of valid DHCPOFFERs we received */
int requested_servers=0;
int requested_responses=0;
-int request_specific_address=FALSE;
-int received_requested_address=FALSE;
+bool request_specific_address=false;
+bool received_requested_address=false;
int verbose=0;
struct in_addr requested_address;
@@ -491,7 +488,7 @@ int send_dhcp_discover(int sock){
discover_packet.options[opts++]=DHCPDISCOVER;
/* the IP address we're requesting */
- if(request_specific_address==TRUE){
+ if(request_specific_address){
discover_packet.options[opts++]=DHCP_OPTION_REQUESTED_ADDRESS;
discover_packet.options[opts++]='\x04';
memcpy(&discover_packet.options[opts],&requested_address,sizeof(requested_address));
@@ -792,7 +789,7 @@ int add_requested_server(struct in_addr server_address){
return ERROR;
new_server->server_address=server_address;
- new_server->answered=FALSE;
+ new_server->answered=false;
new_server->next=requested_server_list;
requested_server_list=new_server;
@@ -946,7 +943,7 @@ int get_results(void){
int result;
uint32_t max_lease_time=0;
- received_requested_address=FALSE;
+ received_requested_address=false;
/* checks responses from requested servers */
requested_responses=0;
@@ -962,7 +959,7 @@ int get_results(void){
/* see if we got the address we requested */
if(!memcmp(&requested_address,&temp_offer->offered_address,sizeof(requested_address)))
- received_requested_address=TRUE;
+ received_requested_address=true;
/* see if the servers we wanted a response from talked to us or not */
if(!memcmp(&temp_offer->server_address,&temp_server->server_address,sizeof(temp_server->server_address))){
@@ -973,9 +970,9 @@ int get_results(void){
printf(_(" (duplicate)"));
printf(_("\n"));
}
- if(temp_server->answered == FALSE){
+ if(!temp_server->answered){
requested_responses++;
- temp_server->answered=TRUE;
+ temp_server->answered=true;
}
}
}
@@ -994,7 +991,7 @@ int get_results(void){
/* see if we got the address we requested */
if(!memcmp(&requested_address,&temp_offer->offered_address,sizeof(requested_address)))
- received_requested_address=TRUE;
+ received_requested_address=true;
}
}
@@ -1005,7 +1002,7 @@ int get_results(void){
result=STATE_CRITICAL;
else if(requested_responses<requested_servers)
result=STATE_WARNING;
- else if(request_specific_address==TRUE && received_requested_address==FALSE)
+ else if(request_specific_address && !received_requested_address)
result=STATE_WARNING;
if(result==0) /* garrett honeycutt 2005 */
@@ -1028,8 +1025,8 @@ int get_results(void){
if(requested_servers>0)
printf(_(", %s%d of %d requested servers responded"),((requested_responses<requested_servers) && requested_responses>0)?"only ":"",requested_responses,requested_servers);
- if(request_specific_address==TRUE)
- printf(_(", requested address (%s) was %soffered"),inet_ntoa(requested_address),(received_requested_address==TRUE)?"":_("not "));
+ if(request_specific_address)
+ printf(_(", requested address (%s) was %soffered"),inet_ntoa(requested_address),(received_requested_address)?"":_("not "));
printf(_(", max lease time = "));
if(max_lease_time==DHCP_INFINITE_TIME)
@@ -1090,7 +1087,7 @@ int call_getopt(int argc, char **argv){
case 'r': /* address we are requested from DHCP servers */
resolve_host(optarg,&requested_address);
- request_specific_address=TRUE;
+ request_specific_address=true;
break;
case 't': /* timeout */
More information about the Commits
mailing list