crimson/osd: should capture reply_t struct by reference

before this change, the iterator is captured by value, but it could be
invalidated when more elements are inserted into the container.

Signed-off-by: Kefu Chai <kchai@redhat.com>

diff --git a/src/crimson/osd/heartbeat.cc b/src/crimson/osd/heartbeat.cc
index 9fee3b91d3f35de08502656ab9a2189d314bfc1e..c075ad6867dabffeaff6eaf386126e47459463d8 100644 (file)
--- a/src/crimson/osd/heartbeat.cc
+++ b/src/crimson/osd/heartbeat.cc
@@ -297,7 +297,7 @@ seastar::future<> Heartbeat::send_heartbeats()
       std::vector<ceph::net::ConnectionRef> conns{info.con_front,
                                                   info.con_back};
       return seastar::parallel_for_each(std::move(conns),
-        [=] (auto con) {
+        [sent_stamp, &reply=reply->second, this] (auto con) {
           if (con) {
             auto min_message = static_cast<uint32_t>(
               local_conf()->osd_heartbeat_min_size);
@@ -307,7 +307,7 @@ seastar::future<> Heartbeat::send_heartbeats()
                                                sent_stamp,
                                                min_message);
             return con->send(ping).then([&reply] {
-              reply->second.unacknowledged++;
+              reply.unacknowledged++;
               return seastar::now();
             });
           } else {