Replace 'volatile int' to 'AtomicInteger' (Security for multiplayer)

imesense · Dec 18, 2024 · cedce36 · cedce36
1 parent 0ddf616
commit cedce36
Showing 1 changed file with 6 additions and 4 deletions.
diff --git a/...-1.12.2/src/main/java/org/imesense/dynamicspawncontrol/event/OnEventOvergrowingGrass.java b/...-1.12.2/src/main/java/org/imesense/dynamicspawncontrol/event/OnEventOvergrowingGrass.java
@@ -16,6 +16,8 @@
 import org.imesense.dynamicspawncontrol.core.logfile.Log;
 import org.imesense.dynamicspawncontrol.core.util.CodeGeneric;
 
+import java.util.concurrent.atomic.AtomicInteger;
+
 /**
  *
  */
@@ -25,12 +27,12 @@ public final class OnEventOvergrowingGrass
     /**
      *
      */
-    private static volatile int _TICK_COUNTER = 0;
+    private static boolean instanceExists = false;
 
     /**
      *
      */
-    private static boolean instanceExists = false;
+    private static final AtomicInteger TICK_COUNTER = new AtomicInteger(0);
 
     /**
      *
@@ -60,12 +62,12 @@ public static void onWorldTick_0(TickEvent.WorldTickEvent worldTickEvent)
             return;
         }
 
-        if (++_TICK_COUNTER < 20)
+        if (TICK_COUNTER.incrementAndGet() < 20)
         {
             return;
         }
 
-        _TICK_COUNTER = 0;
+        TICK_COUNTER.set(0);
 
         World world = worldTickEvent.world;