mon: Delete logger in destructor

In theory we could race on the 'logger' object in a similar fashion to
https://tracker.ceph.com/issues/48386 where the race is on the
Paxos::logger object.

Signed-off-by: Brad Hubbard <bhubbard@redhat.com>

diff --git a/src/mon/Monitor.cc b/src/mon/Monitor.cc
index dad67ab4d460239e0a26f739c8fbc08793d88a03..66d3a190efbf217d51e7c875e41140dbcf4a5fd2 100644 (file)
--- a/src/mon/Monitor.cc
+++ b/src/mon/Monitor.cc
@@ -287,6 +287,7 @@ Monitor::~Monitor()
 
   delete config_key_service;
   delete paxos;
+  delete logger;
   ceph_assert(session_map.sessions.empty());
 }
 
@@ -1082,8 +1083,6 @@ void Monitor::shutdown()
 
   if (logger) {
     cct->get_perfcounters_collection()->remove(logger);
-    delete logger;
-    logger = NULL;
   }
   if (cluster_logger) {
     if (cluster_logger_registered)