.

wcfs/client/wcfs.cpp

@@ -702,6 +702,10 @@ error _FileH::close() {
     _FileH& fileh = *this;
     Conn    wconn = fileh.wconn;
 
+    // XXX decref open count
+    // XXX fileh.close can be called several times and after first call another
+    // XXX fileh could be opened for the same foid. Be careful not to interfere with it.
+
     // XXX locking ok?
     wconn->_atMu.RLock();
     defer([&]() {
@@ -722,11 +726,8 @@ error _FileH::close() {
         return E(fmt::errorf("unwatch: %s", v(ack)));
 
     // remove fileh from wconn._filehTab
-    // fileh.close can be called several times and after first call another
-    // fileh could be opened for the same foid. Be careful not to erase it.
     wconn->_mu.Lock();
-    // XXX decref open count
-    if (wconn->_filehTab.get(fileh.foid)._ptr() == &fileh)
+    if (wconn->_filehTab.get(fileh.foid)._ptr() == &fileh) // XXX -> assert ?
         wconn->_filehTab.erase(fileh.foid);
     wconn->_mu.Unlock();