Validate timeperiod segments

fixes Icinga#8309
red61wjt · Jan 29, 2015 · 2f44581 · 2f44581
1 parent a3e0c9d
commit 2f44581
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 0 deletions.
diff --git a/lib/config/configitem.cpp b/lib/config/configitem.cpp
@@ -381,6 +381,7 @@ bool ConfigItem::ActivateItems(void)
 	}
 
 	upq.Join();
+	upq.ReportExceptions("ConfigItem");
 
 #ifdef I2_DEBUG
 	BOOST_FOREACH(const DynamicType::Ptr& type, DynamicType::GetTypes()) {

diff --git a/lib/icinga/legacytimeperiod.cpp b/lib/icinga/legacytimeperiod.cpp
@@ -379,6 +379,9 @@ void LegacyTimePeriod::ProcessTimeRanges(const String& timeranges, tm *reference
 
 	BOOST_FOREACH(const String& range, ranges) {
 		Dictionary::Ptr segment = ProcessTimeRange(range, reference);
+		if (segment->Get("begin") >= segment->Get("end"))
+			BOOST_THROW_EXCEPTION(std::invalid_argument("Time period segment ends before it begins"));
+
 		result->Add(segment);
 	}
 }