unlink the new path from the replaced inode in rename

Author: mersinvald

src/inode_table.rs

@@ -165,7 +165,9 @@ impl InodeTable {
             lookups = entry.lookups;
             if lookups == 0 {
                 delete = true;
-                self.by_path.remove(entry.path.as_ref().unwrap());
+                if let Some(path) = entry.path.as_ref() {
+                    self.by_path.remove(path);
+                }
             }
         }
 
@@ -180,6 +182,12 @@ impl InodeTable {
     /// Change an inode's path to a different one, without changing the inode number.
     /// Lookup counts remain unchanged, even if this is replacing another file.
     pub fn rename(&mut self, oldpath: &Path, newpath: Arc<PathBuf>) {
+        // unlink the new path from the previous inode holding it
+        // otherwise, when kernel signals to forget the replaced inode, the path is removed from the by_path table
+        if let Some(dst) = self.by_path.remove(Pathish::new(&newpath)) {
+            self.table[dst].path = None;
+        }
+
         let idx = self.by_path.remove(Pathish::new(oldpath)).unwrap();
         self.table[idx].path = Some(newpath.clone());
         self.by_path.insert(newpath, idx); // this can replace a path with a new inode