Fix a potential race condition which could delete a segment file afte…

…r it's brought back into service.
cojen · Oct 20, 2024 · 5bbedbb · 5bbedbb
1 parent f73c3fb
commit 5bbedbb
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/src/main/java/org/cojen/tupl/repl/FileTermLog.java b/src/main/java/org/cojen/tupl/repl/FileTermLog.java
@@ -1069,6 +1069,11 @@ Segment segmentForWriting(SegmentWriter writer, long position) throws IOExceptio
                 if (startPosition != startSegment.mStartPosition || startSegment.mMaxLength != 0) {
                     throw new AssertionError(startSegment);
                 }
+                // Wait for all work tasks to complete, ensuring that the truncate operation
+                // against the segment is finished before attempting to untruncate it.
+                synchronized (mWorker) {
+                    mWorker.join(false);
+                }
                 startSegment.untruncate(maxLength);
                 segment = startSegment;
             }