Merge pull request #1811 from crimilo/2.0

Fix race condition and exception when registering check permissions during rapid consecutive player joins on server start

src/main/java/ac/grim/grimac/manager/CheckManager.java

@@ -47,8 +47,11 @@
 import org.bukkit.permissions.Permission;
 import org.bukkit.permissions.PermissionDefault;
 
+import java.util.concurrent.atomic.AtomicBoolean;
+
 public class CheckManager {
     private static boolean inited;
+    private static final AtomicBoolean initedAtomic = new AtomicBoolean(false);
 
     ClassToInstanceMap<PacketCheck> packetChecks;
     ClassToInstanceMap<PositionCheck> positionCheck;
@@ -336,7 +339,12 @@ public <T extends PostPredictionCheck> T getPostPredictionCheck(Class<T> check)
     }
 
     private void init() {
+        // Fast non-thread safe check
         if (inited) return;
+        // Slow thread safe check
+        if (!initedAtomic.compareAndSet(false, true)) return;
+        inited = true;
+
         for (AbstractCheck check : allChecks.values()) {
             if (check.getCheckName() != null) {
                 String permissionName = "grim.exempt." + check.getCheckName().toLowerCase();
@@ -349,7 +357,5 @@ private void init() {
                 }
             }
         }
-
-        inited = true;
     }
 }