Fix LogReader destruction problem
Null pointer exception occurred white calling Deregister on log reader
that does not have any registered nodes. LogReader constructor resizes
states_ to 1, but does not assign the first element.
Change-Id: If446d6cdbef297d9e38842e9bffa213b1dfd2d3c
Signed-off-by: James Kuszmaul <james.kuszmaul@bluerivertech.com>
diff --git a/aos/events/logging/log_reader.cc b/aos/events/logging/log_reader.cc
index 5b437ba..3c46de9 100644
--- a/aos/events/logging/log_reader.cc
+++ b/aos/events/logging/log_reader.cc
@@ -1177,7 +1177,9 @@
// Make sure that things get destroyed in the correct order, rather than
// relying on getting the order correct in the class definition.
for (std::unique_ptr<State> &state : states_) {
- state->Deregister();
+ if (state) {
+ state->Deregister();
+ }
}
event_loop_factory_unique_ptr_.reset();