Add range checking to ConfigTag::getFloat.

2 files changed, 6 insertions, 3 deletions

diff --git a/src/configparser.cpp b/src/configparser.cpp
index 5ee86c811..8a3042eba 100644
--- a/src/configparser.cpp
+++ b/src/configparser.cpp
@@ -524,12 +524,15 @@ unsigned long ConfigTag::getDuration(const std::string& key, unsigned long def,
 	return ret;
 }
 
-double ConfigTag::getFloat(const std::string &key, double def)
+double ConfigTag::getFloat(const std::string& key, double def, double min, double max)
 {
 	std::string result;
 	if (!readString(key, result))
 		return def;
-	return strtod(result.c_str(), NULL);
+
+	double res = strtod(result.c_str(), NULL);
+	CheckRange(tag, key, res, def, min, max);
+	return res;
 }
 
 bool ConfigTag::getBool(const std::string &key, bool def)
diff --git a/src/coremods/core_xline/core_xline.cpp b/src/coremods/core_xline/core_xline.cpp
index d6c804748..ab80ab4ed 100644
--- a/src/coremods/core_xline/core_xline.cpp
+++ b/src/coremods/core_xline/core_xline.cpp
@@ -28,7 +28,7 @@ bool InsaneBan::MatchesEveryone(const std::string& mask, MatcherBase& test, User
 	if (insane->getBool(confkey))
 		return false;
 
-	float itrigger = insane->getFloat("trigger", 95.5);
+	float itrigger = insane->getFloat("trigger", 95.5, 0.0, 100.0);
 
 	long matches = test.Run(mask);