heartbeatmap: fix stupid race

atomic_t is probably better here, actually... :/

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

diff --git a/src/common/HeartbeatMap.cc b/src/common/HeartbeatMap.cc
index dc63d8ebb5c4866c65deec989938917c453be83b..8e47b590c4631d70b63cc51a59151125750b05ae 100644 (file)
--- a/src/common/HeartbeatMap.cc
+++ b/src/common/HeartbeatMap.cc
@@ -68,7 +68,8 @@ bool HeartbeatMap::is_healthy()
        p != m_workers.end();
        ++p) {
     heartbeat_handle_d *h = *p;
-    if (h->timeout && h->timeout < now) {
+    time_t timeout = h->timeout;
+    if (timeout && timeout < now) {
       ldout(m_cct, 0) << "is_healthy " << h->thread << " '" << h->name << "'"
                      << " timed out" << dendl;
       healthy = false;