Merge pull request #743 from pshriwise/overlap_fix

svalinn · Jun 20, 2021 · 1a97a9a · 1a97a9a
2 parents 3401ad9 + 19796eb
commit 1a97a9a
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 0 deletions.
diff --git a/news/PR-0743.rst b/news/PR-0743.rst
@@ -0,0 +1,13 @@
+**Added:** None
+
+**Changed:**
+
+  - Threadsafe fix for the overlap check test.
+
+**Deprecated:** None
+
+**Removed:** None
+
+**Fixed:** None
+
+**Security:** None
diff --git a/src/overlap_check/overlap.cpp b/src/overlap_check/overlap.cpp
@@ -31,6 +31,7 @@ ErrorCode check_location_for_overlap(std::shared_ptr<GeomQueryTool>& GQT,
   }
 
   if (vols_found.size() > 1) {
+#pragma omp critical
     overlap_map[vols_found] = loc;
   }
 
@@ -51,6 +52,7 @@ ErrorCode check_location_for_overlap(std::shared_ptr<GeomQueryTool>& GQT,
   }
 
   if (vols_found.size() > 1) {
+#pragma omp critical
     overlap_map[vols_found] = loc;
   }