Improve input validation

motis-project · Nov 25, 2024 · e484c18 · e484c18
1 parent b6e7c2e
commit e484c18
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/include/nigiri/routing/sanitize_query.h b/include/nigiri/routing/sanitize_query.h
@@ -6,7 +6,11 @@
 namespace nigiri::routing {
 
 inline void sanitize_query(query& q) {
-  if (q.max_travel_time_.count() <= 0 || q.max_travel_time_ > kMaxTravelTime) {
+  // Ensure upper bound required by algorithm is a valid duration
+  constexpr auto const kAdditionalRequiredTime = decltype(kMaxTravelTime){1};
+  static_assert(kMaxTravelTime + kAdditionalRequiredTime <=
+                decltype(kMaxTravelTime)::max());
+  if (q.max_travel_time_.count() < 0 || q.max_travel_time_ > kMaxTravelTime) {
     q.max_travel_time_ = kMaxTravelTime;
   }
 }