src/utils_threshold.c: Fix the configuration of {Failure,Warning}Max and inverted...

The first issue is a simple copy'n'paste error, the second a nasty oversight.

diff --git a/src/utils_threshold.c b/src/utils_threshold.c
index ea4b5916201cd28b0626194c9e54b29ee8cd8954..d0ab2b9f430948c72960a487dec38ae3c6c9c61a 100644 (file)
--- a/src/utils_threshold.c
+++ b/src/utils_threshold.c
@@ -146,9 +146,9 @@ static int ut_config_type_max (threshold_t *th, oconfig_item_t *ci)
   }
 
   if (strcasecmp (ci->key, "WarningMax") == 0)
-    th->warning_min = ci->values[0].value.number;
+    th->warning_max = ci->values[0].value.number;
   else
-    th->failure_min = ci->values[0].value.number;
+    th->failure_max = ci->values[0].value.number;
 
   return (0);
 } /* int ut_config_type_max */
@@ -536,13 +536,17 @@ int ut_check_threshold (const data_set_t *ds, const value_list_t *vl)
     int is_failure = 0;
 
     if ((th->flags & UT_FLAG_INVERT) != 0)
+    {
       is_inverted = 1;
+      is_warning--;
+      is_failure--;
+    }
     if ((!isnan (th->failure_min) && (th->failure_min > values[i]))
        || (!isnan (th->failure_max) && (th->failure_max < values[i])))
-      is_failure = is_inverted - 1;
+      is_failure++;
     if ((!isnan (th->warning_min) && (th->warning_min > values[i]))
        || (!isnan (th->warning_max) && (th->warning_max < values[i])))
-      is_warning = is_inverted - 1;
+      is_warning++;
 
     if ((is_failure != 0) || (is_warning != 0))
     {