diff options
| author |  Lorenz <12514511+RincewindsHat@users.noreply.github.com>
| 2022-09-11 06:29:17 +0200 |
|---|---|---|
| committer |  GitHub <noreply@github.com>
| 2022-09-11 06:29:17 +0200 |
| commit | fbbc9fcbd566db60058047870fbd6a2105e96663 (patch) | |
| tree | ef100ea34f3a8b5abad2c0d62794255a01792225 /plugins/check_swap.c | |
| parent | 3ad5fe9d84138da1451429bfac3b9b4024393d25 (diff) | |
| download | monitoring-plugins-fbbc9fcbd566db60058047870fbd6a2105e96663.tar.gz | |
Check swap compiler warnings (#1756)
* Fix compiler warnings
* Fix superfluous whitespaces
Diffstat (limited to 'plugins/check_swap.c')
| -rw-r--r-- | plugins/check_swap.c | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/plugins/check_swap.c b/plugins/check_swap.c index ff58b15f..a607da1e 100644 --- a/plugins/check_swap.c +++ b/plugins/check_swap.c @@ -410,7 +410,6 @@ check_swap(float free_swap_mb, float total_swap_mb) uint64_t usage_percentage = ((total_swap_mb - free_swap_mb) / total_swap_mb) * 100; if (crit.is_percentage && - usage_percentage >= 0 && crit.value != 0 && usage_percentage >= (100 - crit.value)) { @@ -418,7 +417,6 @@ check_swap(float free_swap_mb, float total_swap_mb) } if (warn.is_percentage && - usage_percentage >= 0 && warn.value != 0 && usage_percentage >= (100 - warn.value)) { @@ -475,10 +473,9 @@ process_arguments (int argc, char **argv) if (is_uint64(optarg, &warn.value)) { if (warn.value > 100) { usage4 (_("Warning threshold percentage must be <= 100!")); - } else { - break; } } + break; } else { /* It's Bytes */ warn.is_percentage = 0; @@ -506,10 +503,9 @@ process_arguments (int argc, char **argv) if (is_uint64(optarg, &crit.value)) { if (crit.value> 100) { usage4 (_("Critical threshold percentage must be <= 100!")); - } else { - break; } } + break; } else { /* It's Bytes */ crit.is_percentage = 0; @@ -527,6 +523,7 @@ process_arguments (int argc, char **argv) if ((no_swap_state = mp_translate_state(optarg)) == ERROR) { usage4 (_("no-swap result must be a valid state name (OK, WARNING, CRITICAL, UNKNOWN) or integer (0-3).")); } + break; case 'v': /* verbose */ verbose++; break; |