Make addCloseTask thread safe

(cherry picked from commit 325de11)
gsmet · Aug 3, 2021 · aee7a90 · aee7a90
1 parent c47aa86
commit aee7a90
Showing 1 changed file with 8 additions and 2 deletions.
diff --git a/...ts/bootstrap/core/src/main/java/io/quarkus/bootstrap/classloading/QuarkusClassLoader.java b/...ts/bootstrap/core/src/main/java/io/quarkus/bootstrap/classloading/QuarkusClassLoader.java
@@ -535,7 +535,9 @@ public Class<?> visibleDefineClass(String name, byte[] b, int off, int len) thro
     }
 
     public void addCloseTask(Runnable task) {
-        closeTasks.add(task);
+        synchronized (closeTasks) {
+            closeTasks.add(task);
+        }
     }
 
     @Override
@@ -546,7 +548,11 @@ public void close() {
             }
             closed = true;
         }
-        for (Runnable i : closeTasks) {
+        List<Runnable> tasks;
+        synchronized (closeTasks) {
+            tasks = new ArrayList<>(closeTasks);
+        }
+        for (Runnable i : tasks) {
             try {
                 i.run();
             } catch (Throwable t) {