[monitoring-plugins] check_real: clang-format
Lorenz Kästle
git at monitoring-plugins.org
Wed Mar 12 18:00:15 CET 2025
Module: monitoring-plugins
Branch: master
Commit: a8b7df88113ae08d1b34f6efcd7d6d971e2b7c22
Author: Lorenz Kästle <12514511+RincewindsHat at users.noreply.github.com>
Date: Wed Mar 12 16:54:32 2025 +0100
URL: https://www.monitoring-plugins.org/repositories/monitoring-plugins/commit/?id=a8b7df88
check_real: clang-format
---
plugins/check_real.c | 108 +++++++++++++++++++++++++++++++--------------------
1 file changed, 65 insertions(+), 43 deletions(-)
diff --git a/plugins/check_real.c b/plugins/check_real.c
index 369a88b1..3737f69d 100644
--- a/plugins/check_real.c
+++ b/plugins/check_real.c
@@ -66,8 +66,9 @@ int main(int argc, char **argv) {
/* Parse extra opts if any */
argv = np_extra_opts(&argc, argv, progname);
- if (process_arguments(argc, argv) == ERROR)
+ if (process_arguments(argc, argv) == ERROR) {
usage4(_("Could not parse arguments"));
+ }
/* initialize alarm signal handling */
signal(SIGALRM, socket_timeout_alarm_handler);
@@ -78,8 +79,9 @@ int main(int argc, char **argv) {
/* try to connect to the host at the given port number */
int socket;
- if (my_tcp_connect(server_address, server_port, &socket) != STATE_OK)
+ if (my_tcp_connect(server_address, server_port, &socket) != STATE_OK) {
die(STATE_CRITICAL, _("Unable to connect to %s on port %d\n"), server_address, server_port);
+ }
/* Part I - Server Check */
@@ -100,16 +102,18 @@ int main(int argc, char **argv) {
result = recv(socket, buffer, MAX_INPUT_BUFFER - 1, 0);
/* return a CRITICAL status if we couldn't read any data */
- if (result == -1)
+ if (result == -1) {
die(STATE_CRITICAL, _("No data received from %s\n"), host_name);
+ }
char *status_line = NULL;
/* make sure we find the response we are looking for */
if (!strstr(buffer, server_expect)) {
- if (server_port == PORT)
+ if (server_port == PORT) {
printf("%s\n", _("Invalid REAL response received from host"));
- else
+ } else {
printf(_("Invalid REAL response received from host on port %d\n"), server_port);
+ }
} else {
/* else we got the REAL string, so check the return code */
@@ -119,33 +123,37 @@ int main(int argc, char **argv) {
status_line = (char *)strtok(buffer, "\n");
- if (strstr(status_line, "200"))
+ if (strstr(status_line, "200")) {
result = STATE_OK;
+ }
/* client errors result in a warning state */
- else if (strstr(status_line, "400"))
+ else if (strstr(status_line, "400")) {
result = STATE_WARNING;
- else if (strstr(status_line, "401"))
+ } else if (strstr(status_line, "401")) {
result = STATE_WARNING;
- else if (strstr(status_line, "402"))
+ } else if (strstr(status_line, "402")) {
result = STATE_WARNING;
- else if (strstr(status_line, "403"))
+ } else if (strstr(status_line, "403")) {
result = STATE_WARNING;
- else if (strstr(status_line, "404"))
+ } else if (strstr(status_line, "404")) {
result = STATE_WARNING;
+ }
/* server errors result in a critical state */
- else if (strstr(status_line, "500"))
+ else if (strstr(status_line, "500")) {
result = STATE_CRITICAL;
- else if (strstr(status_line, "501"))
+ } else if (strstr(status_line, "501")) {
result = STATE_CRITICAL;
- else if (strstr(status_line, "502"))
+ } else if (strstr(status_line, "502")) {
result = STATE_CRITICAL;
- else if (strstr(status_line, "503"))
+ } else if (strstr(status_line, "503")) {
result = STATE_CRITICAL;
+ }
- else
+ else {
result = STATE_UNKNOWN;
+ }
}
/* Part II - Check stream exists and is ok */
@@ -176,10 +184,11 @@ int main(int argc, char **argv) {
} else {
/* make sure we find the response we are looking for */
if (!strstr(buffer, server_expect)) {
- if (server_port == PORT)
+ if (server_port == PORT) {
printf("%s\n", _("Invalid REAL response received from host"));
- else
+ } else {
printf(_("Invalid REAL response received from host on port %d\n"), server_port);
+ }
} else {
/* else we got the REAL string, so check the return code */
@@ -190,33 +199,37 @@ int main(int argc, char **argv) {
status_line = (char *)strtok(buffer, "\n");
- if (strstr(status_line, "200"))
+ if (strstr(status_line, "200")) {
result = STATE_OK;
+ }
/* client errors result in a warning state */
- else if (strstr(status_line, "400"))
+ else if (strstr(status_line, "400")) {
result = STATE_WARNING;
- else if (strstr(status_line, "401"))
+ } else if (strstr(status_line, "401")) {
result = STATE_WARNING;
- else if (strstr(status_line, "402"))
+ } else if (strstr(status_line, "402")) {
result = STATE_WARNING;
- else if (strstr(status_line, "403"))
+ } else if (strstr(status_line, "403")) {
result = STATE_WARNING;
- else if (strstr(status_line, "404"))
+ } else if (strstr(status_line, "404")) {
result = STATE_WARNING;
+ }
/* server errors result in a critical state */
- else if (strstr(status_line, "500"))
+ else if (strstr(status_line, "500")) {
result = STATE_CRITICAL;
- else if (strstr(status_line, "501"))
+ } else if (strstr(status_line, "501")) {
result = STATE_CRITICAL;
- else if (strstr(status_line, "502"))
+ } else if (strstr(status_line, "502")) {
result = STATE_CRITICAL;
- else if (strstr(status_line, "503"))
+ } else if (strstr(status_line, "503")) {
result = STATE_CRITICAL;
+ }
- else
+ else {
result = STATE_UNKNOWN;
+ }
}
}
}
@@ -224,15 +237,17 @@ int main(int argc, char **argv) {
/* Return results */
if (result == STATE_OK) {
- if (check_critical_time && (end_time - start_time) > critical_time)
+ if (check_critical_time && (end_time - start_time) > critical_time) {
result = STATE_CRITICAL;
- else if (check_warning_time && (end_time - start_time) > warning_time)
+ } else if (check_warning_time && (end_time - start_time) > warning_time) {
result = STATE_WARNING;
+ }
/* Put some HTML in here to create a dynamic link */
printf(_("REAL %s - %d second response time\n"), state_text(result), (int)(end_time - start_time));
- } else
+ } else {
printf("%s\n", status_line);
+ }
/* close the connection */
close(socket);
@@ -252,16 +267,18 @@ int process_arguments(int argc, char **argv) {
{"verbose", no_argument, 0, 'v'}, {"version", no_argument, 0, 'V'},
{"help", no_argument, 0, 'h'}, {0, 0, 0, 0}};
- if (argc < 2)
+ if (argc < 2) {
return ERROR;
+ }
for (int i = 1; i < argc; i++) {
- if (strcmp("-to", argv[i]) == 0)
+ if (strcmp("-to", argv[i]) == 0) {
strcpy(argv[i], "-t");
- else if (strcmp("-wt", argv[i]) == 0)
+ } else if (strcmp("-wt", argv[i]) == 0) {
strcpy(argv[i], "-w");
- else if (strcmp("-ct", argv[i]) == 0)
+ } else if (strcmp("-ct", argv[i]) == 0) {
strcpy(argv[i], "-c");
+ }
}
int option_char;
@@ -269,18 +286,20 @@ int process_arguments(int argc, char **argv) {
int option = 0;
option_char = getopt_long(argc, argv, "+hvVI:H:e:u:p:w:c:t:", longopts, &option);
- if (option_char == -1 || option_char == EOF)
+ if (option_char == -1 || option_char == EOF) {
break;
+ }
switch (option_char) {
case 'I': /* hostname */
case 'H': /* hostname */
- if (server_address)
+ if (server_address) {
break;
- else if (is_host(optarg))
+ } else if (is_host(optarg)) {
server_address = optarg;
- else
+ } else {
usage2(_("Invalid hostname/address"), optarg);
+ }
break;
case 'e': /* string to expect in response header */
server_expect = optarg;
@@ -341,14 +360,17 @@ int process_arguments(int argc, char **argv) {
}
}
- if (server_address == NULL)
+ if (server_address == NULL) {
usage4(_("You must provide a server to check"));
+ }
- if (host_name == NULL)
+ if (host_name == NULL) {
host_name = strdup(server_address);
+ }
- if (server_expect == NULL)
+ if (server_expect == NULL) {
server_expect = strdup(EXPECT);
+ }
return OK;
}
More information about the Commits
mailing list