.

wcfs/client/wcfs.cpp

@@ -556,6 +556,15 @@ error _Conn::resync(zodb::Tid at) {
     wconn._atMu.RUnlock();
     etrace("");
 
+    // bring wconn + fileh + mmaps down on error
+    bool retok = false;
+    defer([&]() {
+        if (!retok)
+            // XXX log?
+            //fprintf(stderr, "\n\nTODO: bring wconn + fileh + mmaps down on error\n\n\n");
+            wconn.close(); // ignore error
+    });
+
     // XXX downErr -> E
     // XXX at ^ (increases)
 
@@ -587,13 +596,6 @@ error _Conn::resync(zodb::Tid at) {
     if (err != nil)
         return E(err);
 
-    bool retok = false;
-    defer([&]() {
-        if (!retok)
-            //panic("TODO: bring wconn + fileh + mmaps down on error");  // XXX
-            fprintf(stderr, "\n\nTODO: bring wconn + fileh + mmaps down on error\n\n\n");
-    });
-
     // set new wconn.at early, so that e.g. Conn.open running simultaneously
     // to second part of resync (see below) uses new at.
     wconn.at = at;
@@ -611,7 +613,7 @@ error _Conn::resync(zodb::Tid at) {
         FileH        f    = fit.second;
 
         // TODO if file has no mappings and was not used during whole prev
-        // cycle - forget and stop watching it
+        // cycle - forget and stop watching it?
 
         // "opening" or "closing" fileh - their setup/teardown is currently
         // handled by Conn.open and FileH.close correspondingly.