diff options
| author | RincewindsHat <12514511+RincewindsHat@users.noreply.github.com> | 2023-10-18 20:37:17 +0200 |
|---|---|---|
| committer | RincewindsHat <12514511+RincewindsHat@users.noreply.github.com> | 2023-10-18 20:37:17 +0200 |
| commit | bbd2bd2fa745a83633ea9b19ce6975e858dc78f1 (patch) | |
| tree | 67c4e9db8bf83a5198545a6c3dda47271bdb8610 /plugins | |
| parent | 3aff9c8d13253994034f2743c0223e7acd13f384 (diff) | |
| download | monitoring-plugins-bbd2bd2fa745a83633ea9b19ce6975e858dc78f1.tar.gz | |
check_ups: Use C99 booleans
Diffstat (limited to 'plugins')
| -rw-r--r-- | plugins/check_ups.c | 44 |
1 files changed, 22 insertions, 22 deletions
diff --git a/plugins/check_ups.c b/plugins/check_ups.c index 68737c4b..d1ded620 100644 --- a/plugins/check_ups.c +++ b/plugins/check_ups.c | |||
| @@ -75,8 +75,8 @@ char *server_address; | |||
| 75 | char *ups_name = NULL; | 75 | char *ups_name = NULL; |
| 76 | double warning_value = 0.0; | 76 | double warning_value = 0.0; |
| 77 | double critical_value = 0.0; | 77 | double critical_value = 0.0; |
| 78 | int check_warn = FALSE; | 78 | bool check_warn = false; |
| 79 | int check_crit = FALSE; | 79 | bool check_crit = false; |
| 80 | int check_variable = UPS_NONE; | 80 | int check_variable = UPS_NONE; |
| 81 | int supported_options = UPS_NONE; | 81 | int supported_options = UPS_NONE; |
| 82 | int status = UPSSTATUS_NONE; | 82 | int status = UPSSTATUS_NONE; |
| @@ -86,7 +86,7 @@ double ups_battery_percent = 0.0; | |||
| 86 | double ups_load_percent = 0.0; | 86 | double ups_load_percent = 0.0; |
| 87 | double ups_temperature = 0.0; | 87 | double ups_temperature = 0.0; |
| 88 | char *ups_status; | 88 | char *ups_status; |
| 89 | int temp_output_c = 0; | 89 | bool temp_output_c = false; |
| 90 | 90 | ||
| 91 | int determine_status (void); | 91 | int determine_status (void); |
| 92 | int get_ups_variable (const char *, char *); | 92 | int get_ups_variable (const char *, char *); |
| @@ -205,21 +205,21 @@ main (int argc, char **argv) | |||
| 205 | ups_utility_deviation = ups_utility_voltage - 120.0; | 205 | ups_utility_deviation = ups_utility_voltage - 120.0; |
| 206 | 206 | ||
| 207 | if (check_variable == UPS_UTILITY) { | 207 | if (check_variable == UPS_UTILITY) { |
| 208 | if (check_crit==TRUE && ups_utility_deviation>=critical_value) { | 208 | if (check_crit && ups_utility_deviation>=critical_value) { |
| 209 | result = STATE_CRITICAL; | 209 | result = STATE_CRITICAL; |
| 210 | } | 210 | } |
| 211 | else if (check_warn==TRUE && ups_utility_deviation>=warning_value) { | 211 | else if (check_warn && ups_utility_deviation>=warning_value) { |
| 212 | result = max_state (result, STATE_WARNING); | 212 | result = max_state (result, STATE_WARNING); |
| 213 | } | 213 | } |
| 214 | xasprintf (&data, "%s", | 214 | xasprintf (&data, "%s", |
| 215 | perfdata ("voltage", (long)(1000*ups_utility_voltage), "mV", | 215 | perfdata ("voltage", (long)(1000*ups_utility_voltage), "mV", |
| 216 | check_warn, (long)(1000*warning_value), | 216 | check_warn, (long)(1000*warning_value), |
| 217 | check_crit, (long)(1000*critical_value), | 217 | check_crit, (long)(1000*critical_value), |
| 218 | TRUE, 0, FALSE, 0)); | 218 | true, 0, false, 0)); |
| 219 | } else { | 219 | } else { |
| 220 | xasprintf (&data, "%s", | 220 | xasprintf (&data, "%s", |
| 221 | perfdata ("voltage", (long)(1000*ups_utility_voltage), "mV", | 221 | perfdata ("voltage", (long)(1000*ups_utility_voltage), "mV", |
| 222 | FALSE, 0, FALSE, 0, TRUE, 0, FALSE, 0)); | 222 | false, 0, false, 0, true, 0, false, 0)); |
| 223 | } | 223 | } |
| 224 | } | 224 | } |
| 225 | 225 | ||
| @@ -234,21 +234,21 @@ main (int argc, char **argv) | |||
| 234 | xasprintf (&message, "%sBatt=%3.1f%% ", message, ups_battery_percent); | 234 | xasprintf (&message, "%sBatt=%3.1f%% ", message, ups_battery_percent); |
| 235 | 235 | ||
| 236 | if (check_variable == UPS_BATTPCT) { | 236 | if (check_variable == UPS_BATTPCT) { |
| 237 | if (check_crit==TRUE && ups_battery_percent <= critical_value) { | 237 | if (check_crit && ups_battery_percent <= critical_value) { |
| 238 | result = STATE_CRITICAL; | 238 | result = STATE_CRITICAL; |
| 239 | } | 239 | } |
| 240 | else if (check_warn==TRUE && ups_battery_percent<=warning_value) { | 240 | else if (check_warn && ups_battery_percent<=warning_value) { |
| 241 | result = max_state (result, STATE_WARNING); | 241 | result = max_state (result, STATE_WARNING); |
| 242 | } | 242 | } |
| 243 | xasprintf (&data, "%s %s", data, | 243 | xasprintf (&data, "%s %s", data, |
| 244 | perfdata ("battery", (long)ups_battery_percent, "%", | 244 | perfdata ("battery", (long)ups_battery_percent, "%", |
| 245 | check_warn, (long)(warning_value), | 245 | check_warn, (long)(warning_value), |
| 246 | check_crit, (long)(critical_value), | 246 | check_crit, (long)(critical_value), |
| 247 | TRUE, 0, TRUE, 100)); | 247 | true, 0, true, 100)); |
| 248 | } else { | 248 | } else { |
| 249 | xasprintf (&data, "%s %s", data, | 249 | xasprintf (&data, "%s %s", data, |
| 250 | perfdata ("battery", (long)ups_battery_percent, "%", | 250 | perfdata ("battery", (long)ups_battery_percent, "%", |
| 251 | FALSE, 0, FALSE, 0, TRUE, 0, TRUE, 100)); | 251 | false, 0, false, 0, true, 0, true, 100)); |
| 252 | } | 252 | } |
| 253 | } | 253 | } |
| 254 | 254 | ||
| @@ -263,21 +263,21 @@ main (int argc, char **argv) | |||
| 263 | xasprintf (&message, "%sLoad=%3.1f%% ", message, ups_load_percent); | 263 | xasprintf (&message, "%sLoad=%3.1f%% ", message, ups_load_percent); |
| 264 | 264 | ||
| 265 | if (check_variable == UPS_LOADPCT) { | 265 | if (check_variable == UPS_LOADPCT) { |
| 266 | if (check_crit==TRUE && ups_load_percent>=critical_value) { | 266 | if (check_crit && ups_load_percent>=critical_value) { |
| 267 | result = STATE_CRITICAL; | 267 | result = STATE_CRITICAL; |
| 268 | } | 268 | } |
| 269 | else if (check_warn==TRUE && ups_load_percent>=warning_value) { | 269 | else if (check_warn && ups_load_percent>=warning_value) { |
| 270 | result = max_state (result, STATE_WARNING); | 270 | result = max_state (result, STATE_WARNING); |
| 271 | } | 271 | } |
| 272 | xasprintf (&data, "%s %s", data, | 272 | xasprintf (&data, "%s %s", data, |
| 273 | perfdata ("load", (long)ups_load_percent, "%", | 273 | perfdata ("load", (long)ups_load_percent, "%", |
| 274 | check_warn, (long)(warning_value), | 274 | check_warn, (long)(warning_value), |
| 275 | check_crit, (long)(critical_value), | 275 | check_crit, (long)(critical_value), |
| 276 | TRUE, 0, TRUE, 100)); | 276 | true, 0, true, 100)); |
| 277 | } else { | 277 | } else { |
| 278 | xasprintf (&data, "%s %s", data, | 278 | xasprintf (&data, "%s %s", data, |
| 279 | perfdata ("load", (long)ups_load_percent, "%", | 279 | perfdata ("load", (long)ups_load_percent, "%", |
| 280 | FALSE, 0, FALSE, 0, TRUE, 0, TRUE, 100)); | 280 | false, 0, false, 0, true, 0, true, 100)); |
| 281 | } | 281 | } |
| 282 | } | 282 | } |
| 283 | 283 | ||
| @@ -300,21 +300,21 @@ main (int argc, char **argv) | |||
| 300 | } | 300 | } |
| 301 | 301 | ||
| 302 | if (check_variable == UPS_TEMP) { | 302 | if (check_variable == UPS_TEMP) { |
| 303 | if (check_crit==TRUE && ups_temperature>=critical_value) { | 303 | if (check_crit && ups_temperature>=critical_value) { |
| 304 | result = STATE_CRITICAL; | 304 | result = STATE_CRITICAL; |
| 305 | } | 305 | } |
| 306 | else if (check_warn == TRUE && ups_temperature>=warning_value) { | 306 | else if (check_warn && ups_temperature>=warning_value) { |
| 307 | result = max_state (result, STATE_WARNING); | 307 | result = max_state (result, STATE_WARNING); |
| 308 | } | 308 | } |
| 309 | xasprintf (&data, "%s %s", data, | 309 | xasprintf (&data, "%s %s", data, |
| 310 | perfdata ("temp", (long)ups_temperature, tunits, | 310 | perfdata ("temp", (long)ups_temperature, tunits, |
| 311 | check_warn, (long)(warning_value), | 311 | check_warn, (long)(warning_value), |
| 312 | check_crit, (long)(critical_value), | 312 | check_crit, (long)(critical_value), |
| 313 | TRUE, 0, FALSE, 0)); | 313 | true, 0, false, 0)); |
| 314 | } else { | 314 | } else { |
| 315 | xasprintf (&data, "%s %s", data, | 315 | xasprintf (&data, "%s %s", data, |
| 316 | perfdata ("temp", (long)ups_temperature, tunits, | 316 | perfdata ("temp", (long)ups_temperature, tunits, |
| 317 | FALSE, 0, FALSE, 0, TRUE, 0, FALSE, 0)); | 317 | false, 0, false, 0, true, 0, false, 0)); |
| 318 | } | 318 | } |
| 319 | } | 319 | } |
| 320 | 320 | ||
| @@ -508,7 +508,7 @@ process_arguments (int argc, char **argv) | |||
| 508 | } | 508 | } |
| 509 | break; | 509 | break; |
| 510 | case 'T': /* FIXME: to be improved (ie "-T C" for Celsius or "-T F" for Fahrenheit) */ | 510 | case 'T': /* FIXME: to be improved (ie "-T C" for Celsius or "-T F" for Fahrenheit) */ |
| 511 | temp_output_c = 1; | 511 | temp_output_c = true; |
| 512 | break; | 512 | break; |
| 513 | case 'u': /* ups name */ | 513 | case 'u': /* ups name */ |
| 514 | ups_name = optarg; | 514 | ups_name = optarg; |
| @@ -524,7 +524,7 @@ process_arguments (int argc, char **argv) | |||
| 524 | case 'c': /* critical time threshold */ | 524 | case 'c': /* critical time threshold */ |
| 525 | if (is_intnonneg (optarg)) { | 525 | if (is_intnonneg (optarg)) { |
| 526 | critical_value = atoi (optarg); | 526 | critical_value = atoi (optarg); |
| 527 | check_crit = TRUE; | 527 | check_crit = true; |
| 528 | } | 528 | } |
| 529 | else { | 529 | else { |
| 530 | usage2 (_("Critical time must be a positive integer"), optarg); | 530 | usage2 (_("Critical time must be a positive integer"), optarg); |
| @@ -533,7 +533,7 @@ process_arguments (int argc, char **argv) | |||
| 533 | case 'w': /* warning time threshold */ | 533 | case 'w': /* warning time threshold */ |
| 534 | if (is_intnonneg (optarg)) { | 534 | if (is_intnonneg (optarg)) { |
| 535 | warning_value = atoi (optarg); | 535 | warning_value = atoi (optarg); |
| 536 | check_warn = TRUE; | 536 | check_warn = true; |
| 537 | } | 537 | } |
| 538 | else { | 538 | else { |
| 539 | usage2 (_("Warning time must be a positive integer"), optarg); | 539 | usage2 (_("Warning time must be a positive integer"), optarg); |
