commit 6d6b228837c9596c8996d531ca7a3a97ff063888
author James Kuszmaul <james.kuszmaul@bluerivertech.com> Wed May 22 09:51:15 2024 -0700
committer James Kuszmaul <james.kuszmaul@bluerivertech.com> Fri Jul 05 12:47:40 2024 -0700
tree 89d5a662715384d4e2dfdf139a6a6111691c2925
parent 1d39146a3b81f80c1d56b0fcd8b4cad879f2c44e

Fix simulated_event_loop_test under asan

We were capturing some variables by reference in a lambda and using them
beyond their lifetimes.

Change-Id: Ie105f3a311014fe513f34eae0c431cb0e2c1e8fc
Signed-off-by: James Kuszmaul <james.kuszmaul@bluerivertech.com>

aos/events/simulated_event_loop_test.cc

diff --git a/aos/events/simulated_event_loop_test.cc b/aos/events/simulated_event_loop_test.cc
index 429d39c..a317db8 100644
--- a/aos/events/simulated_event_loop_test.cc
+++ b/aos/events/simulated_event_loop_test.cc
@@ -1787,6 +1787,8 @@
   const UUID pi2_boot1 = UUID::Random();
   const UUID pi3_boot0 = UUID::Random();
   UUID expected_boot_uuid = pi2_boot0;
+  int boot_number = 0;
+  monotonic_clock::time_point expected_connection_time;
 
   message_bridge::TestingTimeConverter time(
       configuration::NodesCount(&config.message()));
@@ -1853,8 +1855,7 @@
 
   int pi1_server_statistics_count = 0;
   bool first_pi1_server_statistics = true;
-  int boot_number = 0;
-  monotonic_clock::time_point expected_connection_time = pi1->monotonic_now();
+  expected_connection_time = pi1->monotonic_now();
   pi1_remote_timestamp->MakeWatcher(
       "/pi1/aos",
       [&pi1_server_statistics_count, &expected_boot_uuid,