fix : add synchronized on the returned list when traversing it Splite…

…rator
ShiftLeftSecurity · Mar 21, 2024 · 0272bdb · 0272bdb
1 parent 0f90909
commit 0272bdb
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/core/src/main/java/overflowdb/ReferenceManager.java b/core/src/main/java/overflowdb/ReferenceManager.java
@@ -152,8 +152,10 @@ private void safelyClearReferences(final List<NodeRef> refsToClear) {
    * useful when saving the graph
    */
   public void clearAllReferences() {
-    nodesWriter.writeAndClearBatched(clearableRefs.spliterator(), clearableRefs.size());
-    logger.debug("cleared all clearable references");
+    synchronized (clearableRefs){
+      nodesWriter.writeAndClearBatched(clearableRefs.spliterator(), clearableRefs.size());
+      logger.debug("cleared all clearable references");
+    }
   }
 
   @Override