Fix concurrency issue with AssetProducerThread.shutdown()

RPTools · Nov 18, 2024 · 9ee4cb9 · 9ee4cb9
1 parent aebe225
commit 9ee4cb9
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/src/main/java/net/rptools/maptool/server/MapToolServer.java b/src/main/java/net/rptools/maptool/server/MapToolServer.java
@@ -20,6 +20,7 @@
 import java.util.Map;
 import java.util.Map.Entry;
 import java.util.Random;
+import java.util.concurrent.atomic.AtomicBoolean;
 import javax.annotation.Nullable;
 import javax.swing.SwingUtilities;
 import net.rptools.clientserver.ConnectionFactory;
@@ -536,15 +537,15 @@ public void shutdown() {
   ////
   // CLASSES
   private class AssetProducerThread extends Thread {
-    private boolean stop = false;
+    private final AtomicBoolean stop = new AtomicBoolean(false);
 
     public AssetProducerThread() {
       setName("AssetProducerThread");
     }
 
     @Override
     public void run() {
-      while (!stop) {
+      while (!stop.get()) {
         Entry<String, AssetTransferManager> entryForException = null;
         try {
           boolean lookForMore = false;
@@ -575,7 +576,7 @@ public void run() {
     }
 
     public void shutdown() {
-      stop = true;
+      stop.set(true);
     }
   }
 }