From 4fdf9d340b50266db08aa8a38c45400da2f928d8 Mon Sep 17 00:00:00 2001 From: Alexey Rybalchenko Date: Thu, 24 Jun 2021 11:08:52 +0200 Subject: [PATCH] Fix stack-use-after-scope --- fairmq/zeromq/Context.h | 1 - test/region/_region.cxx | 2 +- 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/fairmq/zeromq/Context.h b/fairmq/zeromq/Context.h index 8443b7b25..f9ec24ed6 100644 --- a/fairmq/zeromq/Context.h +++ b/fairmq/zeromq/Context.h @@ -94,7 +94,6 @@ class Context { std::unique_lock lock(fMtx); while (fRegionEventsSubscriptionActive) { - while (!fRegionEvents.empty()) { auto i = fRegionEvents.front(); fRegionEventCallback(i); diff --git a/test/region/_region.cxx b/test/region/_region.cxx index cccf43839..d31de36da 100644 --- a/test/region/_region.cxx +++ b/test/region/_region.cxx @@ -110,7 +110,7 @@ void RegionEventSubscriptions(const string& transport) ASSERT_EQ(region2->GetSize(), size2); ASSERT_EQ(factory->SubscribedToRegionEvents(), false); - factory->SubscribeToRegionEvents([&](FairMQRegionInfo info) { + factory->SubscribeToRegionEvents([&, id1, id2, ptr1, ptr2](FairMQRegionInfo info) { LOG(info) << ">>> " << info.event << ": " << (info.managed ? "managed" : "unmanaged") << ", id: " << info.id