From 6c7d7bfd6b84e762287f2dd131cb2bdd857507d2 Mon Sep 17 00:00:00 2001 From: NovaFox161 Date: Fri, 23 Jul 2021 13:38:02 -0500 Subject: [PATCH] Fix some thread safety issues due to not using thread safe list --- .../discal/core/object/network/discal/NetworkInfo.java | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/core/src/main/java/org/dreamexposure/discal/core/object/network/discal/NetworkInfo.java b/core/src/main/java/org/dreamexposure/discal/core/object/network/discal/NetworkInfo.java index 595c2035..44173a07 100644 --- a/core/src/main/java/org/dreamexposure/discal/core/object/network/discal/NetworkInfo.java +++ b/core/src/main/java/org/dreamexposure/discal/core/object/network/discal/NetworkInfo.java @@ -11,9 +11,9 @@ import org.json.JSONObject; import org.springframework.stereotype.Component; import reactor.core.publisher.Mono; -import java.util.ArrayList; import java.util.Comparator; import java.util.List; +import java.util.concurrent.CopyOnWriteArrayList; /** * @author NovaFox161 @@ -26,7 +26,7 @@ import java.util.List; @SuppressWarnings("Duplicates") @Component public class NetworkInfo { - private final List clients = new ArrayList<>(); + private final List clients = new CopyOnWriteArrayList<>(); private int calCount; private int announcementCount; @@ -46,7 +46,7 @@ public class NetworkInfo { //Getters public List getClients() { - return new ArrayList<>(this.clients); + return new CopyOnWriteArrayList<>(this.clients); } public boolean doesClientExist(final int clientIndex) {