Don't allow invalid max-transaction-time

GrimAnticheat · Sep 24, 2024 · ba8c411 · ba8c411
1 parent da1468e
commit ba8c411
Showing 1 changed file with 10 additions and 1 deletion.
diff --git a/src/main/java/ac/grim/grimac/manager/ConfigManager.java b/src/main/java/ac/grim/grimac/manager/ConfigManager.java
@@ -3,6 +3,7 @@
 import ac.grim.grimac.GrimAC;
 import ac.grim.grimac.GrimAPI;
 import ac.grim.grimac.utils.anticheat.LogUtil;
+import ac.grim.grimac.utils.math.GrimMath;
 import github.scarsz.configuralize.DynamicConfig;
 import github.scarsz.configuralize.Language;
 import lombok.Getter;
@@ -77,7 +78,15 @@ public void reload() {
         } catch (Exception e) {
             throw new RuntimeException("Failed to load config", e);
         }
-        maxPingTransaction = config.getIntElse("max-transaction-time", 60);
+
+        final int configuredMaxTransactionTime = config.getIntElse("max-transaction-time", 60);
+        maxPingTransaction = (int) GrimMath.clamp(configuredMaxTransactionTime, 0, 180);
+        if (maxPingTransaction != configuredMaxTransactionTime) {
+            LogUtil.warn("Detected invalid max-transaction-time! This setting is clamped between 0 and 180 to prevent issues. " +
+                    "Changed: " + configuredMaxTransactionTime + " -> " + maxPingTransaction);
+            LogUtil.warn("Attempting to disable or set this too high can result in memory usage issues.");
+        }
+
         ignoredClientPatterns.clear();
         for (String string : config.getStringList("client-brand.ignored-clients")) {
             try {