Guard direct memory writes during closing to avoid SEGV

Fixes ChronicleEnterprise/Chronicle-Map-Enterprise#73
OpenHFT · Oct 12, 2021 · 0a8e703 · 0a8e703
1 parent 1dfde0f
commit 0a8e703
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/src/main/java/net/openhft/chronicle/hash/impl/VanillaChronicleHash.java b/src/main/java/net/openhft/chronicle/hash/impl/VanillaChronicleHash.java
@@ -724,6 +724,11 @@ public final void checkKey(final Object key) {
         }
     }
 
+    public void throwExceptionIfClosing() throws IllegalStateException {
+        if (this.isClosing())
+            throw new ChronicleHashClosedException(this.getClass().getName() + " closing", Jvm.getValue(this, "closedHere"));
+    }
+
     @Override
     public void throwExceptionIfClosed() throws IllegalStateException {
         if (this.isClosed())

diff --git a/src/main/java/net/openhft/chronicle/map/ReplicatedChronicleMap.java b/src/main/java/net/openhft/chronicle/map/ReplicatedChronicleMap.java
@@ -324,7 +324,7 @@ public byte identifier() {
 
     @Override
     public ModificationIterator acquireModificationIterator(final byte remoteIdentifier) {
-        throwExceptionIfClosed();
+        throwExceptionIfClosing();
 
         ModificationIterator modificationIterator = modificationIterators.get(remoteIdentifier);
         if (modificationIterator != null)