mds: fix some memory leak

Fixes: http://tracker.ceph.com/issues/24289
Signed-off-by: "Yan, Zheng" <zyan@redhat.com>
(cherry picked from commit e7c149b93dc384ee4a2c8250c502548d12535123)

diff --git a/src/mds/Beacon.cc b/src/mds/Beacon.cc
index ea59c57b827c43b5e6ce3a57ff5982b26a0ebbd1..e94390f65bf4fbb834ade342b9bcfbc85547058f 100644 (file)
--- a/src/mds/Beacon.cc
+++ b/src/mds/Beacon.cc
@@ -142,6 +142,7 @@ void Beacon::handle_mds_beacon(MMDSBeacon *m)
     dout(10) << "handle_mds_beacon " << ceph_mds_state_name(m->get_state())
             << " seq " << m->get_seq() << " dne" << dendl;
   }
+  m->put();
 }
 
 

diff --git a/src/mds/Migrator.cc b/src/mds/Migrator.cc
index 1e2a3a302f6520269e53e09ddc7859e8aee4a87f..f1ccdd898a6690e695153fc8b2cfbdde73897ec2 100644 (file)
--- a/src/mds/Migrator.cc
+++ b/src/mds/Migrator.cc
@@ -3326,8 +3326,11 @@ void Migrator::handle_export_caps(MExportCaps *ex)
   assert(in->is_auth());
 
   // FIXME
-  if (!in->can_auth_pin())
+  if (!in->can_auth_pin()) {
+    ex->put();
     return;
+  }
+
   in->auth_pin(this);
 
   C_M_LoggedImportCaps *finish = new C_M_LoggedImportCaps(

diff --git a/src/mds/Server.cc b/src/mds/Server.cc
index 5132ece9e57cd433facaeb8f95e0104d3bd6c8b3..04a85d2673830ec35b28f960a347c8031019e470 100644 (file)
--- a/src/mds/Server.cc
+++ b/src/mds/Server.cc
@@ -1959,6 +1959,7 @@ void Server::handle_slave_request(MMDSSlaveRequest *m)
       } else {
        mdcache->request_finish(mdr);
       }
+      m->put();
       return;
     }
   }