Fix: PROCESS_HOST_CHECK_RESULT misinterprets the given status code.

Fixes #5462

diff --git a/lib/icinga/externalcommandprocessor.cpp b/lib/icinga/externalcommandprocessor.cpp
index 0b206b94fc14173d827d2a026a0d03b68f676054..ae309989d9968fa0f837e9dd2435683db4892fdf 100644 (file)
--- a/lib/icinga/externalcommandprocessor.cpp
+++ b/lib/icinga/externalcommandprocessor.cpp
@@ -235,7 +235,17 @@ void ExternalCommandProcessor::ProcessHostCheckResult(double time, const std::ve
 
        int exitStatus = Convert::ToDouble(arguments[1]);
        CheckResult::Ptr result = PluginUtility::ParseCheckOutput(arguments[2]);
-       result->SetState(PluginUtility::ExitStatusToState(exitStatus));
+
+       ServiceState state;
+
+       if (exitStatus == 0)
+               state = StateOK;
+       else if (exitStatus == 1)
+               state = StateCritical;
+       else
+               BOOST_THROW_EXCEPTION(std::invalid_argument("Invalid status code: " + arguments[2]));
+
+       result->SetState(state);
 
        result->SetScheduleStart(time);
        result->SetScheduleEnd(time);