client: additional sanity checks on link/unlink

Signed-off-by: Sage Weil <sage@newdream.net>

diff --git a/src/client/Client.cc b/src/client/Client.cc
index 9be3ebbc37264005f918bc73dd444b0ab4453cae..8e86e8354c676c159836c7b78431ac6cb70bee21 100644 (file)
--- a/src/client/Client.cc
+++ b/src/client/Client.cc
@@ -1820,6 +1820,8 @@ Dentry* Client::link(Dir *dir, const string& name, Inode *in, Dentry *dn)
     if (in->dir)
       dn->get();  // dir -> dn pin
 
+    assert(in->dn_set.count(dn) == 0);
+
     // only one parent for directories!
     if (in->is_dir() && !in->dn_set.empty()) {
       Dentry *olddn = in->get_first_parent();
@@ -1846,6 +1848,7 @@ void Client::unlink(Dentry *dn, bool keepdir)
     if (in->dir)
       dn->put();        // dir -> dn pin
     dn->inode = 0;
+    assert(in->dn_set.count(dn));
     in->dn_set.erase(dn);
     ldout(cct, 20) << "unlink  inode " << in << " parents now " << in->dn_set << dendl; 
     put_inode(in);