TECH: Dump: throw when writing meta-data in read-only mode

Chrono24 · Dec 20, 2023 · 61e0e20 · 61e0e20
1 parent bcbba48
commit 61e0e20
Showing 1 changed file with 9 additions and 3 deletions.
diff --git a/dump/src/util/dump/Dump.java b/dump/src/util/dump/Dump.java
@@ -405,7 +405,9 @@ public void close() throws IOException {
       if ( _updateRaf != null ) {
          _updateRaf.close();
       }
-      writeMeta();
+      if ( !isReadonly() ) {
+         writeMeta();
+      }
       if ( _metaRaf != null ) {
          _metaRaf.close();
          _metaRaf = null;
@@ -489,7 +491,7 @@ public void flush() throws IOException {
     */
    public void flushMeta() throws IOException {
       if ( isReadonly() ) {
-         return;
+         throw new AccessControlException("Flushing meta-data not allowed in read-only mode.");
       }
 
       writeMeta();
@@ -846,6 +848,10 @@ public E updateLast( E item ) throws IOException {
     * @return false if the dump was already locked
     */
    protected boolean acquireFileLock() {
+      if ( isReadonly() ) {
+         return false;
+      }
+
       synchronized ( this ) {
          try {
             if ( _dumpLock != null ) {
@@ -1068,7 +1074,7 @@ void writeDictionary() throws IOException {
 
    void writeMeta() throws IOException {
       if ( isReadonly() ) {
-         return;
+         throw new AccessControlException("Writing meta-data not allowed in read-only mode.");
       }
 
       getMetaRAF().seek(0);